Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
* 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