[SVN] r8569 (TextMate)
Allan Odgaard
throw-away-2 at macromates.com
Fri Nov 30 02:02:08 UTC 2007
• allow pasting to be aborted
Conceptually this was just giving a “kill curl” lambda block to the call_with_progress, but effectively turned out to be more involved, since any shell execution done in the call_with_progress block caused the cancel lambda block to not to be called, so no calling out to `mktemp`, and the find_ext was indirectly calling `ps` (to find the TM app path). So I had to change these things.
Additionally `curl` is no longer called with %x{…} (since we need a pid to kill, well, actually, I made it kill all processes with pgid matching the current pid, since at the time of providing the cancel block, curl’s pid is not known) — but since the result from curl is needed, handling curl’s stdout (by creating a pipe) was required…
Anyway, it works :) Now we just need some graceful error reporting for when pasting fails.
Changed:
U trunk/Bundles/TextMate.tmbundle/Commands/Paste Selection : Line Online….tmCommand
More information about the textmate-dev
mailing list