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

Re: [PATCH] svneditor.bat

From: Daniel Shahaf <d.s_at_daniel.shahaf.co.il>
Date: Mon, 21 Apr 2008 09:20:58 +0300

Mark Reibert wrote on Sun, 20 Apr 2008 at 22:52 -0700:
> On Mon, 2008-04-21 at 01:48 -0400, Karl Fogel wrote:
> > Daniel Shahaf <d.s_at_daniel.shahaf.co.il> writes:
> > >
> > > Index: subversion/tests/cmdline/svneditor.bat
> > > ===================================================================
> > > --- subversion/tests/cmdline/svneditor.bat (revision 30700)
> > > +++ subversion/tests/cmdline/svneditor.bat (working copy)
> > > @@ -1,5 +1,5 @@
> > > @echo off
> > > rem the svneditor.py script is expected to be in the same directory as the
> > > rem .bat file
> > > -%~dp0\svneditor.py %*
> > > +python %~dp0\svneditor.py %*
>
> Is it safe to assume python will be in %PATH% ?
>

If *.py files are (currently) runnable by saying their name, I think it's
reasonable to assume that python.exe is in the PATH. As I said
originally, I think the instructions in INSTALL imply that python.exe is
in PATH, too.

I do not think that associating *.py files to Python is a safe default in
the first place; I stopped doing that after one day I opened a *.py file
attachment and -- due to that association -- it was run (by Python)
instead of opened in a text editor.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-04-22 08:21:33 CEST

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