bug-gnulib
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: gnulib-tool.py: Don't default to 'build-aux' for --auxdir.


From: Bruno Haible
Subject: Re: gnulib-tool.py: Don't default to 'build-aux' for --auxdir.
Date: Tue, 02 Apr 2024 14:22:02 +0200

Hi Collin,

> lines 7251-7274 in gnulib-tool.sh:
> 
>     # Analyze configure.ac.
>     guessed_auxdir="."
>     guessed_libtool=false
>     ...
>       /AC_CONFIG_AUX_DIR/ {
>         s,^.*AC_CONFIG_AUX_DIR([[ ]*\([^]"$`\\)]*\).*$,guessed_auxdir="\1",p
>       }
>     ...
>     if test -z "$auxdir"; then
>       auxdir="$guessed_auxdir"
>     fi
> 
> I think that this means if no --auxdir is passed to gnulib-tool, use
> whatever is found in 'AC_CONFIG_AUX_DIR' of configure.ac. If not found
> use the destination directory.

Yes, that's what it means.

> It seems that GLConfig.__getitem__(), used for indexing [1] would
> always return 'build-aux' when given the 'auxdir' key.

Not exactly. What this code did:

        if y in self.table:
            if y == "auxdir":
                if self.table['auxdir']:
                    return self.table['auxdir']
                return "build-aux"
            return self.table[y]

is to return 'build-aux' instead of returning ''.

> This seems correct to me since all tests still pass and in freedink:
> 
>     env GNULIB_TOOL_IMPL=sh+py gnulib-tool ./bootstrap
> 
> works correctly.

Thanks! Applied, with a corrected ChangeLog wording.

Bruno






reply via email to

[Prev in Thread] Current Thread [Next in Thread]