Re: Converting svnperms.py to use subprocess (so it works on !unix)

On Mon, 2008-09-08 at 10:07 +0000, Trent Nelson wrote:
> If you try and use svnperms.py on Windows, you get a pretty funky
> error:
[...]
> The commands module, which svnperms.py uses to run underlying
> svnlook commands, only works on Unix. I'd like to convert it
> to use subprocess instead, which should make it a lot more
> platform independent than it currently is.
>
> Before I do, just wanted to ping the list to see if I'm missing
> something obvious.

Well, the "subprocess" module only exists since Python 2.4. That would
increase the minumum required version of Python for this script from 2.2
(at a guess) to 2.4. That may well be the best thing to do, but I don't
know.

Other than that, it sounds like an obvious and worthwhile win.

- Julian

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-09-08 16:58:46 CEST