Re: pgsql: Refactor background psql TAP functions

From: Daniel Gustafsson <dgustafsson(at)postgresql(dot)org>
To: Damir Belyalov <dam(dot)bel07(at)gmail(dot)com>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Peter Eisentraut <peter(dot)eisentraut(at)enterprisedb(dot)com>, pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: Re: pgsql: Refactor background psql TAP functions
Date: 2023-05-19 21:33:28
Message-ID: 2C39B660-D495-4540-B3CC-BD7BB9B37E57@postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

> On 17 May 2023, at 14:38, Daniel Gustafsson <dgustafsson(at)postgresql(dot)org> wrote:
>
>> On 16 May 2023, at 21:43, Daniel Gustafsson <dgustafsson(at)postgresql(dot)org> wrote:
>>
>>> On 16 May 2023, at 21:04, Damir Belyalov <dam(dot)bel07(at)gmail(dot)com> wrote:
>>
>>> There is another perl module: AdjustUpgrade.pm that is not in Makefile. Do we need it?
>>
>> Yes, I think it should be installed too and skimming the linked thread I see no
>> mention of the omission being deliberate. I will take care of it.
>
> The attached is the proposed fix, which will have to be backpatched to v11 to
> match the original commit. Unless there are reasons to treat this module
> differently from the others wrt installation I'll go ahead and fix it in time
> for beta1.

After another round of thinking I opted out of pushing this, as the module is
so tied to pg_upgrade and the regress env it's use in installed pgxs is
questionable. While I would prefer a 1:1 between the modules in the source
tree and the installed tree, this module doesn't seem like a great fit for
installation.

--
Daniel Gustafsson

In response to

Browse pgsql-committers by date

  From Date Subject
Next Message Bruce Momjian 2023-05-21 01:03:38 pgsql: doc: PG 16 relnotes, misc updates
Previous Message Tom Lane 2023-05-19 21:28:15 pgsql: Add 0245f8db3 to .git-blame-ignore-revs.