[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] add new AC_PROG_AR helper
From: |
Zack Weinberg |
Subject: |
Re: [PATCH] add new AC_PROG_AR helper |
Date: |
Mon, 24 Jan 2022 09:35:24 -0500 |
User-agent: |
Cyrus-JMAP/3.5.0-alpha0-4585-ga9d9773056-fm-20220113.001-ga9d97730 |
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.
Did you ever get around to checking whether there was code in automake and/or
libtool that is now redundant to this macro?
zw