From: Justin Pryzby <pryzby@telsasoft.com>
To: Andrew Dunstan <andrew@dunslane.net>
Cc: Jehan-Guillaume de Rorthais <jgdr@dalibo.com>
Cc: pgsql-hackers@postgresql.org
Subject: Re: Fix proposal for comparaison bugs in PostgreSQL::Version
Date: Thu, 3 Nov 2022 13:11:18 -0500
Message-ID: <20221103181118.GX16921@telsasoft.com> (raw)
In-Reply-To: <ea18cfc3-7f8e-8975-b095-226b1865e2e7@dunslane.net>
References: <20220628225325.53d97b8d@karst>
<ea18cfc3-7f8e-8975-b095-226b1865e2e7@dunslane.net>
On Tue, Jun 28, 2022 at 06:17:40PM -0400, Andrew Dunstan wrote:
> Nice catch, but this looks like massive overkill. I think we can very
> simply fix the test in just a few lines of code, instead of a 190 line
> fix and a 130 line TAP test.
>
> It was never intended to be able to compare markers like rc1 vs rc2, and
> I don't see any need for it. If you can show me a sane use case I'll
> have another look, but right now it seems quite unnecessary.
>
> Here's my proposed fix.
>
> diff --git a/src/test/perl/PostgreSQL/Version.pm b/src/test/perl/PostgreSQL/Version.pm
> index 8f70491189..8d4dbbf694 100644
> --- a/src/test/perl/PostgreSQL/Version.pm
Is this still an outstanding issue ?
--
Justin
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-hackers@postgresql.org
Cc: pryzby@telsasoft.com, andrew@dunslane.net, jgdr@dalibo.com
Subject: Re: Fix proposal for comparaison bugs in PostgreSQL::Version
In-Reply-To: <20221103181118.GX16921@telsasoft.com>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox