On 10/6/07, Lieven Govaerts <firstname.lastname@example.org> wrote:
> Eric Gillespie wrote:
> > email@example.com writes:
> >> Follow up r26920, fix update_tests.py 38: on Windows, using the
> >> apr calls svn can only execute external applications that are
> >> considered executable (.bat, .exe etc). So when we want svn to
> >> call .py scripts we have to use the same trick as for the hook
> >> scripts, ie. add a batch file to wrap the .py file.
> > Huh. I only added a new use in update_tests. How did the rest
> > of these pass? Have these tests been subtly broken on Windows
> > all this time?
> > commit_tests.py:2271: svntest.main.use_editor('append_foo')
> > prop_tests.py:71: svntest.main.use_editor('foo_to_bar')
> > prop_tests.py:91: svntest.main.use_editor('identity')
> > prop_tests.py:1190: svntest.main.use_editor('foo_to_bar')
> > prop_tests.py:1210: svntest.main.use_editor('identity')
> No they haven't.
> The problem lies in the --accept=launch test. For all other testcases in
> update_accept_conflicts and for the tests you mention above, Subversion
> is only calling the svneditor.py script once through a system call.
> Example: svn_cl__edit_file_externally in svn/util.c.
> For the --accept=launch test, subversion is also launching a separate
> executable, not through a system call, but through an apr call in
> svn_cl__merge_file_externally in svn/util.c.
> The system call starts up a shell, so on Windows it can run whatever
> scripts you have as executable on your system.
> I don't know why the other call uses the apr function, I assume because
> it needs to know the return value of the script.
Really, we should only be using apr calls, but for historical reasons
$SVN_EDITOR needs to support 'commands' which contain spaces and
flags, which means shell parsing (which means thinking carefully about
escaping, which sucks).
David Glasser | glasser_at_davidglasser.net | http://www.davidglasser.net/
To unsubscribe, e-mail: firstname.lastname@example.org
For additional commands, e-mail: email@example.com
Received on Sun Oct 7 08:28:22 2007