Re: pgsql: Remove the adminpack contrib extension - Mailing list pgsql-committers

From Daniel Gustafsson
Subject Re: pgsql: Remove the adminpack contrib extension
Date
Msg-id 0CFB76D0-0510-48B2-9916-1199F93BC28C@yesql.se
Whole thread Raw
In response to pgsql: Remove the adminpack contrib extension  (Daniel Gustafsson <dgustafsson@postgresql.org>)
Responses Re: pgsql: Remove the adminpack contrib extension
List pgsql-committers
> On 4 Mar 2024, at 12:43, Daniel Gustafsson <dgustafsson@postgresql.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




pgsql-committers by date:

Previous
From: Peter Eisentraut
Date:
Subject: pgsql: Put back required #include
Next
From: Daniel Gustafsson
Date:
Subject: pgsql: Adjust pg_upgrade crossversion test for adminpack