public inbox for newlib@sourceware.org
 help / color / mirror / Atom feed
* [PATCH] libgloss: switch to AM_PROG_AR
@ 2022-02-25  4:44 Mike Frysinger
  2022-02-25 15:11 ` Corinna Vinschen
  0 siblings, 1 reply; 2+ messages in thread
From: Mike Frysinger @ 2022-02-25  4:44 UTC (permalink / raw)
  To: newlib

Now that we require Automake 1.15, we can use this macro rather than
set the tool up ourselves.  The current code doesn't properly search
for a prefixed ar tool as-is.
---
 libgloss/Makefile.in          |   1 +
 libgloss/aarch64/aclocal.m4   | 112 ++++++++++++++++++++++
 libgloss/aarch64/configure    | 176 +++++++++++++++++++++++++++++++++-
 libgloss/aarch64/configure.ac |   3 +-
 libgloss/aclocal.m4           |  60 ++++++++++++
 libgloss/arm/aclocal.m4       | 112 ++++++++++++++++++++++
 libgloss/arm/configure        | 176 +++++++++++++++++++++++++++++++++-
 libgloss/arm/configure.ac     |   3 +-
 libgloss/configure            | 144 ++++++++++++++++++++++++++--
 libgloss/configure.ac         |   4 +-
 10 files changed, 776 insertions(+), 15 deletions(-)

diff --git a/libgloss/aarch64/configure.ac b/libgloss/aarch64/configure.ac
index cd91c2212d05..d691f11a714c 100644
--- a/libgloss/aarch64/configure.ac
+++ b/libgloss/aarch64/configure.ac
@@ -11,8 +11,7 @@ AC_PROG_INSTALL
 AC_NO_EXECUTABLES
 AC_PROG_CC
 AM_PROG_AS
-AR=${AR-ar}
-AC_SUBST(AR)
+AM_PROG_AR
 LD=${LD-ld}
 AC_SUBST(LD)
 AC_PROG_RANLIB
diff --git a/libgloss/arm/configure.ac b/libgloss/arm/configure.ac
index 2affeb9716c9..44f5f73962e4 100644
--- a/libgloss/arm/configure.ac
+++ b/libgloss/arm/configure.ac
@@ -31,8 +31,7 @@ AC_PROG_INSTALL
 AC_NO_EXECUTABLES
 AC_PROG_CC
 AM_PROG_AS
-AR=${AR-ar}
-AC_SUBST(AR)
+AM_PROG_AR
 LD=${LD-ld}
 AC_SUBST(LD)
 AC_PROG_RANLIB
diff --git a/libgloss/configure.ac b/libgloss/configure.ac
index a656fe1853fb..43e7916631e0 100644
--- a/libgloss/configure.ac
+++ b/libgloss/configure.ac
@@ -251,9 +251,7 @@ AM_CONDITIONAL(CONFIG_LIBNOSYS, test x$config_libnosys = xtrue)
 
 AC_PROG_CC
 AM_PROG_AS
-AC_CHECK_PROG(AR, ar, ar, :)
-AR=${AR-ar}
-AC_SUBST(AR)
+AM_PROG_AR
 LD=${LD-ld}
 AC_SUBST(LD)
 AC_PROG_RANLIB
-- 
2.34.1


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] libgloss: switch to AM_PROG_AR
  2022-02-25  4:44 [PATCH] libgloss: switch to AM_PROG_AR Mike Frysinger
@ 2022-02-25 15:11 ` Corinna Vinschen
  0 siblings, 0 replies; 2+ messages in thread
From: Corinna Vinschen @ 2022-02-25 15:11 UTC (permalink / raw)
  To: newlib

On Feb 24 23:44, Mike Frysinger wrote:
> Now that we require Automake 1.15, we can use this macro rather than
> set the tool up ourselves.  The current code doesn't properly search
> for a prefixed ar tool as-is.
> ---
>  libgloss/Makefile.in          |   1 +
>  libgloss/aarch64/aclocal.m4   | 112 ++++++++++++++++++++++
>  libgloss/aarch64/configure    | 176 +++++++++++++++++++++++++++++++++-
>  libgloss/aarch64/configure.ac |   3 +-
>  libgloss/aclocal.m4           |  60 ++++++++++++
>  libgloss/arm/aclocal.m4       | 112 ++++++++++++++++++++++
>  libgloss/arm/configure        | 176 +++++++++++++++++++++++++++++++++-
>  libgloss/arm/configure.ac     |   3 +-
>  libgloss/configure            | 144 ++++++++++++++++++++++++++--
>  libgloss/configure.ac         |   4 +-
>  10 files changed, 776 insertions(+), 15 deletions(-)

ACK


Thx,
Corinna


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2022-02-25 15:11 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-02-25  4:44 [PATCH] libgloss: switch to AM_PROG_AR Mike Frysinger
2022-02-25 15:11 ` Corinna Vinschen

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).