pgsql: Provide for testing on python3 modules when under MSVC

Started by Andrew Dunstanabout 8 years ago6 messagescomitters
Jump to latest
#1Andrew Dunstan
andrew@dunslane.net

Provide for testing on python3 modules when under MSVC

This should have been done some years ago as promised in commit
c4dcdd0c2. However, better late than never.

Along the way do a little housekeeping, including using a simpler test
for the python version being tested, and removing a redundant subroutine
parameter. These changes only apply back to release 9.5.

Backpatch to all live releases.

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/966268c7621c4bca534961440b497a3270395fc2

Modified Files
--------------
src/tools/msvc/Install.pm | 8 +---
src/tools/msvc/vcregress.pl | 94 +++++++++++++++++++++++++++++++--------------
2 files changed, 68 insertions(+), 34 deletions(-)

#2Andrew Dunstan
andrew@dunslane.net
In reply to: Andrew Dunstan (#1)
Re: pgsql: Provide for testing on python3 modules when under MSVC

On Fri, May 4, 2018 at 3:58 PM, Andrew Dunstan <andrew@dunslane.net> wrote:

Provide for testing on python3 modules when under MSVC

This should have been done some years ago as promised in commit
c4dcdd0c2. However, better late than never.

Along the way do a little housekeeping, including using a simpler test
for the python version being tested, and removing a redundant subroutine
parameter. These changes only apply back to release 9.5.

Backpatch to all live releases.

Looks like this broke. I know I tested it. I'll be back in a couple of
hours to fix it.

cheers

andrew

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

#3Tom Lane
tgl@sss.pgh.pa.us
In reply to: Andrew Dunstan (#1)
Re: pgsql: Provide for testing on python3 modules when under MSVC

Andrew Dunstan <andrew@dunslane.net> writes:

Provide for testing on python3 modules when under MSVC

The MSVC critters in the buildfarm seem not to like this.

regards, tom lane

#4Andrew Dunstan
andrew@dunslane.net
In reply to: Tom Lane (#3)
Re: pgsql: Provide for testing on python3 modules when under MSVC

On May 4, 2018, at 7:03 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:

Andrew Dunstan <andrew@dunslane.net> writes:

Provide for testing on python3 modules when under MSVC

The MSVC critters in the buildfarm seem not to like this.

Will fix in an hour or two

Cheers

Andrew

#5Andrew Dunstan
andrew@dunslane.net
In reply to: Tom Lane (#3)
Re: pgsql: Provide for testing on python3 modules when under MSVC

On Fri, May 4, 2018 at 7:03 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:

Andrew Dunstan <andrew@dunslane.net> writes:

Provide for testing on python3 modules when under MSVC

The MSVC critters in the buildfarm seem not to like this.

It appears that the three failing critters (whelk, thrips and
woodlouse) are misconfigured, in that they don't have the Python
directory in their PATH setting, so of course plpython3u won't load,
as it won't be able to locate the Python DLL. This hasn't mattered up
to now as they haven't been doing any testing, but it's exposed by our
enabling testing.

On its latest run dory, which also builds with Python3, has passed all
the python tests, so it doesn't (yet) appear that there is anything
wrong with the patch.

cheers

andrew

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

#6Christian Ullrich
chris@chrullrich.net
In reply to: Andrew Dunstan (#5)
RE: pgsql: Provide for testing on python3 modules when under MSVC

* Andrew Dunstan wrote:

On Fri, May 4, 2018 at 7:03 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:

Andrew Dunstan <andrew@dunslane.net> writes:

Provide for testing on python3 modules when under MSVC

The MSVC critters in the buildfarm seem not to like this.

It appears that the three failing critters (whelk, thrips and
woodlouse) are misconfigured, in that they don't have the Python
directory in their PATH setting, so of course plpython3u won't load,

Oops. Fixed.

Thanks for pointing this out.

--
Christian