[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 2/2] configure: rename AR_FLAGS to ARFLAGS
From: |
arnold |
Subject: |
Re: [PATCH 2/2] configure: rename AR_FLAGS to ARFLAGS |
Date: |
Sun, 31 Oct 2021 01:58:08 -0600 |
User-agent: |
Heirloom mailx 12.5 7/5/10 |
Thank you for the patches. I will get them integrated.
In the future, please include ChangeLog entries in your diffs as well.
Thanks!
Arnold
Sam James <sam@gentoo.org> wrote:
> ARFLAGS is more idiomatic/the canonical variable
> for controlling flags passed to AR.
>
> Signed-off-by: Sam James <sam@gentoo.org>
> ---
> ChangeLog | 1 +
> configure | 2 +-
> configure.ac | 2 +-
> 3 files changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/ChangeLog b/ChangeLog
> index d2d7b954..bd1463b1 100644
> --- a/ChangeLog
> +++ b/ChangeLog
> @@ -1,6 +1,7 @@
> 2021-10-29 Sam James <sam@gentoo.org>
>
> * configure.ac: Fix AR_FLAGS assignment typo.
> + * configure.ac: Rename AR_FLAGS to ARFLAGS.
>
> 2021-10-27 Arnold D. Robbins <arnold@skeeve.com>
>
> diff --git a/configure b/configure
> index 31c5eafa..eeef58c6 100755
> --- a/configure
> +++ b/configure
> @@ -7066,7 +7066,7 @@ unknown)
> esac
>
>
> -AR_FLAGS=cru # set default
> +ARFLAGS=cru # set default
>
>
> # This is mainly for my use during testing and development.
> diff --git a/configure.ac b/configure.ac
> index dca2dbe1..31fdfff2 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -111,7 +111,7 @@ AC_PROG_MAKE_SET
> # support/ builds libsupport.a, allow for cross version of ar
> AM_PROG_AR
>
> -AR_FLAGS=cru # set default
> +ARFLAGS=cru # set default
> AC_SUBST(AR)
>
> # This is mainly for my use during testing and development.
> --
> 2.33.1
>