Re: [PATCH] add new AC_PROG_AR helper

[Mike Frysinger]

On 24 Jan 2022 09:35, Zack Weinberg wrote:
> On Wed, Jan 19, 2022, at 12:22 AM, Mike Frysinger wrote:
> > On 22 Feb 2021 11:26, Zack Weinberg wrote:
> >> On Mon, Feb 22, 2021 at 11:15 AM Mike Frysinger wrote:
> >> > This mirrors the existing RANLIB and related toolchain macros that
> >> > autoconf already exports.  Some projects assume `ar` is available
> >> > which isn't always safe, so provide a macro that probes the full
> >> > toolchain settings.
> >> >
> >> > This also makes it easier to use AC_REQUIRE with the macro instead
> >> > of duplicating the AC_CHECK_TOOL call in projects.
> >> 
> >> LGTM.  Could you please check whether there is code in automake and/or
> >> libtool that is now redundant to this macro?
> >
> > are there plans to merge this ?
> 
> Sorry about that, I thought you would merge it yourself (do you not have
> commit access for autoconf?)  It's merged now.  I had to add a change to
> tests/local.at to allow AR to be set.

ah, i see, that can be confusing.  i don't have access to autoconf.  am i
supposed to ? :)

> Did you ever get around to checking whether there was code in automake and/or
> libtool that is now redundant to this macro?

i posted some findings:
https://lists.gnu.org/archive/html/autoconf-patches/2021-02/msg00011.html

but i was going to wait for it to be canonical before proposing any patches.
-mike

signature.asc
Description: PGP signature