From: | Daniel Gustafsson <daniel(at)yesql(dot)se> |
---|---|
To: | Daniel Gustafsson <dgustafsson(at)postgresql(dot)org> |
Cc: | pgsql-committers(at)lists(dot)postgresql(dot)org |
Subject: | Re: pgsql: Remove the adminpack contrib extension |
Date: | 2024-03-04 13:01:03 |
Message-ID: | 0CFB76D0-0510-48B2-9916-1199F93BC28C@yesql.se |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
> On 4 Mar 2024, at 12:43, Daniel Gustafsson <dgustafsson(at)postgresql(dot)org> wrote:
>
> Remove the adminpack contrib extension
The pg_upgrade Xversion test on crake failed on this, I'm trying to reproduce
the cross-version test locally to verify the below diff but it's right now
failing on an unrelated thing. If anyone has a working setup locally to test I
would love the below taken for a spin against something in 16->12.
diff --git a/src/test/perl/PostgreSQL/Test/AdjustUpgrade.pm b/src/test/perl/PostgreSQL/Test/AdjustUpgrade.pm
index c0da0aaff8..8dc1a65e3d 100644
--- a/src/test/perl/PostgreSQL/Test/AdjustUpgrade.pm
+++ b/src/test/perl/PostgreSQL/Test/AdjustUpgrade.pm
@@ -106,6 +106,16 @@ sub adjust_database_contents
'drop extension if exists test_ext7');
}
+ # we removed the adminpack extension in v17
+ if ($old_version >= 12 && $old_version < 17)
+ {
+ _add_st($result, 'regression',
+ 'drop extension if exists adminpack');
+ _add_st($result, 'postgres',
+ 'drop database contrib_regression_adminpack');
+ delete($dbnames{'contrib_regression_adminpack'});
+ }
+
# we removed this test-support function in v17
if ($old_version >= 15 && $old_version < 17)
{
--
Daniel Gustafsson
From | Date | Subject | |
---|---|---|---|
Next Message | Daniel Gustafsson | 2024-03-04 13:39:38 | pgsql: Adjust pg_upgrade crossversion test for adminpack |
Previous Message | Peter Eisentraut | 2024-03-04 12:22:49 | pgsql: Put back required #include |