From: Noah Misch <noah@leadboat.com>
To: Andrew Dunstan <andrew@dunslane.net>
Cc: Michael Paquier <michael@paquier.xyz>
Cc: Robert Haas <robertmhaas@gmail.com>
Cc: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Alvaro Herrera <alvherre@alvh.no-ip.org>
Cc: Mark Dilger <mark.dilger@enterprisedb.com>
Cc: PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Subject: Re: Postgres perl module namespace
Date: Tue, 7 Sep 2021 21:00:11 -0700
Message-ID: <20210908040011.GA2476997@rfd.leadboat.com> (raw)
In-Reply-To: <780f82bb-3a25-0c6f-154a-9f24b9324dba@dunslane.net>
References: <3967644.1628688608@sss.pgh.pa.us>
<7b565e73-4adc-ee9f-bc61-34817298978d@dunslane.net>
<CA+TgmobdrGo2PcxLYXr3DLVVHxt-Xp9dmkSsRuXSNRBasgrGxg@mail.gmail.com>
<YSXZt7sJWJEz76F3@paquier.xyz>
<CA+TgmoazCbbtxGEUaFFkArYxq_8r1iwL4kNG+fzJTzcDPRVfBw@mail.gmail.com>
<ce3dc94c-2826-e2b3-42f7-dd0cecf9526c@dunslane.net>
<20210904061949.GA2326091@rfd.leadboat.com>
<da8a9278-3366-4871-4823-b929b3b5e91a@dunslane.net>
<YTWiXYixl/bM0Gft@paquier.xyz>
<780f82bb-3a25-0c6f-154a-9f24b9324dba@dunslane.net>
On Tue, Sep 07, 2021 at 07:43:47AM -0400, Andrew Dunstan wrote:
> On 9/6/21 1:08 AM, Michael Paquier wrote:
> > On Sat, Sep 04, 2021 at 09:58:08AM -0400, Andrew Dunstan wrote:
> >> On 9/4/21 2:19 AM, Noah Misch wrote:
> >>> plperl uses PostgreSQL:: as the first component of its Perl module namespace.
> >>> We shouldn't use both PostgreSQL:: and Postgres:: in the same source tree, so
> >>> this change should not use Postgres::.
> >> Good point. Here's the same thing using PostgreSQL::Test
> > A minor point: this introduces PostgreSQL::Test::PostgresVersion.
> > Could be this stripped down to PostgreSQL::Test::Version instead?
>
> That name isn't very clear - what is it the version of, PostgreSQL or
> the test?
Fair.
> There's nothing very test-specific about this module - it simply
> encapsulates a Postgres version string. So maybe it should just be
> PostgreSQL::Version.
Could be fine, but that name could be useful as a CPAN module. These modules
don't belong on CPAN, so I'd keep PostgreSQL::Test::PostgresVersion. There's
only one reference in the tree, so optimizing that particular name is less
exciting.
(I wondered about using PGXS:: as the namespace for all these modules, since
it's short and "pgxs" is the closest thing to a name for the PostgreSQL build
system. Overall, I didn't convince myself about it being an improvement.)
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: noah@leadboat.com, andrew@dunslane.net, michael@paquier.xyz, robertmhaas@gmail.com, tgl@sss.pgh.pa.us, alvherre@alvh.no-ip.org, mark.dilger@enterprisedb.com, pgsql-hackers@lists.postgresql.org
Subject: Re: Postgres perl module namespace
In-Reply-To: <20210908040011.GA2476997@rfd.leadboat.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