Re: Fix vcregress plpython3 warning

From: Andrew Dunstan <andrew(at)dunslane(dot)net>
To: Juan José Santamaría Flecha <juanjo(dot)santamaria(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Fix vcregress plpython3 warning
Date: 2022-01-07 13:30:38
Message-ID: ae44b67e-ab76-f2ca-3539-e8427092a623@dunslane.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers


On 1/7/22 07:20, Juan José Santamaría Flecha wrote:
> Hi,
>
> When running the plcheck in Windows we get the following warning, it
> is visible in the cfbots [1]:
>
> Use of uninitialized value $1 in concatenation (.) or string at
> src/tools/msvc/vcregress.pl <http://vcregress.pl> line 350.
>
> This points to mangle_plpython3 subroutine. The attached patch
> addresses the problem.
>
> [1] http://commitfest.cputube.org/

Yeah, this code is not a model of clarity though. I had to think through
it and I write quite a bit of perl. I would probably write it something
like this:

s/EXTENSION (.*?)plpython2?u/EXTENSION $1plpython3u/g ;

cheers

andrew

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

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Euler Taveira 2022-01-07 13:47:44 Re: row filtering for logical replication
Previous Message Amit Langote 2022-01-07 13:10:23 Re: sqlsmith: ERROR: XX000: bogus varno: 2