2001-08-02 Tim Van Holder <tim.van.holder@pandora.be>
[automake.git] / HACKING
blob8c4974ace3fd4a402e30dec74e556ddf8c87335a
1 ================================================================
2 = This file
4 * This file attempts to describe the rules to use when hacking
5   automake.
7 * Don't put this file into the distribution.  Don't mention it in the
8   ChangeLog.
11 ================================================================
12 = Administrivia
14 * If you incorporate a change from somebody on the net:
15   First, if it is a large change, you must make sure they have signed the
16   appropriate paperwork.
17   Second, be sure to add their name and email address to THANKS
19 * If a change fixes a test, mention the test in the ChangeLog entry.
21 * If somebody reports a new bug, mention his name in the ChangeLog entry
22   and in the test case you write.  Put him into THANKS.
24 * The correct response to most actual bugs is to write a new test case
25   which demonstrates the bug.  Then fix the bug, re-run the test suite,
26   and check everything in.
28 * Some files in the automake package are not owned by automake.  These
29   files should never be edited here.  These files are COPYING, INSTALL,
30   ansi2knr.1, ansi2knr.c, config.guess config.sub, install-sh, mdate-sh,
31   missing, mkinstalldirs, texinfo.tex
33 * Changes other than bug fixes must be mentioned in NEWS
36 ================================================================
37 = Naming
39 * We've adopted the convention that internal AC_SUBSTs should be
40   named with a leading `am__', and internally generated targets should
41   be named with a leading `am--'.  This convention is very new
42   (as of Feb 7 2001) and so it isn't yet universally used.  But all
43   new code should use it.
45   We used to use `_am_' as the prefix for an internal AC_SUBST.
46   However, it turns out that NEWS-OS 4.2R complains if a Makefile
47   variable begins with `_'.  Yay for them.  I changed the target
48   naming convention just to be safe.
50 ================================================================
51 = Editing `.am' files
53 * Always use $(...) and not ${...}
55 * Use `:', not `true'.  Use `exit 1', not `false'.
57 * Use `##' comments liberally.  Comment anything even remotely
58   unusual.
60 * Never use basename or dirname.  Instead use sed
62 * If you run `cd' within back-quotes, make sure you set `CDPATH=:',
63   otherwise the directory name may be printed, depending on CDPATH.
65 * For install and uninstall rules, if a loop is required, it should be
66   silent.  Then the body of the loop itself should print each
67   "important" command it runs.  The printed commands should be preceded
68   by a single space.
71 ================================================================
72 = Editing automake.in and aclocal.in
74 * Follow existing indentation style.
76 * Perl 5 is now OK.
79 ================================================================
80 = Test suite
82 * Use "make check" and "make maintainer-check" liberally
84 * Make sure each test file is executable
87 ================================================================
88 = Release procedure
90 * Fetch new versions of the files that are maintained by the FSF.
91   Commit.  Unfortunately you need an FSF account to do this.
92   (You can also use `make fetch', but that is still woefully incomplete.)
94 * Update NEWS.  For an alpha release, update README-alpha.
96 * Update the version number in configure.in.
97   (The idea is that every other alpha number will be a net release.
98   The repository will always have its own "odd" number so we can easily
99   distinguish net and repo versions.)
101 * Configure, build, and install.
103 * Run aclocal, automake, and autoconf.
105 * Commit
107 * Run `make cvs-dist'
109 * Put new release on ftp site and send announcement.
110   (If not an alpha, announcement must also go to FSF.)
112 * Update version number in configure.in to next alpha number.
113   Re-run autoconf and commit.