[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] hooks-scripts: fix svn2feed.py on Windows

From: Peter Samuelson <peter_at_p12n.org>
Date: Sat, 28 Jun 2008 15:46:11 -0500

[Niels de Vos]
> It seems that popen() does not allow a list as first parameter. The
> Python-docs explicitly note that on UNIX a sequence can be passed,
> Windows is not mentioned.

How annoying. By passing a list, you don't have to worry about quoting
your arguments, to account for spaces and such in filenames. Your
patch will not work, for example, if a repository path is under
C:\Documents and Settings, or if svnlook_cmd is under C:\Program Files.

I suggest doing something along the lines of

  cmd = '"%s" info -r%s "%s"' % svnlook_cmd, revision, self.repos_path

Peter Samuelson | org-tld!p12n!peter | http://p12n.org/

Received on 2008-06-28 22:46:25 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.