[FYI] {master} depcomp tests: do not specify LDFLAGS for static libraries

2012-02-13 Thread Peter Rosin
Commit v1.11-1848-gb3f34ca accidentally specified LDFLAGS for libbaz even when built as a static library, which is not allowed. * tests/depcomp.sh (src/Makefile.am): Don't add any LDFLAGS when libbaz is built as a static library. --- tests/depcomp.sh |6 -- 1 files changed, 4 insertions(+

[FYI] {master} depcomp tests: avoid redundant runs for non-libtool test

2012-02-13 Thread Peter Rosin
When running the dependency tests without libtool, it's not meaningful to run the tests three-fold, the libtool configure options are simply ignored anyway. * tests/depcomp.sh: Only run the tests three-fold when libtool is in use. --- tests/depcomp.sh | 23 ++- 1 files chang

Re: Merging 'yacc-work' into master

2012-02-13 Thread Peter Rosin
Peter Rosin skrev 2012-02-14 00:56: > Stefano Lattarini skrev 2012-02-12 15:59: >> I have cloned 'master' into a temporary branch 'yl-work-for-master', >> merged master in there, and pushed. You can find the resulting branch >> here: >> >> http://git.savannah.gnu.org/cgit/automake.git/log/?h=yl-

Re: Merging 'yacc-work' into master

2012-02-13 Thread Peter Rosin
Stefano Lattarini skrev 2012-02-12 15:59: > I have cloned 'master' into a temporary branch 'yl-work-for-master', > merged master in there, and pushed. You can find the resulting branch > here: > > http://git.savannah.gnu.org/cgit/automake.git/log/?h=yl-work-for-master > > Below are the commit

[FYI] {master} tests: avoid yet other spurious failures on NetBSD

2012-02-13 Thread Stefano Lattarini
* tests/tap-signal.tap: Add a "strategically placed" extra 'echo' command and a temporary silencing of xtraces; they are required to avoid possible garbled output with NetBSD make, which would miss some final newlines in the expected places and thus mess up our TAP output. --- tests/tap-signal.tap

[FYI] {master} tests: avoid a spurious failure on NetBSD

2012-02-13 Thread Stefano Lattarini
* tests/test-driver-is-distributed.test: Rewrite to avoid having to edit configure.in; such editing was subtly botched on NetBSD (the 'AC_CONFIG_AUX_DIR' call got misplaced, ending up *before* the 'AC_INIT' call), and that caused an hard-to-track bug. Since we are at it, extend coverage a little.

[FYI] {master} tests: avoid spurious failure on NetBSD

