Re: PATH manipulation in 001_libpq_pipeline.pl fails on windows
Andrew Dunstan <andrew@dunslane.net>
From: Andrew Dunstan <andrew@dunslane.net>
To: Andres Freund <andres@anarazel.de>, pgsql-hackers@postgresql.org,
Alvaro Herrera <alvherre@2ndquadrant.com>
Date: 2021-10-01T18:07:42Z
Lists: pgsql-hackers
On 9/30/21 5:40 PM, Andres Freund wrote:
> Hi,
>
> For me 001_libpq_pipeline.pl doesn't reliably work on windows, because it
> tries to add something to PATH, using unix syntax (vs ; used on windows).
>
> $ENV{PATH} = "$ENV{TESTDIR}:$ENV{PATH}";
>
> If the first two elements in PATH are something needed, this can cause the
> test to fail... I'm surprised this doesn't cause problems on the buildfarm - a
> plain
> perl src\tools\msvc\vcregress.pl taptest src\test\modules\libpq_pipeline\
> fails for me.
Not sure. That's certainly an error.
But why are we mangling the PATH at all? Wouldn't it be better just to
call command_ok with "$ENV{TESTDIR}/libpg_pipeline" ?
cheers
andrew
--
Andrew Dunstan
EDB: https://www.enterprisedb.com
Commits
-
Reference test binary using TESTDIR in 001_libpq_pipeline.pl.
- 0baa33da0fd1 14.1 landed
- 795862c280c5 15.0 landed