Re: pgsql: Refactor the sslfiles Makefile target for ease of use

From: Daniel Gustafsson <daniel(at)yesql(dot)se>
To: pgsql-committers(at)lists(dot)postgresql(dot)org, Andrew Dunstan <andrew(at)dunslane(dot)net>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Subject: Re: pgsql: Refactor the sslfiles Makefile target for ease of use
Date: 2021-10-20 10:53:26
Message-ID: 0D295F43-806D-4B3F-AB98-F941A19E0271@yesql.se
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

> On 19 Oct 2021, at 20:17, Daniel Gustafsson <dgustafsson(at)postgresql(dot)org> wrote:
>
> Refactor the sslfiles Makefile target for ease of use

This made prairiedog fall over in the SSL tests [0], with what seems a pretty
straightforward reason: 32- vs 64-bit Perl.

Integer overflow in hexadecimal number at t/001_ssltests.pl line 493.
Illegal hexadecimal digit '
' ignored at t/001_ssltests.pl line 493.
Hexadecimal number > 0xffffffff non-portable at t/001_ssltests.pl line 493.

The conversion of the serialnumber from hex to base10 requires a 64-bit capable
Perl, and my guess is that prairiedog has a 32-bit Perl. I installed 5.8.3 in
64-bit mode and verified that it does work there (using the perlbrew recipe in
3eb1f4d097454 which is fantastic).

Attached diff passes tests for me, but I don't have a 32-bit system handy so I
prefer some more eyes on it before pushing. The most portable way I could find
to determine 32/64 bit in Perl was to use Config and check ivsize. It works in
my 5.8.3 installation, but adding Andrew on CC to get a Perl expert opinion on
that.

--
Daniel Gustafsson https://vmware.com/

[0] https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=prairiedog&dt=2021-10-20%2005%3A32%3A46

Attachment Content-Type Size
ssltests_32bit.diff application/octet-stream 898 bytes

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Dilip Kumar 2021-10-20 11:47:02 Re: pgsql: Document XLOG_INCLUDE_XID a little better
Previous Message Amit Kapila 2021-10-20 10:47:26 Re: pgsql: Document XLOG_INCLUDE_XID a little better