[systemd-devel] [PATCH 1/2] configure.ac: strip off trailing slashed from $rootprefix

Daniel Mack daniel at zonque.org
Sat May 30 01:28:51 PDT 2015


On 05/30/2015 08:50 AM, Mike Gilbert wrote:
> On Fri, May 29, 2015 at 8:05 PM, Daniel Mack <daniel at zonque.org> wrote:
>> Make sure the variable set via --with-rootprefix= does not contain a
>> trailing slash, so man pages can use entities like "&rootprefix;/lib"
>> without ending up having double slashes.
>> ---
>>  configure.ac | 3 ++-
>>  1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/configure.ac b/configure.ac
>> index 92654a6..55b73de 100644
>> --- a/configure.ac
>> +++ b/configure.ac
>> @@ -1396,7 +1396,8 @@ AC_ARG_WITH([zshcompletiondir],
>>
>>  AC_ARG_WITH([rootprefix],
>>          AS_HELP_STRING([--with-rootprefix=DIR], [rootfs directory prefix for config files and kernel modules]),
>> -        [], [with_rootprefix=${ac_default_prefix}])
>> +        [with_rootprefix=`echo ${withval} | sed -e s,/*$,,`],
>> +        [with_rootprefix=${ac_default_prefix}])
> 
> Why do you pipe it through sed when a simple shell parameter expansion would do?
> 
> with_rootprefix=${withval%/}

Isn't that's a bash'ism which we try to avoid at other places? FWIW, we
use sed to strip off trailing dashes from $host for EFI_ARCH. Also, that
one only replaces one trailing slash, not all of them.

Anyway, Michael's AX_NORMALIZE_PATH approach seems to do the right
thing, so let's go for that.


Thanks,
Daniel



More information about the systemd-devel mailing list