From 60c07719e78c69112f6ca793ec09b7237c79d10d Mon Sep 17 00:00:00 2001 From: Nils Dijk Date: Fri, 22 Sep 2023 16:24:18 +0200 Subject: [PATCH] Use the $(DLSUFFIX) instead of hard coded extensions for cdc (#7221) When cdc got added the makefiles hardcoded the `.so` extension instead of using the platform specifc `$(DLSUFFIX)` variable used by `pgxs.mk`. Also don't remove installed cdc artifacts on `make clean`. --- src/backend/distributed/cdc/Makefile.decoder | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/src/backend/distributed/cdc/Makefile.decoder b/src/backend/distributed/cdc/Makefile.decoder index 963f8bf15..8548cef47 100644 --- a/src/backend/distributed/cdc/Makefile.decoder +++ b/src/backend/distributed/cdc/Makefile.decoder @@ -14,11 +14,6 @@ override CPPFLAGS += -DDECODER=\"$(DECODER)\" -I$(citus_abs_top_srcdir)/include install: install-cdc -clean: clean-cdc - install-cdc: mkdir -p '$(citus_decoders_dir)' - $(INSTALL_SHLIB) citus_$(DECODER).so '$(citus_decoders_dir)/$(DECODER).so' - -clean-cdc: - rm -f '$(DESTDIR)$(datadir)/$(datamoduledir)/citus_decoders/$(DECODER).so' + $(INSTALL_SHLIB) citus_$(DECODER)$(DLSUFFIX) '$(citus_decoders_dir)/$(DECODER)$(DLSUFFIX)'