perl-cross icon indicating copy to clipboard operation
perl-cross copied to clipboard

unneeded SIG prefix in $Config{sig_name}

Open mstetson opened this issue 8 years ago • 1 comments

The module documentation says that signal names in the Config module's sig_name list do not have a SIG prefix, but cnf/configure_sigs.sh puts one in. This prevents, for example, installing SmokePing using a version of Perl built with perl-cross. Here's a fix:

--- cnf/configure_sigs.sh.orig	2017-02-08 05:33:12.000000000 -0700
+++ cnf/configure_sigs.sh	2017-02-08 05:33:31.000000000 -0700
@@ -29,7 +29,7 @@
 		num=`grep 'number ' try.out | sed -e 's/[^0-9]//g'`
 		if [ -n "$num" -a "$num" != 0 ]; then
 			msg "   got SIG$sig = $num" >&2
-			signals="$signals SIG$sig"
+			signals="$signals $sig"
 			siginit="$siginit, \"$sig\""
 			signums="$signums $num"
 			signumi="$signumi, $num"

mstetson avatar Feb 12 '17 04:02 mstetson

Applied, thanks.

arsv avatar Feb 12 '17 20:02 arsv