* PATCH: Regenerate aclocal.m4
@ 2006-01-16 22:16 Mark Mitchell
2006-01-16 22:18 ` Daniel Jacobowitz
0 siblings, 1 reply; 5+ messages in thread
From: Mark Mitchell @ 2006-01-16 22:16 UTC (permalink / raw)
To: gdb-patches
While making other changes, I changed configure.ac and went to
regenerate configure. I got this error from autoconf-2.59:
configure:1354: error: possibly undefined macro: AM_CONDITIONAL
If this token and others are legitimate, please use m4_pattern_allow.
See the Autoconf documentation.
and a broken configure script; the generated script had references to
AM_CONDITIONAL in it.
The problem is that aclocal.m4 is not correct. In particular, it's
definition of AM_MAINTAINER_MODE relies on AM_CONDITIONAL, but no
definition of AM_CONDITIONAL is present. I used aclocal to regenerate
aclocal.m4, which fixed the problem. (The current version of the
configure script was correct; I hypothesize that it was generated with
some other aclocal.m4, not the one checked in.)
I've attached the aclocal.m4 diff, even though this is a generated
file, so that we can all see what's changing.
OK to commit?
--
Mark Mitchell
CodeSourcery
mark@codesourcery.com
(650) 331-3385 x713
2006-01-16 Mark Mitchell <mark@codesourcery.com>
* aclocal.m4: Regenerate.
Index: aclocal.m4
===================================================================
RCS file: /cvs/src/src/gdb/aclocal.m4,v
retrieving revision 1.17
diff -c -5 -p -r1.17 aclocal.m4
*** aclocal.m4 17 Dec 2005 22:33:59 -0000 1.17
--- aclocal.m4 16 Jan 2006 22:11:56 -0000
***************
*** 25,36 ****
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
# GNU General Public License for more details.
# You should have received a copy of the GNU General Public License
# along with this program; if not, write to the Free Software
! # Foundation, Inc., 51 Franklin Street, Fifth Floor,
! # Boston, MA 02110-1301, USA.
# serial 3
AC_DEFUN([AM_MAINTAINER_MODE],
[AC_MSG_CHECKING([whether to enable maintainer-specific portions of Makefiles])
--- 25,81 ----
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
# GNU General Public License for more details.
# You should have received a copy of the GNU General Public License
# along with this program; if not, write to the Free Software
! # Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA
! # 02111-1307, USA.
!
! # serial 6
!
! # AM_CONDITIONAL(NAME, SHELL-CONDITION)
! # -------------------------------------
! # Define a conditional.
! AC_DEFUN([AM_CONDITIONAL],
! [AC_PREREQ(2.52)dnl
! ifelse([$1], [TRUE], [AC_FATAL([$0: invalid condition: $1])],
! [$1], [FALSE], [AC_FATAL([$0: invalid condition: $1])])dnl
! AC_SUBST([$1_TRUE])
! AC_SUBST([$1_FALSE])
! if $2; then
! $1_TRUE=
! $1_FALSE='#'
! else
! $1_TRUE='#'
! $1_FALSE=
! fi
! AC_CONFIG_COMMANDS_PRE(
! [if test -z "${$1_TRUE}" && test -z "${$1_FALSE}"; then
! AC_MSG_ERROR([[conditional "$1" was never defined.
! Usually this means the macro was only invoked conditionally.]])
! fi])])
!
! # Add --enable-maintainer-mode option to configure.
! # From Jim Meyering
!
! # Copyright (C) 1996, 1998, 2000, 2001, 2002, 2003, 2004
! # Free Software Foundation, Inc.
!
! # This program is free software; you can redistribute it and/or modify
! # it under the terms of the GNU General Public License as published by
! # the Free Software Foundation; either version 2, or (at your option)
! # any later version.
!
! # This program is distributed in the hope that it will be useful,
! # but WITHOUT ANY WARRANTY; without even the implied warranty of
! # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
! # GNU General Public License for more details.
!
! # You should have received a copy of the GNU General Public License
! # along with this program; if not, write to the Free Software
! # Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA
! # 02111-1307, USA.
# serial 3
AC_DEFUN([AM_MAINTAINER_MODE],
[AC_MSG_CHECKING([whether to enable maintainer-specific portions of Makefiles])
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: PATCH: Regenerate aclocal.m4
2006-01-16 22:16 PATCH: Regenerate aclocal.m4 Mark Mitchell
@ 2006-01-16 22:18 ` Daniel Jacobowitz
2006-01-16 22:36 ` Mark Mitchell
0 siblings, 1 reply; 5+ messages in thread
From: Daniel Jacobowitz @ 2006-01-16 22:18 UTC (permalink / raw)
To: Mark Mitchell; +Cc: gdb-patches
On Mon, Jan 16, 2006 at 02:16:39PM -0800, Mark Mitchell wrote:
> OK to commit?
Yes thanks.
--
Daniel Jacobowitz
CodeSourcery
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: PATCH: Regenerate aclocal.m4
2006-01-16 22:18 ` Daniel Jacobowitz
@ 2006-01-16 22:36 ` Mark Mitchell
2006-01-17 6:10 ` Jim Blandy
0 siblings, 1 reply; 5+ messages in thread
From: Mark Mitchell @ 2006-01-16 22:36 UTC (permalink / raw)
To: Daniel Jacobowitz; +Cc: gdb-patches
Daniel Jacobowitz wrote:
> On Mon, Jan 16, 2006 at 02:16:39PM -0800, Mark Mitchell wrote:
>
>>OK to commit?
>
> Yes thanks.
Committed.
--
Mark Mitchell
CodeSourcery
mark@codesourcery.com
(650) 331-3385 x713
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: PATCH: Regenerate aclocal.m4
2006-01-16 22:36 ` Mark Mitchell
@ 2006-01-17 6:10 ` Jim Blandy
2006-01-17 19:35 ` Mark Kettenis
0 siblings, 1 reply; 5+ messages in thread
From: Jim Blandy @ 2006-01-17 6:10 UTC (permalink / raw)
To: Mark Mitchell; +Cc: Daniel Jacobowitz, gdb-patches
I thought regenerating aclocal.m4 and friends with the same version of
autoconf / automake is considered an obvious fix. Is that not so?
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: PATCH: Regenerate aclocal.m4
2006-01-17 6:10 ` Jim Blandy
@ 2006-01-17 19:35 ` Mark Kettenis
0 siblings, 0 replies; 5+ messages in thread
From: Mark Kettenis @ 2006-01-17 19:35 UTC (permalink / raw)
To: jimb; +Cc: mark, drow, gdb-patches
> Date: Mon, 16 Jan 2006 22:10:42 -0800
> From: Jim Blandy <jimb@red-bean.com>
>
> I thought regenerating aclocal.m4 and friends with the same version of
> autoconf / automake is considered an obvious fix. Is that not so?
Sure, but it sometimes is a bit unclear what the right version of
autoconf/automake is. AFAIK we only state the minimum version to use.
And then of course sometimes people install patched versions, or have
additional automake/autoconf macros installed on their systems. It's
a bit of mess really, so I can understan Mark asking for an ok.
Mark
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2006-01-17 19:35 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2006-01-16 22:16 PATCH: Regenerate aclocal.m4 Mark Mitchell
2006-01-16 22:18 ` Daniel Jacobowitz
2006-01-16 22:36 ` Mark Mitchell
2006-01-17 6:10 ` Jim Blandy
2006-01-17 19:35 ` Mark Kettenis
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox