Re: [PATCH] {master} missing: do not touch timestamps; only warn for out-of-date files

2012-06-20 Thread Dave Hart
On Wed, Jun 20, 2012 at 10:33 PM, Dave Hart wrote: > > I had no idea missing would monkey with timestamps like that.  I'm > wholeheartedly in favor of removing that capability in the name of > predictably correct results. > > While I agree fix-timestamps.sh might be a clearer name, using the > nam

Re: [PATCH] {master} missing: do not touch timestamps; only warn for out-of-date files

2012-06-20 Thread Dave Hart
On Wed, Jun 20, 2012 at 9:30 PM, Stefano Lattarini wrote: > Before this change, the missing script had a twofold role: > >  - it warned the user if some required maintainer tools was missing, >    or too old; > >  - in such a case, it tried to "fix" the timestamp of the files that >    should have

Re: [FYI] {maint} bootstrap: overhaul and improve

2012-06-20 Thread Stefano Lattarini
On 06/20/2012 06:45 PM, Eric Blake wrote: > >> -else >> - include ./Makefile >> - include $(srcdir)/syntax-checks.mk >> + # Any target but 'bootstrap' specified in an unconfigured tree >> + # is an error, env when the user is running GNU make. > > did you mean 'even' instead of 'env'? > Oops,

[PATCH] {master} missing: do not touch timestamps; only warn for out-of-date files

2012-06-20 Thread Stefano Lattarini
Before this change, the missing script had a twofold role: - it warned the user if some required maintainer tools was missing, or too old; - in such a case, it tried to "fix" the timestamp of the files that should have been rebuilt by that tool (without actually updating the file

Re: [FYI] {maint} bootstrap: overhaul and improve

2012-06-20 Thread Stefano Lattarini
On 06/20/2012 06:45 PM, Eric Blake wrote: > On 06/20/2012 10:39 AM, Stefano Lattarini wrote: >> This fixes several weaknesses and buglets in the 'bootstrap' convenience >> target offered in GNUmakefile (and its supporting code). Refer to the >> extensive code comments in there for more details. >>

Re: [FYI] {maint} bootstrap: overhaul and improve

2012-06-20 Thread Eric Blake
On 06/20/2012 10:39 AM, Stefano Lattarini wrote: > This fixes several weaknesses and buglets in the 'bootstrap' convenience > target offered in GNUmakefile (and its supporting code). Refer to the > extensive code comments in there for more details. > > * GNUmakefile: Almost completely rewritten.

[FYI] Merge branch 'maint' into master

2012-06-20 Thread Stefano Lattarini
commit 7c6310c3a069ad18b3be277be606c7109f8e9a82 Merge: 5741ca2 fe4d80c Author: Stefano Lattarini Date: Wed Jun 20 18:35:57 2012 +0200 Merge branch 'maint' * maint: bootstrap: overhaul and improve docs: quote 'like this', not `like this'

[FYI] {maint} bootstrap: overhaul and improve

2012-06-20 Thread Stefano Lattarini
This fixes several weaknesses and buglets in the 'bootstrap' convenience target offered in GNUmakefile (and its supporting code). Refer to the extensive code comments in there for more details. * GNUmakefile: Almost completely rewritten. Signed-off-by: Stefano Lattarini --- GNUmakefile | 105

[FYI] {maint} docs: quote 'like this', not `like this'

2012-06-20 Thread Stefano Lattarini
* doc/automake.texi: Fix the reported warnings and informative messages from automake to quote 'like this' rather than as `like this'. Do the same for comments and some text in our examples. Signed-off-by: Stefano Lattarini --- doc/automake.texi | 18 +- 1 file changed, 9 inse

Re: [FYI] [ng] maintcheck: remove a test that can't work for Automake-NG

2012-06-20 Thread Stefano Lattarini
On 06/20/2012 04:40 PM, Stefano Lattarini wrote: > * syntax-checks.mk (sc_tests_makefile_variable_order): Remove. This > rule checked that all variables definitions were listed before rules > in the automake-generated Makefile.in files; but this invariant is not > guaranteed anymore by Automake-NG

[FYI] [ng] maintcheck: remove a test that can't work for Automake-NG

2012-06-20 Thread Stefano Lattarini
* syntax-checks.mk (sc_tests_makefile_variable_order): Remove. This rule checked that all variables definitions were listed before rules in the automake-generated Makefile.in files; but this invariant is not guaranteed anymore by Automake-NG, and in fact it will break more and more as our work pro

[FYI] {master} fixup: tests: special requirement 'makeinfo-html' removed from ./defs

2012-06-20 Thread Stefano Lattarini
This should have ideally been part of earlier commit v1.12.1-93-g3c64d54. * defs (makeinfo-html): Remove. Signed-off-by: Stefano Lattarini --- defs |7 --- 1 file changed, 7 deletions(-) diff --git a/defs b/defs index a6d2b4a..0b01f59 100644 --- a/defs +++ b/defs @@ -854,13 +854,6 @@ d

[FYI] {master} maintcheck: guard against obsolete $required entries

2012-06-20 Thread Stefano Lattarini
* syntax-check.mk: Add rules guarding against the use of the obsolete test requirements 'texi2dvi-o' and 'makeinfo-html'. Signed-off-by: Stefano Lattarini --- syntax-checks.mk | 14 ++ 1 file changed, 14 insertions(+) diff --git a/syntax-checks.mk b/syntax-checks.mk index bd16063.

[FYI] {master} tests: drop requirement 'makeinfo-html'; 'makeinfo' is enough

2012-06-20 Thread Stefano Lattarini
* defs: Since we now require Texinfo 4.9 or later in the Automake generated Makefiles, and since the '--html' option of makeinfo, as well as its capability to generate HTML output, has been supported since Texinfo 4.0 (according to the Texinfo NEWS file), the requirement 'makeinfo-html' is obsolete

[FYI] {master} tests: drop requirement 'texi2dvi-o'; 'texi2dvi' is enough

2012-06-20 Thread Stefano Lattarini
* defs: Since we now require Texinfo 4.9 or later in the Automake generated Makefiles, and since the '-o' option of texi2dvi has been supported since Texinfo 4.1, the requirement 'texi2dvi-o' is obsolete -- the simpler requirement 'texi2dvi' is enough. * t/silent8.sh ($required): Adjust by using si

Re: [PATCH] texi: require Texinfo >= 4.9, related enhancements

2012-06-20 Thread Stefano Lattarini
I've improved the commit message and code comments (adding the details about the Texinfo bug we've spoken about), and pushed. The final, amended patch is attached, for reference. Regards, Stefano >From 205c7572497789676bea3167b9cdeede69236a16 Mon Sep 17 00:00:00 2001 Message-Id: <205c7572497789