From da8889ccd7ea0782a22a2300abc8fb801ecfa160 Mon Sep 17 00:00:00 2001 From: Robert Haas Date: Mon, 6 Jul 2026 15:34:12 -0400 Subject: [PATCH] Use PG_MODULE_MAGIC_EXT in newly introduced modules We forgot to use the PG_MODULE_MAGIC_EXT in some newly added modules: pg_plan_advice, pg_stash_advice and the pgrepack output plugin and instead used the older PG_MODULE_MAGIC macro. Author: Andreas Karlsson Discussion: http://postgr.es/m/ad7b910c-d145-4120-994d-2e55c456aa75@proxel.se Backpatch-through: 19 --- contrib/pg_plan_advice/pg_plan_advice.c | 5 ++++- contrib/pg_stash_advice/pg_stash_advice.c | 5 ++++- src/backend/replication/pgrepack/pgrepack.c | 5 ++++- 3 files changed, 12 insertions(+), 3 deletions(-) diff --git a/contrib/pg_plan_advice/pg_plan_advice.c b/contrib/pg_plan_advice/pg_plan_advice.c index 299b0d02a86..7cd753ee171 100644 --- a/contrib/pg_plan_advice/pg_plan_advice.c +++ b/contrib/pg_plan_advice/pg_plan_advice.c @@ -28,7 +28,10 @@ #include "storage/dsm_registry.h" #include "utils/guc.h" -PG_MODULE_MAGIC; +PG_MODULE_MAGIC_EXT( + .name = "pg_plan_advice", + .version = PG_VERSION +); /* GUC variables */ char *pg_plan_advice_advice = NULL; diff --git a/contrib/pg_stash_advice/pg_stash_advice.c b/contrib/pg_stash_advice/pg_stash_advice.c index 1858c6a135a..777ff374599 100644 --- a/contrib/pg_stash_advice/pg_stash_advice.c +++ b/contrib/pg_stash_advice/pg_stash_advice.c @@ -22,7 +22,10 @@ #include "utils/guc.h" #include "utils/memutils.h" -PG_MODULE_MAGIC; +PG_MODULE_MAGIC_EXT( + .name = "pg_stash_advice", + .version = PG_VERSION +); /* Shared memory hash table parameters */ static dshash_parameters pgsa_stash_dshash_parameters = { diff --git a/src/backend/replication/pgrepack/pgrepack.c b/src/backend/replication/pgrepack/pgrepack.c index 959551f5724..5c5095bde4e 100644 --- a/src/backend/replication/pgrepack/pgrepack.c +++ b/src/backend/replication/pgrepack/pgrepack.c @@ -18,7 +18,10 @@ #include "replication/snapbuild.h" #include "utils/memutils.h" -PG_MODULE_MAGIC; +PG_MODULE_MAGIC_EXT( + .name = "pgrepack", + .version = PG_VERSION +); static void repack_startup(LogicalDecodingContext *ctx, OutputPluginOptions *opt, bool is_init);