[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[sr #110215] AC_EGREP_HEADER appears to be broken in master
From: |
Zack Weinberg |
Subject: |
[sr #110215] AC_EGREP_HEADER appears to be broken in master |
Date: |
Fri, 10 Jul 2020 13:43:50 -0400 (EDT) |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Firefox/68.0 |
Update of sr #110215 (project autoconf):
Status: Need Info => Confirmed
_______________________________________________________
Follow-up Comment #3:
Never mind, I found the discussion on bug-autoconf. This *is* arguably a
regression from 2.69. Copying over the relevant bits of that discussion:
Nick Bowler:
> ... the long-obsolete AC_HEADER_STDC, previously used internally by
> AC_INCLUDES_DEFAULT, used AC_EGREP_HEADER. The AC_HEADER_STDC macro
> is now a no-op (and is not used at all within Autoconf anymore), so
> that change is likely what made the first use of AC_EGREP_HEADER the
> one inside the if condition, causing the observed results.
Eric Blake:
> We already have autoupdate remove a call to AC_HEADER_STDC and
> replace it with the direct side-effect results of setting
> STDC_HEADERS and TIME_WITH_SYS_TIME; should we also tweak that to
> preserve the side-effect of AC_REQUIRES([AC_EGREP_HEADER)], to
> minimize the pain on external projects unaware that they were
> depending on that side-effect? Or maybe just tweak the NEWS blurb to
> mention that users depending on $GREP being set early may need to be
> careful?
Having autoupdate insert AC_REQUIRE([AC_PROG_EGREP]) for AC_HEADER_STDC,
and mentioning this issue in NEWS, both seem like good ideas to me.
_______________________________________________________
Reply to this item at:
<https://savannah.gnu.org/support/?110215>
_______________________________________________
Message sent via Savannah
https://savannah.gnu.org/