This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH] sim: use AC_REQUIRE with AC_PROG_CC
- From: Mike Frysinger <vapier at gentoo dot org>
- To: Joel Brobecker <brobecker at adacore dot com>
- Cc: gdb-patches at sourceware dot org, toolchain-devel at blackfin dot uclinux dot org
- Date: Mon, 17 Oct 2011 13:15:06 -0400
- Subject: Re: [PATCH] sim: use AC_REQUIRE with AC_PROG_CC
- References: <1310418640-32168-1-git-send-email-vapier@gentoo.org> <20111017165724.GC17942@adacore.com>
On Monday 17 October 2011 12:57:24 Joel Brobecker wrote:
> > 2011-07-11 Mike Frysinger <vapier@gentoo.org>
> >
> > * aclocal.m4 (SIM_AC_COMMON): Call AC_REQUIRE on AC_PROG_CC.
> > Delete direct call to AC_PROG_CC.
>
> Looks, OK, except why did you move the call up?
>
> Looking at the autoconf documentation, AC_CONFIG_HEADER does not need
> to be called right after AC_INIT, it is only customary. So if there is
> a reason for us to change the location of the use of AC_PROG_CC, we
> probably want to fix the comment as well.
i don't think there is a requirement for the AC_REQUIRE to be where it is;
i'll have to double check. the reason i moved it is that the convention i've
seen when reading other GNU projects is to put unconditional AC_REQUIRE calls
at the top of m4 defines. SIM_AC_COMMON is probably a bit of a special case
though due to it mostly being the normal configure.ac file.
i can move it back if you like (and i verify that it works fine).
-mike