[PATCH] plperl: use xsubpp -output unconditionally

Поиск
Список
Период
Сортировка
От Dagfinn Ilmari Mannsåker
Тема [PATCH] plperl: use xsubpp -output unconditionally
Дата
Msg-id 87tt3u3bup.fsf@wibble.ilmari.org
обсуждение исходный текст
Ответы Re: [PATCH] plperl: use xsubpp -output unconditionally
Список pgsql-hackers
Hi hackers,

I noticed that the PL/Perl makefile makes the use of the -output option
conditional on coverage+vpath, because that option is not supported
until Perl 5.9.3.  But these days we require Perl 5.14, so we can remove
that conditional (which also matches what Meson does).  Please see the
attached.

- ilmari

From cb22019c0f9d42f2dffd7399594192458420f4a7 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Dagfinn=20Ilmari=20Manns=C3=A5ker?= <ilmari@ilmari.org>
Date: Wed, 2 Jul 2025 15:22:21 +0100
Subject: [PATCH] Make use of xsubpp -output unconditional

It was kept conditional on coverage+vpath because Perl before 5.9.3
didn't support it, but we now require 5.14, so that's unnecessary.

This also matches what Meson does.
---
 src/pl/plperl/GNUmakefile | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/src/pl/plperl/GNUmakefile b/src/pl/plperl/GNUmakefile
index 558c764aadb..a0c59cd8e27 100644
--- a/src/pl/plperl/GNUmakefile
+++ b/src/pl/plperl/GNUmakefile
@@ -91,13 +91,7 @@ all: all-lib
 
 %.c: %.xs
     @if [ x"$(perl_privlibexp)" = x"" ]; then echo "configure switch --with-perl was not specified."; exit 1; fi
-# xsubpp -output option is required for coverage+vpath, but requires Perl 5.9.3
-ifeq ($(enable_coverage)$(vpath_build),yesyes)
     $(PERL) $(XSUBPPDIR)/ExtUtils/xsubpp -typemap $(perl_privlibexp)/ExtUtils/typemap -output $@ $<
-else
-    $(PERL) $(XSUBPPDIR)/ExtUtils/xsubpp -typemap $(perl_privlibexp)/ExtUtils/typemap $< >$@
-endif
-
 
 install: all install-lib install-data
 
-- 
2.50.0


В списке pgsql-hackers по дате отправления: