Re: Perl 5.26 and windows build system

From: Andrew Dunstan <andrew(dot)dunstan(at)2ndquadrant(dot)com>
To: Victor Wagner <vitus(at)wagner(dot)pp(dot)ru>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Perl 5.26 and windows build system
Date: 2018-10-17 21:16:15
Message-ID: 03d5227c-2727-847d-15c0-3a87e0b11f83@2ndQuadrant.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 10/17/2018 04:38 AM, Victor Wagner wrote:
> Colleagues,
>
> Since Active State stopped to distribute perl 5.22, we decided to
> upgrade installer builds to most use recent version available
> (5.26.1.2601 now).
>
> But upstream perl changes policy around this version and no longer
> adds current directory to the module search path.
>
> This doesn't break work of build.pl, which allready handles module
> search path itself, but breaks install.pl, mkvcbuild.pl and
> vcregress.pl, which
> expect to be started from src/tools/msvc.
>
> Simple adding
>
> use lib ".";
>
> to the beginning of these script solves problem.

The buildfarm code uses this pattern:

use File::Spec;

BEGIN
{
    use lib File::Spec->rel2abs(dirname(__FILE__));
}

So that the added path is not the current directory but the one where
the file lives. I think that's probably a better way to do it.

>
> BTW, have anyone experienced some success using Strawberry perl instead
> of Active perl both for building postgres and as PL/Perl engine?
>
> Active State seems to abandon support for 32-bit windows and strawberry
> perl license allows redistribution.

I have not. OTOH I'm not sure I care that much about 32bit Windows
either. I have heard of people having considerable difficulties with
Strawberry Perl.

cheers

andrew

--
Andrew Dunstan https://www.2ndQuadrant.com
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Andrew Dunstan 2018-10-17 21:22:35 Re: Perl 5.26 and windows build system
Previous Message Andres Freund 2018-10-17 21:08:40 Re: pageinspect: add tuple_data_record()