On 14.05.2018 08:54, jor...@apache.org wrote: > Author: jorton > Date: Mon May 14 06:54:55 2018 > New Revision: 1831540 > > URL: http://svn.apache.org/viewvc?rev=1831540&view=rev > Log: > * subversion/tests/cmdline/svntest/main.py > (open_pipe): Run Python tests under the same Python executable for > all platforms rather than just Win32 (rather than finding one from > $PATH via /usr/bin/env). > > Reviewed by: brane > > Modified: > subversion/trunk/subversion/tests/cmdline/svntest/main.py > > Modified: subversion/trunk/subversion/tests/cmdline/svntest/main.py > URL: > http://svn.apache.org/viewvc/subversion/trunk/subversion/tests/cmdline/svntest/main.py?rev=1831540&r1=1831539&r2=1831540&view=diff > ============================================================================== > --- subversion/trunk/subversion/tests/cmdline/svntest/main.py (original) > +++ subversion/trunk/subversion/tests/cmdline/svntest/main.py Mon May 14 > 06:54:55 2018 > @@ -456,9 +456,9 @@ def open_pipe(command, bufsize=-1, stdin > should be passed to wait_on_pipe.""" > command = [str(x) for x in command] > > - # On Windows subprocess.Popen() won't accept a Python script as > - # a valid program to execute, rather it wants the Python executable. > - if (sys.platform == 'win32') and (command[0].endswith('.py')): > + # Always run python scripts under the same Python executable as used > + # for the test suite. > + if command[0].endswith('.py'): > command.insert(0, sys.executable) > > command_string = command[0] + ' ' + ' '.join(map(_quote_arg, command[1:]))
Can you please propose this and related changes you made for backport to 1.10.x? Thanks! -- Brane