Re: bug#11356: automake 1.12 and (C) 2011

2012-05-01 Thread Peter Johansson
Hi Stefano, Sorry about this late reply. On 04/28/2012 12:34 AM, Stefano Lattarini wrote: --- a/bootstrap +++ b/bootstrap @@ -77,6 +77,8 @@ dosubst () { rm -f $2 in=`echo $1 | sed 's,^.*/,,'` + current_year=`date +%Y` test -n $current_year \ +|| { echo $me: cannot get current

Re: bug#11356: automake 1.12 and (C) 2011

2012-05-01 Thread Stefano Lattarini
Hi Peter, thanks for the feedback. On 05/01/2012 02:23 AM, Peter Johansson wrote: Hi Stefano, Sorry about this late reply. On 04/28/2012 12:34 AM, Stefano Lattarini wrote: --- a/bootstrap +++ b/bootstrap @@ -77,6 +77,8 @@ dosubst () { rm -f $2 in=`echo $1 | sed 's,^.*/,,'`

Re: [FYI] {maint} news: Automake 1.13 will require Autoconf = 2.65

2012-05-01 Thread Stefano Lattarini
Hi Dave. On 05/01/2012 01:30 AM, Dave Hart wrote: On Mon, Apr 30, 2012 at 19:45, Stefano Lattarini stefano.lattar...@gmail.com wrote: + - Autoconf 2.65 or later will be required by the next major Automake +version (1.13). Until now, Automake has been working with any +Autoconf

[FYI] {maint} fixup: botched edit in bootstrap.sh

2012-05-01 Thread Stefano Lattarini
* bootstrap.sh (dosubst): Fix botched option (G instead of g) in a sed s/// expression. Issue introduced in today's commit 'v1.12-17-g5f810d0'. Signed-off-by: Stefano Lattarini stefano.lattar...@gmail.com --- bootstrap.sh |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 00/10] Add support for Objective C++ in Automake

2012-05-01 Thread Stefano Lattarini
Original recent threads: http://lists.gnu.org/archive/html/automake-patches/2012-04/msg00153.html http://lists.gnu.org/archive/html/automake-patches/2012-04/msg00204.html Original old thread, going back to June 2009: http://lists.gnu.org/archive/html/automake-patches/2009-07/msg00016.html with

[PATCH 01/10] objc++: initial support for Objective C++

2012-05-01 Thread Stefano Lattarini
From: Peter Breitenlohner p...@mppmu.mpg.de Original thread (dating back to almost three years ago): http://lists.gnu.org/archive/html/automake-patches/2009-07/msg00016.html * automake.in: Register new language 'objcxx'. (lang_objcxx_rewrite): New subroutine. (resolve_linker): Add OBJCXXLINK.

[PATCH 02/10] objc++: add documentation

2012-05-01 Thread Stefano Lattarini
From: Peter Breitenlohner p...@mppmu.mpg.de * doc/automake.texi (Objective C++ Support): New node. (How the Linker is Chosen, Support for Other Languages): Adjust. Signed-off-by: Peter Breitenlohner p...@mppmu.mpg.de Signed-off-by: Stefano Lattarini stefano.lattar...@gmail.com ---

[PATCH 03/10] objc++: test support for '.mm' suffix in _SOURCES entries

2012-05-01 Thread Stefano Lattarini
From: Peter Breitenlohner p...@mppmu.mpg.de * t/ext.sh: Enhance by adding a file with the '.mm' suffix in the 'foo_SOURCES' definition. * t/nodep2.sh: Likewise. Co-authored-by: Stefano Lattarini stefano.lattar...@gmail.com Signed-off-by: Peter Breitenlohner p...@mppmu.mpg.de Signed-off-by:

[PATCH 04/10] news: announce initial support for Objective C++

2012-05-01 Thread Stefano Lattarini
Signed-off-by: Stefano Lattarini stefano.lattar...@gmail.com --- NEWS |5 + 1 file changed, 5 insertions(+) diff --git a/NEWS b/NEWS index b415be6..7d0abcd 100644 --- a/NEWS +++ b/NEWS @@ -7,6 +7,11 @@ New in 1.12.1: m4 macro are deprecated, eliciting a warning in the 'obsolete'

[PATCH 07/10] objc: reorganize basic tests

2012-05-01 Thread Stefano Lattarini
* t/objc.sh, t/objc2.sh: Removed, merged into ... * t/objc-basic.sh: ... this new test. * t/objcxx-basic.sh: Add reference to new sister test. * t/list-of-tests.mk: Update. Signed-off-by: Stefano Lattarini stefano.lattar...@gmail.com --- t/list-of-tests.mk|3 +-- t/{objc2.sh =

[PATCH 06/10] objc++, objc: add first semantic tests

2012-05-01 Thread Stefano Lattarini
* t/objcxx-minidemo.sh: New test. * t/objc-minidemo.sh: Likewise. * t/list-of-tests.mk: Add them. Co-authored-by: Peter Breitenlohner p...@mppmu.mpg.de Signed-off-by: Stefano Lattarini stefano.lattar...@gmail.com --- t/list-of-tests.mk |2 ++ t/objc-minidemo.sh | 76

[PATCH 08/10] objc, objc++: test automatic dependency tracking

2012-05-01 Thread Stefano Lattarini
* t/objc-deps.sh, t/objcxx-deps.sh: New tests. * t/list-of-tests.mk: Add them. Signed-off-by: Stefano Lattarini stefano.lattar...@gmail.com --- t/list-of-tests.mk |2 ++ t/objc-deps.sh | 72 ++ t/objcxx-deps.sh | 81

[PATCH 09/10] objc, objc++: add stress test

2012-05-01 Thread Stefano Lattarini
From: Peter Breitenlohner p...@mppmu.mpg.de * t/objc-megademo.sh: New test, trying out a package using all of C, C++, Objective C and Objective C++ at the same time. * t/list-of-tests.mk: Add it. Signed-off-by: Stefano Lattarini stefano.lattar...@gmail.com --- t/list-of-tests.mk |1 +

[PATCH 10/10] objc, objc++: test support for compilation flags

2012-05-01 Thread Stefano Lattarini
* t/objc-flags.sh, t/objcxx-flags.sh: New tests. * t/list-of-tests.mk: Add them. Co-authored-by: Peter Breitenlohner p...@mppmu.mpg.de Signed-off-by: Stefano Lattarini stefano.lattar...@gmail.com --- t/list-of-tests.mk |2 ++ t/objc-flags.sh| 62

[FYI] {maint} silent rules: support for them is always active now

2012-05-01 Thread Stefano Lattarini
Before this change, support for silent rules was optional and activated only if the 'silent-rules' option was specified. The rationale behind that behaviour was that the silent-rules machinery originally only worked with make implementations supporting nested variables expansions, which isn't (or

Re: [FYI] {maint} silent rules: support for them is always active now

2012-05-01 Thread Stefano Lattarini
On 05/01/2012 04:05 PM, Stefano Lattarini wrote: [FYI] {maint} silent rules: support for them is always active now Oops, wrong, this was *not* an FYI: I haven't pushed the patch yet, and I'd love a review before I do so. Sorry for the noise, Stefabi

Re: [trivial PATCH] Fix typo in aclocal m4 directory README

2012-05-01 Thread Stefano Lattarini
Hi Andrew. On 05/01/2012 04:18 PM, Andrew Eikum wrote: Signed-off-by: Andrew Eikum aei...@codeweavers.com Noticed this while poking around the system today. m4/acdir/README |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) I've applied the patch in your name, adding you to

Re: [trivial PATCH] Fix typo in aclocal m4 directory README

2012-05-01 Thread Stefano Lattarini
On 05/01/2012 05:41 PM, Andrew Eikum wrote: On Tue, May 01, 2012 at 05:11:13PM +0200, Stefano Lattarini wrote: On 05/01/2012 04:18 PM, Andrew Eikum wrote: Signed-off-by: Andrew Eikum aei...@codeweavers.com Noticed this while poking around the system today. m4/acdir/README |2 +- 1

[PATCH 0/5] Convert automake testsuite to the use of a POSIX shell

2012-05-01 Thread Stefano Lattarini
This patches are the first step in that direction. I will push in 72 hours if there is no objection. Stefano Lattarini (5): tests: shell running test scripts is now named AM_TEST_RUNNER_SHELL configure: search a sturdy POSIX shell to be used in the testsuite tests: remove obsolete uses of

[PATCH 2/5] configure: search a sturdy POSIX shell to be used in the testsuite

2012-05-01 Thread Stefano Lattarini
* configure.ac: Add code (partially inspired to checks in gnulib's 'tests/init.sh') to search for a good-enough, not-buggy POSIX/XSI shell to be used in our testsuite. Accordingly AC_SUBSTitute the variable 'AM_TEST_RUNNER_SHELL'. * NEWS: Update. Signed-off-by: Stefano Lattarini

[PATCH 3/5] tests: remove obsolete uses of $sh_errexit_works

2012-05-01 Thread Stefano Lattarini
After the last changes, configure will ensure that the shell selected to run the test scripts can correctly propagate exit status to the exit trap when 'set -e' is in effect. * configure.ac (sh_errexit_works): Do not AC_SUBST it anymore. * defs-static.in (sh_errexit_works): Do not initialize from

[PATCH 4/5] test defs: fix indentation (cosmetic change)

2012-05-01 Thread Stefano Lattarini
* defs (exit trap): Fix indentation left botched by previous patch 'v1.11b-51-g626bf65'. Signed-off-by: Stefano Lattarini stefano.lattar...@gmail.com --- defs | 113 +- 1 file changed, 56 insertions(+), 57 deletions(-) diff --git

[PATCH 5/5] tests: fix a spurious failure with dash

2012-05-01 Thread Stefano Lattarini
The dash shell, at least version 0.5.5.1, doesn't always bail out with a syntax error when a stray fi in encountered: $ dash -c :; fi; echo stat = $? stat = 0 See also the relevant bug report: http://permalink.gmane.org/gmane.comp.shells.dash/717 This behaviour was causing a spurious

Re: [trivial PATCH] Fix typo in aclocal m4 directory README

2012-05-01 Thread Andrew Eikum
On Tue, May 01, 2012 at 05:11:13PM +0200, Stefano Lattarini wrote: On 05/01/2012 04:18 PM, Andrew Eikum wrote: Signed-off-by: Andrew Eikum aei...@codeweavers.com Noticed this while poking around the system today. m4/acdir/README |2 +- 1 file changed, 1 insertion(+), 1

Re: [PATCH 0/5] Convert automake testsuite to the use of a POSIX shell

2012-05-01 Thread Eric Blake
On 05/01/2012 10:04 AM, Stefano Lattarini wrote: This patches are the first step in that direction. I will push in 72 hours if there is no objection. Heads up - I'd like to review this series closely, but am not sure I will finish my review in 72 hours. Just so I know what to look for - is

Re: [PATCH 0/5] Convert automake testsuite to the use of a POSIX shell

2012-05-01 Thread Stefano Lattarini
Hi Eric. On 05/01/2012 06:18 PM, Eric Blake wrote: On 05/01/2012 10:04 AM, Stefano Lattarini wrote: This patches are the first step in that direction. I will push in 72 hours if there is no objection. Heads up - I'd like to review this series closely, but am not sure I will finish my

[PATCH] {maint} aclocal: error out again on unrecognized arguments

2012-05-01 Thread Stefano Lattarini
Starting from commit v1.11-662-g52246cc of XXX, XXX, aclocal has been silently ignoring non-option arguments instead of correctly reporting them. Fix this regression. * t/aclocal.sh: Update to catch the regression. * aclocal.in (parse_arguments): Explicitly reject non-option arguments. * NEWS:

Re: [PATCH] {maint} aclocal: error out again on unrecognized arguments

2012-05-01 Thread Eric Blake
On 05/01/2012 04:14 PM, Stefano Lattarini wrote: Starting from commit v1.11-662-g52246cc of XXX, XXX, aclocal has Did you mean to fix the XXX? been silently ignoring non-option arguments instead of correctly reporting them. Fix this regression. +* Bugs introduced by 1.11.3: + + -

Re: [PATCH] {maint} aclocal: error out again on unrecognized arguments

2012-05-01 Thread Stefano Lattarini
HI Eric, thanks for the super-quick review. On 05/02/2012 12:18 AM, Eric Blake wrote: On 05/01/2012 04:14 PM, Stefano Lattarini wrote: Starting from commit v1.11-662-g52246cc of XXX, XXX, aclocal has Did you mean to fix the XXX? Oops, I used a stale git.msg file when committing. Fixed like

Re: [PATCH] {maint} aclocal: error out again on unrecognized arguments

2012-05-01 Thread Eric Blake
On 05/01/2012 04:27 PM, Stefano Lattarini wrote: HI Eric, thanks for the super-quick review. On 05/02/2012 12:18 AM, Eric Blake wrote: On 05/01/2012 04:14 PM, Stefano Lattarini wrote: Starting from commit v1.11-662-g52246cc of XXX, XXX, aclocal has Did you mean to fix the XXX? Oops, I