On Sun, Apr 19, 2026 at 12:19 AM Bruno Haible via GNU gzip discussion and
bug reports. <[email protected]> wrote:

> Paul Eggert wrote:
> > Come to think of it, that new use of AC_REQUIRE([AC_CANONICAL_HOST]) at
> > gzip/configure.ac's is problematic.
>
> Yes, I agree.
>
> > I guess it works because Gnulib has
> > already called AC_CANONICAL_HOST by then, but we shouldn't assume that
> > without checking.
>
> Maybe a better way would have been to wrap this code in an AC_DEFUN
> and invoke it just immediately afterwards. This way, one can still use
> AC_REQUIRE.
>
> > I worked around the issue by instead checking that
> > Gnulib has called AC_CANONICAL_HOST, and by using AS_CASE rather than
> > 'case' while I was in the neighborhood.
>
> Unfortunately, there's a typo here: The output in the configure file
> looks like this. Note the "Avoid a crash" line:
>
>    case $DEFS in #(
>   *NO_ASM*) :
>      ;; #(
>   *) :
>     # 'host_os?' ensures Gnulib sets host_os as usual.
>       case ${host_os?} in #(
>   *-musl* | midipix*) :
>     Avoid a crash in musl libc's startup code,
>            # because musl doesn't support TEXTRELs.
>            # https://www.openwall.com/lists/musl/2020/09/25/1
>            ;; #(
>   *) :
>     if cp $srcdir/lib/match.c _match.S &&
>
> Here's a fix.


Thanks for the fix. Pushed.

Reply via email to