bug-gettext
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [bug-gettext] plans for a new release


From: Stefano Lattarini
Subject: Re: [bug-gettext] plans for a new release
Date: Sat, 08 Jun 2013 21:17:31 +0200

On 06/08/2013 06:15 PM, Stefano Lattarini wrote:
> On 06/08/2013 05:32 PM, Stefano Lattarini wrote:
>>
>> [SNIP]
>>
>> The 'build-aux/fixaclocal' script is invoked throughout, but
>> the comments in that script state that it's only required with
>> aclocal from Automake 1.10, while the rest of the build
>> infrastructure assumes Automake >= 1.11.
>>
> Actually, since the fix for this doesn't require the addition of
> any new code, I think I can write a patch myself even without a
> copyright assignment.  I will do so shortly.
>
Here it is (below, after the scissor line).  I've tested with
Automake 1.11.1, 1.12.6, 1.13.3, noticing no issue.

In the process of preparing the patch, I've noticed some few
more minor issues with the Gettext bootstrap process.  I will
follow up with them in new reports on this (bug-gettext) list.

Regards,
  Stefano

---- 8< ---- 8< ---- 8< ---- 8< ---- 8< ---- 8< ---- 8< ---- 8< ----

>From b8e43d69e437c5b0c3f92f15f5f3c77ba936b53b Mon Sep 17 00:00:00 2001
Message-Id: <address@hidden>
From: Stefano Lattarini <address@hidden>
Date: Sat, 8 Jun 2013 18:34:41 +0200
Subject: [PATCH] build: drop the 'fixacocal' auxiliary script

The comments in that script state that its use is only required to
work around issues in aclocal from Automake 1.10, while the rest
of the Gettext build infrastructure assumes Automake >= 1.11 anyway.

* autogen.sh: Drop invocations of 'fixaclocal', simply invoke the
'aclocal' program directly.
* build-aux/fixaclocal: Remove.
* Makefile.am (EXTRA_DIST): Stop distributing it.
(ACLOCAL): Drop redefinition in function of 'fixaclocal'.
* gettext-runtime/Makefile.am (ACLOCAL): Likewise.
* gettext-runtime/libasprintf/Makefile.am (ACLOCAL): Likewise.
* gettext-tools/Makefile.am (ACLOCAL): Likewise.
        * gettext-tools/examples/Makefile.am (ACLOCAL): Likewise.

Signed-off-by: Stefano Lattarini <address@hidden>
---
 ChangeLog                               | 16 +++++++++++++++
 Makefile.am                             |  3 +--
 autogen.sh                              | 10 +++++-----
 build-aux/fixaclocal                    | 35 ---------------------------------
 gettext-runtime/Makefile.am             |  1 -
 gettext-runtime/libasprintf/Makefile.am |  1 -
 gettext-tools/Makefile.am               |  1 -
 gettext-tools/examples/Makefile.am      |  1 -
 8 files changed, 22 insertions(+), 46 deletions(-)
 delete mode 100755 build-aux/fixaclocal

diff --git a/ChangeLog b/ChangeLog
index 9625a75..dfc3ddb 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,19 @@
+2013-06-08  Stefano Lattarini  <address@hidden>  (tiny change)
+
+       build: drop the 'fixacocal' auxiliary script
+       The comments in that script state that its use is only required to
+       work around issues in aclocal from Automake 1.10, while the rest
+       of the Gettext build infrastructure assumes Automake >= 1.11 anyway.
+       * autogen.sh: Drop invocations of 'fixaclocal', simply invoke the
+       'aclocal' program directly.
+       * build-aux/fixaclocal: Remove.
+       * Makefile.am (EXTRA_DIST): Stop distributing it.
+       (ACLOCAL): Drop redefinition in function of 'fixaclocal'.
+       * gettext-runtime/Makefile.am (ACLOCAL): Likewise.
+       * gettext-runtime/libasprintf/Makefile.am (ACLOCAL): Likewise.
+       * gettext-tools/Makefile.am (ACLOCAL): Likewise.
+       * gettext-tools/examples/Makefile.am (ACLOCAL): Likewise.
+
 2013-06-04  Daiki Ueno  <address@hidden>

        Support for Vala.
