Re: multi-install PostgresNode

On 1/28/21 9:24 AM, Alvaro Herrera wrote:
> On 2021-Jan-28, Andrew Dunstan wrote:
>
> ... neat stuff, thanks.
>
>> +            # Windows picks up DLLs from the PATH rather than *LD_LIBRARY_PATH
>> +            # choose the right path separator
>> +            if ($Config{osname} eq 'MSWin32')
>> +            {
>> +               $ENV{PATH} = "$inst/bin;$inst/lib;$ENV{PATH}";
>> +            }
>> +            else
>> +            {
>> +               $ENV{PATH} = "$inst/bin:$inst/lib:$ENV{PATH}";
>> +            }
> Hmm, if only Windows needs lib/ in PATH, then we do we add $inst/lib to
> PATH even when not Windows?



We could, but there's no point AFAICS. *nix dynamic loaders don't use
the PATH on any platform to my knowledge. This is mainly so that Windows
will find libpq.dll correctly.



>
>> +            if (exists $ENV{DYLIB})
>> +            {
>> +                $ENV{DYLIB} = "$inst/lib:$ENV{DYLIB}";
>> +            }
>> +            else
>> +            {
>> +                $ENV{DYLIB} = "$inst/lib}";
> Note extra closing } in the string here.


Oops. fixed, thanks


cheers


andrew

--
Andrew Dunstan
EDB: https://www.enterprisedb.com