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