diff --git a/Makefile.am b/Makefile.am
index 74e8dae..9807288 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -17,14 +17,13 @@
 ## Process this file with automake to produce Makefile.in.

 AUTOMAKE_OPTIONS = 1.5 gnu no-dependencies
-ACLOCAL = build-aux/fixaclocal @ACLOCAL@
 ACLOCAL_AMFLAGS = -I m4

 SUBDIRS = gnulib-local gettext-runtime gettext-tools

 EXTRA_DIST = \
   version.sh DEPENDENCIES PACKAGING HACKING ChangeLog.0 autogen.sh \
-  build-aux/ac-help.sed build-aux/fixaclocal build-aux/moopp \
+  build-aux/ac-help.sed build-aux/moopp \
   m4/fixautomake.m4 m4/woe32-dll.m4 \
   m4/libtool.m4

diff --git a/autogen.sh b/autogen.sh
index 39285ce..f55ce5b 100755
--- a/autogen.sh
+++ b/autogen.sh
@@ -337,14 +337,14 @@ else
 fi

 (cd gettext-runtime/libasprintf
- ../../build-aux/fixaclocal aclocal -I ../../m4 -I ../m4 -I gnulib-m4
+ aclocal -I ../../m4 -I ../m4 -I gnulib-m4
  autoconf
  autoheader && touch config.h.in
  automake --add-missing --copy
 )

 (cd gettext-runtime
- ../build-aux/fixaclocal aclocal -I m4 -I ../m4 -I gnulib-m4
+ aclocal -I m4 -I ../m4 -I gnulib-m4
  autoconf
  autoheader && touch config.h.in
  automake --add-missing --copy
@@ -361,7 +361,7 @@ fi
 cp -p gettext-runtime/ABOUT-NLS gettext-tools/ABOUT-NLS

 (cd gettext-tools/examples
- ../../build-aux/fixaclocal aclocal -I ../../gettext-runtime/m4 -I ../../m4
+ aclocal -I ../../gettext-runtime/m4 -I ../../m4
  autoconf
  automake --add-missing --copy
  # Rebuilding the examples PO files is only rarely needed.
@@ -371,7 +371,7 @@ cp -p gettext-runtime/ABOUT-NLS gettext-tools/ABOUT-NLS
 )

 (cd gettext-tools
- ../build-aux/fixaclocal aclocal -I m4 -I ../gettext-runtime/m4 -I ../m4 -I 
gnulib-m4 -I libgrep/gnulib-m4 -I libgettextpo/gnulib-m4
+ aclocal -I m4 -I ../gettext-runtime/m4 -I ../m4 -I gnulib-m4 -I 
libgrep/gnulib-m4 -I libgettextpo/gnulib-m4
  autoconf
  autoheader && touch config.h.in
  test -d intl || mkdir intl
@@ -392,6 +392,6 @@ cp -p gettext-runtime/ABOUT-NLS gettext-tools/ABOUT-NLS
  fi
 )

-build-aux/fixaclocal aclocal -I m4
+aclocal -I m4
 autoconf
 automake
diff --git a/build-aux/fixaclocal b/build-aux/fixaclocal
deleted file mode 100755
index 10ff8e4..0000000
--- a/build-aux/fixaclocal
+++ /dev/null
@@ -1,35 +0,0 @@
-#!/bin/sh
-# Script for fixing aclocal.m4 files produced by 'aclocal' from automake 1.10.
-# Copyright (C) 2006, 2008 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 3 of the License, 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, see <http://www.gnu.org/licenses/>.
-
-# Usage: fixaclocal aclocal [OPTIONS]
-
-"$@"
-result=$?
-if test $result = 0 && test -f aclocal.m4; then
-  # Remove the block of 4 lines starting with 'm4_if(m4_PACKAGE_VERSION'
-  # and the block of 5 lines starting with 'm4_if(AC_AUTOCONF_VERSION'
-  # (automake <= 1.10.1) or with 'm4_if(m4_defn([AC_AUTOCONF_VERSION])'
-  # (automake >= 1.10.2).
-  sed -e '/m4_if(m4_PACKAGE_VERSION/{N;N;N;d}' -e 
'/m4_if(AC_AUTOCONF_VERSION/{N;N;N;N;d}' -e 
'/m4_if(m4_defn(\[AC_AUTOCONF_VERSION\])/{N;N;N;N;d}' < aclocal.m4 > 
aclocal.m4.tmp
-  if cmp aclocal.m4 aclocal.m4.tmp > /dev/null; then
-    rm -f aclocal.m4.tmp
-  else
-    mv aclocal.m4.tmp aclocal.m4
-  fi
-else
-  exit $result
-fi
diff --git a/gettext-runtime/Makefile.am b/gettext-runtime/Makefile.am
index df5336f..bd8fefc 100644
--- a/gettext-runtime/Makefile.am
+++ b/gettext-runtime/Makefile.am
@@ -17,7 +17,6 @@
 ## Process this file with automake to produce Makefile.in.

 AUTOMAKE_OPTIONS = 1.5 gnu no-dependencies
-ACLOCAL = ../build-aux/fixaclocal @ACLOCAL@
 ACLOCAL_AMFLAGS = -I m4 -I ../m4 -I gnulib-m4

 MAKEINFO = env LANG= LC_MESSAGES= LC_ALL= LANGUAGE= @MAKEINFO@
diff --git a/gettext-runtime/libasprintf/Makefile.am 
b/gettext-runtime/libasprintf/Makefile.am
index 343ae4b..8e876c8 100644
--- a/gettext-runtime/libasprintf/Makefile.am
+++ b/gettext-runtime/libasprintf/Makefile.am
@@ -17,7 +17,6 @@
 ## Process this file with automake to produce Makefile.in.

 AUTOMAKE_OPTIONS = 1.5 gnu no-dependencies
-ACLOCAL = ../../build-aux/fixaclocal @ACLOCAL@
 ACLOCAL_AMFLAGS = -I ../../m4 -I ../m4 -I gnulib-m4
 EXTRA_DIST =
 BUILT_SOURCES =
diff --git a/gettext-tools/Makefile.am b/gettext-tools/Makefile.am
index 3715b8a..e93ab1f 100644
--- a/gettext-tools/Makefile.am
+++ b/gettext-tools/Makefile.am
@@ -17,7 +17,6 @@
 ## Process this file with automake to produce Makefile.in.

 AUTOMAKE_OPTIONS = 1.5 gnu no-dependencies
-ACLOCAL = ../build-aux/fixaclocal @ACLOCAL@
 ACLOCAL_AMFLAGS = -I m4 -I ../gettext-runtime/m4 -I ../m4 -I gnulib-m4 -I 
libgrep/gnulib-m4 -I libgettextpo/gnulib-m4

 SUBDIRS = doc intl gnulib-lib libgrep src libgettextpo po projects styles misc 
man m4 tests gnulib-tests examples
diff --git a/gettext-tools/examples/Makefile.am 
b/gettext-tools/examples/Makefile.am
index f4e93cb..1a73bda 100644
--- a/gettext-tools/examples/Makefile.am
+++ b/gettext-tools/examples/Makefile.am
@@ -17,7 +17,6 @@
 ## Process this file with automake to produce Makefile.in.

 AUTOMAKE_OPTIONS = 1.2 foreign
-ACLOCAL = ../../build-aux/fixaclocal @ACLOCAL@
 ACLOCAL_AMFLAGS = -I ../../gettext-runtime/m4 -I ../../m4
 SUBDIRS = po
 EXTRA_DIST =
-- 
1.8.3.rc3.8.g5e49f30




reply via email to

[Prev in Thread] Current Thread [Next in Thread]