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

Re: [Svnmerge] [PATCH] ver2: remove progname hardcoding in svnmerge,py

From: Giovanni Bajo <rasky_at_develer.com>
Date: 2006-03-31 16:18:43 CEST

Madan U S <madan@collab.net> wrote:

> My previous patch assumes that svnmerge is called as 'svnmerge command'
> from the commandline. However, when I use svnmerge, it typically is
> 'path/to/svnmerge.py command'. The attached patch takes care of all
> variations of using path with actual .py extension, or just 'svnmerge'
from
> command line.

This will crash Windows users which run the program with "/path/to/svnmerge"
(no .py extension), after having registered the .py files. It will also
crash for users which use the binary self-contained version (svnmerge.exe,
which requires no existing Python installation).

I'm not sure why you're going through this in the first place. If we ever
want to rename svnmerge.py to SvnUberMerge, it's just one line change.
Python is known for having incompatibilities problem with argv[0] across
platforms, so it looks like we're facing gratuitous problems.

-- 
Giovanni Bajo
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Mar 31 16:19:33 2006

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.