Re: svn commit: r1831540 - /subversion/trunk/subversion/tests/cmdline/svntest/main.py

[Branko Čibej <br...@apache.org>]

On 14.05.2018 08:54, jorton@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