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
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
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,
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
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.
>>
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.
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'
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
* 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
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
* 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
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
* 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.
* 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
* 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
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
16 matches
Mail list logo