Am I correct to assume Python 2.5, seeing that TM2 is going to be Leopard only? Or is this going into a TM1.x release that should be Tiger compatible? I'm afraid I don't have access to Python 2.3...
Anyway, here is the link to the new file. I can't diff it right now because I'm on a Windows machine, unfortunately. I've made some other minor changes as well (documentation and cleanup).
http://github.com/orestis/pysmell/tree/master/pysmell/tm_dialog.py
On Tue, Nov 4, 2008 at 7:42 AM, Allan Odgaard mailinglist@textmate.org wrote:
On 4 Nov 2008, at 01:44, Orestis Markou wrote:
[...] I uncovered a weird problem when using dialog.py, that is, passing more than 1500 items to tm_dialog through the -p option breaks something. I've hacked together my own tm_dialog.py that uses subprocess to send the plist through stdin, and everything is working again.
I missed you on IRC (CEST) but yes, there is indeed a systme imposed maxium argument size (of about 64 KB), and for that reason, tm_dialog should generally be called with the plist written to its stdin (as you changed it to do).
We'll gladly patch the default dialog.py with your changes, just send the patch here or paste on IRC, and someone will pick it up :)
textmate mailing list textmate@lists.macromates.com http://lists.macromates.com/listinfo/textmate