Re: pgsql: Fix included file path for modern perl

From: Andrew Dunstan <andrew(at)dunslane(dot)net>
To: Andres Freund <andres(at)anarazel(dot)de>
Cc: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: Re: pgsql: Fix included file path for modern perl
Date: 2019-02-06 10:55:12
Message-ID: 813adda8-ffdc-d8c9-46d9-99aa4a2105ae@dunslane.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers


On 2/6/19 4:27 AM, Andres Freund wrote:
> Hi,
>
> On 2019-02-06 00:36:07 +0000, Andrew Dunstan wrote:
>> Fix included file path for modern perl
>>
>> Contrary to the comment on 772d4b76, only paths starting with "./" or
>> "../" are considered relative to the current working directory by perl's
>> "do" function. So this patch converts all the relevant cases to use "./"
>> paths. This only affects MSVC.
>>
>> Backpatch to all live branches.
> Hm, is it possible this broke
> https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=whelk&dt=2019-02-06%2002%3A41%3A15
> and
> https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=dory&dt=2019-02-06%2000%3A45%3A25
> both report something like
> Use of uninitialized value in multiplication (*) at src/tools/msvc/Solution.pm line 181, <$i> line 783.
> Use of uninitialized value in division (/) at src/tools/msvc/Solution.pm line 184, <$i> line 783.
> Illegal division by zero at src/tools/msvc/Solution.pm line 184, <$i> line 783.
>
> starting with this commit.
>

Yes, investigating ...

cheers

andrew

In response to

Browse pgsql-committers by date

  From Date Subject
Next Message Andrew Dunstan 2019-02-06 12:56:27 pgsql: Unify searchpath and do file logic in MSVC build scripts.
Previous Message Andres Freund 2019-02-06 09:27:15 Re: pgsql: Fix included file path for modern perl