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