This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: Restore release indicator to bfd/Makefile.am
- From: Alan Modra <amodra at gmail dot com>
- To: Andreas Schwab <schwab at linux-m68k dot org>
- Cc: Gunther Nikl <gnikl at justmail dot de>, binutils at sourceware dot org
- Date: Mon, 28 Jan 2019 09:31:51 +1030
- Subject: Re: Restore release indicator to bfd/Makefile.am
- References: <20190126170018.00002147@justmail.de> <87y3772l4g.fsf@igel.home>
On Sat, Jan 26, 2019 at 09:56:47PM +0100, Andreas Schwab wrote:
> On Jan 26 2019, Gunther Nikl <gnikl@justmail.de> wrote:
>
> > The GAS configure script checks bfd/Makefile.am for a line starting with
> > "RELEASE=y" to set the default value for run-time sanity checks. This
> > line got lost with "2e98a7bd882356a23076ad4a51a251d9ab1ce070" and since
> > that time GAS is build with sanity checks enabled unless overridden with
> > a configure option.
>
> It should use $development from bfd/development.sh instead.
Like this.
* configure.ac (ac_checking): Set from bfd/development.sh
development variable.
* configure: Regenerate.
diff --git a/gas/configure.ac b/gas/configure.ac
index 105d708253..cc4bae0957 100644
--- a/gas/configure.ac
+++ b/gas/configure.ac
@@ -50,10 +50,9 @@ AC_ARG_ENABLE(targets,
*) enable_targets=$enableval ;;
esac])dnl
-ac_checking=yes
-if grep '^RELEASE=y' ${srcdir}/../bfd/Makefile.am >/dev/null 2>/dev/null ; then
- ac_checking=
-fi
+ac_checking=
+. ${srcdir}/../bfd/development.sh
+test "$development" = true && ac_checking=yes
AC_ARG_ENABLE(checking,
[ --enable-checking enable run-time checks],
[case "${enableval}" in
--
Alan Modra
Australia Development Lab, IBM