OK for maint? Regards, Stefano
From 158f31906d728c75d78272a00f352857c81931a2 Mon Sep 17 00:00:00 2001 From: Stefano Lattarini <stefano.lattar...@gmail.com> Date: Tue, 14 Sep 2010 22:50:15 +0200 Subject: [PATCH] Fixlets for HACKING.
* HACKING (Administrivia): Fixed typo `NES' for `NEWS'. (Naming): Do not refer to a convention dating back to Feb 2001 as "very new". Related rewording. (Working with Git): Now we require autoconf 2.67 for bootstrap, so use that version in the examples. --- ChangeLog | 9 +++++++++ HACKING | 14 +++++++------- 2 files changed, 16 insertions(+), 7 deletions(-) diff --git a/ChangeLog b/ChangeLog index 7b64d55..b3f3627 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,12 @@ +2010-09-14 Stefano Lattarini <stefano.lattar...@gmail.com> + + Fixlets for HACKING. + * HACKING (Administrivia): Fixed typo `NES' for `NEWS'. + (Naming): Do not refer to a convention dating back to Feb 2001 + as "very new". Related rewording. + (Working with Git): Now we require autoconf 2.67 for bootstrap, + so use that version in the examples. + 2010-09-13 Ralf Wildenhues <ralf.wildenh...@gmx.de> * HACKING: Hint at old commits with `git describe' output. diff --git a/HACKING b/HACKING index ecbd0a8..a3e37f0 100644 --- a/HACKING +++ b/HACKING @@ -33,16 +33,16 @@ Most of them are updated before release with `make fetch'. * Changes other than bug fixes must be mentioned in NEWS. Important - bug fixes should be mentioned in NES, too. + bug fixes should be mentioned in NEWS, too. ================================================================ = Naming * We've adopted the convention that internal AC_SUBSTs should be - named with a leading `am__', and internally generated targets should - be named with a leading `am--'. This convention is very new - (as of Feb 7 2001) and so it isn't yet universally used. But all - new code should use it. + named with a leading `am__', and internally generated targets + should be named with a leading `am--'. This convention, although + in place from at least February 2001, isn't yet universally used. + But all new code should use it. We used to use `_am_' as the prefix for an internal AC_SUBST. However, it turns out that NEWS-OS 4.2R complains if a Makefile @@ -98,8 +98,8 @@ not installed as "autoconf", pass it explicitly (along with the accompanying "autom4te") when calling `bootstrap' and `configure'. For example: - $ AUTOCONF=autoconf2.65 AUTOM4TE=autom4te2.65 ./bootstrap - $ ./configure AUTOCONF=autoconf2.65 AUTOM4TE=autom4te2.65 + $ AUTOCONF=autoconf2.67 AUTOM4TE=autom4te2.67 ./bootstrap + $ ./configure AUTOCONF=autoconf2.67 AUTOM4TE=autom4te2.67 * Dependent files aclocal.m4, configure and Makefile.in in all directories should be up to date in the git repository, so that -- 1.7.1
