From: "Regina Obe" <lr@pcorp.us>
To: "'Gregory Stark \(as CFM\)'" <stark.cfm@gmail.com>,
	"'Sandro Santilli'" <strk@kbt.io>
Cc: "'Tom Lane'" <tgl@sss.pgh.pa.us>,
	"'Regina Obe'" <r@pcorp.us>,
	<pgsql-hackers@lists.postgresql.org>
References: <YgakFklJyM5pNdt+@c19> <20221117095734.igldlk6kngr6ogim@c19> <166914379479.1121.7549798686571352890.pgcf@coridan.postgresql.org> <55512.1673304709@sss.pgh.pa.us> <20230110205259.htcuhg3t7shd367d@c19> <387183.1673394631@sss.pgh.pa.us> <000201d92572$7dcd8260$79688720$@pcorp.us> <20230111101636.ybho2vesvatwmqyl@c19> <CAM-w4HPYe0qVmRO36FrXULk+73Mub-ubG1D3V4jRhZQAvieucQ@mail.gmail.com>
In-Reply-To: <CAM-w4HPYe0qVmRO36FrXULk+73Mub-ubG1D3V4jRhZQAvieucQ@mail.gmail.com>
Subject: RE: [PATCH] Support % wildcard in extension upgrade filenames
Date: Mon, 6 Mar 2023 16:37:39 -0500
Message-ID: <000901d95073$e0c5a290$a250e7b0$@pcorp.us>
MIME-Version: 1.0
Content-Type: text/plain;
	charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Thread-Index: AQGYmaaUCwtlrmE55CHyCSUXu9VHkgKopfE4AVtxnv8DArAuyQGehsdEA0I4l/UBwAxw9QKlZ8tAAh6woguu3FY8AA==
Content-Language: en-us
Archived-At: <https://www.postgresql.org/message-id/000901d95073%24e0c5a290%24a250e7b0%24%40pcorp.us>
Precedence: bulk

> This patch too is conflicting on meson.build. Are these two patches
> interdependent?
>=20
> This one looks a bit more controversial. I'm not sure if Tom has been
> convinced and I haven't seen anyone else speak up.
>=20
> Perhaps this needs a bit more discussion of other options to solve =
this issue.
> Maybe it can just be solved with multiple one-line scripts that call =
to a master
> script?
>=20
> --
> Gregory Stark
> As Commitfest Manager

They edit the same file yes so yes conflicts are expected. =20
The wildcard one, Tom was not convinced, so I assume we'll need to=20
go a couple more rounds on it and hopefully we can get something that =
will not be so controversial.

I don't think the schema qualifying required extension feature is =
controversial though so I think that should be able to go and we'll =
rebase our other patch after that.

Thanks,
Regina