In perl.git, the branch blead has been updated <https://perl5.git.perl.org/perl.git/commitdiff/b43c708ad6587be0945a81b1f04142f110ecb471?hp=c8abed4aa5f30e504657d68bbf7b480e264f575f>
- Log ----------------------------------------------------------------- commit b43c708ad6587be0945a81b1f04142f110ecb471 Author: H.Merijn Brand <h.m.br...@xs4all.nl> Date: Thu Jul 25 15:53:40 2019 +0200 Remove trailing '/' from prefix Change how/when the trailing '/' from a prefix is removed. The original code had two problems: 1) it only stripped the trailing slash when value was predefined (either from -Dprefix=/foo/bar/ or from a previous configure run) 2) when it did strip the value then it also caused 'oldprefix' to be set. This in turns causes other predefined (/cached) paths to be ignored. See https://rt.perl.org/Public/Bug/Display.html?id=134284 ----------------------------------------------------------------------- Summary of changes: Configure | 28 ++++++++++++++++------------ 1 file changed, 16 insertions(+), 12 deletions(-) diff --git a/Configure b/Configure index 5c022556d0..3ab45a4c4b 100755 --- a/Configure +++ b/Configure @@ -4169,14 +4169,9 @@ EOSC : determine root of directory hierarchy where package will be installed. case "$prefix" in -'') - dflt=`./loc . /usr/local /usr/local /local /opt /usr` - ;; -*?/) - dflt=`echo "$prefix" | sed 's/.$//'` +'') dflt=`./loc . /usr/local /usr/local /local /opt /usr` ;; -*) - dflt="$prefix" +*) dflt="$prefix" ;; esac $cat <<EOM @@ -4195,16 +4190,25 @@ rp='Installation prefix to use?' . ./getfile oldprefix='' case "$prefix" in -'') ;; -*) - case "$ans" in +'') ;; +*) case "$ans" in "$prefix") ;; *) oldprefix="$prefix";; esac ;; esac -prefix="$ans" -prefixexp="$ansexp" + +case "$ans" in +*?/) prefix=`echo "$ans" | sed 's/.$//'` + ;; +*) prefix="$ans" +esac + +case "$ansexp" in +*?/) prefixexp=`echo "$ansexp" | sed 's/.$//'` + ;; +*) prefixexp="$ansexp" +esac : allow them to override the AFS root case "$afsroot" in -- Perl5 Master Repository