2012-02-13 Thread Stefano Lattarini
* tests/parallel-tests-fd-redirect-exeext.test (configure.in): Tweak so that we don't end up with the generated configure script having two nested subshells that start with '((': ((./conftest$EXEEXT 9>&1) | grep "^foobar") >&5 2>&5 which is unportable and could confuse some shells (e.g., NetBSD

Re: [PATCH] tests: increase coverage for depcomp tests

2012-02-13 Thread Peter Rosin
Stefano Lattarini skrev 2012-02-13 21:13: > On 02/13/2012 09:05 PM, Peter Rosin wrote: >> On platforms requiring that no undefined symbols exist in order >> to build shared libraries (e.g. Windows DLLs), you have to >> explicitly declare that the libtool library you are building >> does not actuall

Re: Merging 'yacc-work' into master

2012-02-13 Thread Peter Rosin
Stefano Lattarini skrev 2012-02-13 21:07: > On 02/13/2012 08:14 PM, Peter Rosin wrote: >> Stefano Lattarini skrev 2012-02-13 18:54: >>> On 02/13/2012 04:28 PM, Peter Rosin wrote: Ah well, sorry for the noise (but some of the typos could still be picked out of that patch I suppose). >

Re: [PATCH] tests: increase coverage for depcomp tests

2012-02-13 Thread Stefano Lattarini
On 02/13/2012 09:05 PM, Peter Rosin wrote: > On platforms requiring that no undefined symbols exist in order > to build shared libraries (e.g. Windows DLLs), you have to > explicitly declare that the libtool library you are building > does not actually have any undefined symbols, for libtool to > e

Re: Merging 'yacc-work' into master

2012-02-13 Thread Stefano Lattarini
On 02/13/2012 08:14 PM, Peter Rosin wrote: > Stefano Lattarini skrev 2012-02-13 18:54: >> On 02/13/2012 04:28 PM, Peter Rosin wrote: >>> Ah well, sorry for the noise (but some of the typos >>> could still be picked out of that patch I suppose). >>> >> Yes. Will you do that, or should I? If you do

[PATCH] tests: increase coverage for depcomp tests

2012-02-13 Thread Peter Rosin
On platforms requiring that no undefined symbols exist in order to build shared libraries (e.g. Windows DLLs), you have to explicitly declare that the libtool library you are building does not actually have any undefined symbols, for libtool to even try to build it as a shared library. Without tha

Re: Merging 'yacc-work' into master

2012-02-13 Thread Peter Rosin
Stefano Lattarini skrev 2012-02-13 18:54: > On 02/13/2012 04:28 PM, Peter Rosin wrote: >> Ah well, sorry for the noise (but some of the typos >> could still be picked out of that patch I suppose). >> > Yes. Will you do that, or should I? If you do, the resulting patch > is pre-approved of course.

[FYI] {yl-for-master} yacc/lex tests: avoid spurious skips with non-GNU make

2012-02-13 Thread Stefano Lattarini
* tests/defs (useless_vpath_rebuild): Fix botched logic that was causing this function to always diagnose non-GNU make implementations as being affected by the "useless rebuilds in VPATH setup" bug (that affects FreeBSD make and causes automake bug#7884). --- tests/defs | 22

Re: Merging 'yacc-work' into master

2012-02-13 Thread Stefano Lattarini
On 02/13/2012 04:28 PM, Peter Rosin wrote: > >> * NEWS: Make it clear that it's the derived C code that has >> to be non-distributed for it to be removed by "make clean", >> not the Yacc/Lex source. Also fix some grammaros and typos. > > I apparently got that backwards. I deluded myself by think

Re: [FYI] {yl-work-for-master} fixup: remove double news entry

2012-02-13 Thread Eric Blake
On 02/13/2012 10:45 AM, Stefano Lattarini wrote: > On 02/13/2012 04:47 PM, Peter Rosin wrote: >> >> I have pushed the following as obvious: >> >> From: Peter Rosin >> Date: Mon, 13 Feb 2012 16:43:34 +0100 >> Subject: [PATCH] fixup: remove double news entry >> >> * NEWS: Remove double (stale) news

Re: [FYI] {yl-work-for-master} fixup: remove double news entry

2012-02-13 Thread Stefano Lattarini
On 02/13/2012 04:47 PM, Peter Rosin wrote: > > I have pushed the following as obvious: > > From: Peter Rosin > Date: Mon, 13 Feb 2012 16:43:34 +0100 > Subject: [PATCH] fixup: remove double news entry > > * NEWS: Remove double (stale) news entry introduced by a botched > merge. > Thanks. And I t

[FYI] {yl-work-for-master} fixup: remove double news entry

2012-02-13 Thread Peter Rosin
Peter Rosin skrev 2012-02-13 16:44: > Oh, I also noticed that the merge into yl-work-for-master is bad with regard > to NEWS, as it still contains the old entry w/o lex (which appears to be the > origin of the confusing wording): > > - C source and header files derived from non-distributed Yacc

Re: Merging 'yacc-work' into master

2012-02-13 Thread Peter Rosin
Peter Rosin skrev 2012-02-13 13:50: > Stefano Lattarini skrev 2012-02-12 15:59: >> I have cloned 'master' into a temporary branch 'yl-work-for-master', >> merged master in there, and pushed. You can find the resulting branch >> here: >> >> http://git.savannah.gnu.org/cgit/automake.git/log/?h=yl-

Re: Merging 'yacc-work' into master

2012-02-13 Thread Peter Rosin
Peter Rosin skrev 2012-02-13 13:50: > Stefano Lattarini skrev 2012-02-12 15:59: >> I have cloned 'master' into a temporary branch 'yl-work-for-master', >> merged master in there, and pushed. You can find the resulting branch >> here: >> >> http://git.savannah.gnu.org/cgit/automake.git/log/?h=yl-

Re: Merging 'yacc-work' into master

2012-02-13 Thread Peter Rosin
Stefano Lattarini skrev 2012-02-12 15:59: > I have cloned 'master' into a temporary branch 'yl-work-for-master', > merged master in there, and pushed. You can find the resulting branch > here: > > http://git.savannah.gnu.org/cgit/automake.git/log/?h=yl-work-for-master > > Below are the commit