]> git.saurik.com Git - bison.git/blame - README-hacking
tests: honor TESTSUITEFLAGS in all the check targets.
[bison.git] / README-hacking
CommitLineData
2ed0e35f
AD
1-*- outline -*-
2
3This file attempts to describe the rules to use when hacking Bison.
9126263e 4Don't put this file into the distribution.
2ed0e35f 5
f414d77d
AD
6Everything related to the development of Bison is on Savannah:
7
26fccd4d 8 http://savannah.gnu.org/projects/bison/
f414d77d
AD
9
10
2ed0e35f
AD
11* Administrivia
12
13** If you incorporate a change from somebody on the net:
14First, if it is a large change, you must make sure they have signed
15the appropriate paperwork. Second, be sure to add their name and
16email address to THANKS.
17
7de42e52 18** If a change fixes a test, mention the test in the commit message.
2ed0e35f
AD
19
20** Bug reports
7de42e52 21If somebody reports a new bug, mention his name in the commit message
2ed0e35f
AD
22and in the test case you write. Put him into THANKS.
23
24The correct response to most actual bugs is to write a new test case
25which demonstrates the bug. Then fix the bug, re-run the test suite,
26and check everything in.
27
f414d77d 28
427c0dda
AD
29* Hacking
30
2ed0e35f
AD
31** Visible changes
32Which include serious bug fixes, must be mentioned in NEWS.
427c0dda
AD
33
34** Translations
35Only user visible strings are to be translated: error messages, bits
36of the .output file etc. This excludes impossible error messages
37(comparable to assert/abort), and all the --trace output which is
38meant for the maintainers only.
2ed0e35f
AD
39
40
26fccd4d
AD
41* Working from the repository
42
43These notes intend to help people working on the checked-out sources.
44These requirements do not apply when building from a distribution tarball.
45
46** Requirements
47
48We've opted to keep only the highest-level sources in the repository.
49This eases our maintenance burden, (fewer merges etc.), but imposes more
50requirements on anyone wishing to build from the just-checked-out sources.
51For example, you have to use the latest stable versions of the maintainer
52tools we depend upon, including:
53
54- Automake <http://www.gnu.org/software/automake/>
55- Autoconf <http://www.gnu.org/software/autoconf/>
56- Flex <http://www.gnu.org/software/flex/>
57- Gettext <http://www.gnu.org/software/gettext/>
58- Gzip <http://www.gnu.org/software/gzip/>
59- Perl <http://www.cpan.org/>
60- Rsync <http://samba.anu.edu.au/rsync/>
61- Tar <http://www.gnu.org/software/tar/>
62
63Valgrind <http://valgrind.org/> is also highly recommended, if
64Valgrind supports your architecture.
65
66Bison is written using Bison grammars, so there are bootstrapping
67issues. The bootstrap script attempts to discover when the C code
68generated from the grammars is out of date, and to bootstrap with an
69out-of-date version of the C code, but the process is not foolproof.
70Also, you may run into similar problems yourself if you modify Bison.
71
72Only building the initial full source tree will be a bit painful.
73Later, after synchronizing from the repository a plain `make' should
74be sufficient.
75
76** First checkout
77
78Obviously, if you are reading these notes, you did manage to check out
79this package from the repository. For the record, you will find all the
80relevant information on:
81
82 http://savannah.gnu.org/git/?group=bison
83
84Bison uses Git submodules: subscriptions to other Git repositories.
85In particular it uses gnulib, the GNU portability library. To ask Git
86to perform the first checkout of the submodules, run
87
88 $ git submodule update --init
89
90Git submodule support is weak before versions 1.6 and later, you
91should probably upgrade Git if your version is older.
92
93The next step is to get other files needed to build, which are
94extracted from other source packages:
95
96 $ ./bootstrap
97
98And there you are! Just
99
100 $ ./configure
101 $ make
102 $ make check
103
104At this point, there should be no difference between your local copy,
105and the master copy:
106
107 $ git diff
108
109should output no difference.
110
111Enjoy!
112
113** Updating
114
115The use of submodules make things somewhat different because git does
116not support recursive operations: submodules must be taken care of
117explicitly by the user.
118
119*** Updating Bison
120
121If you pull a newer version of a branch, say via `git pull', you might
122import requests for updated submodules. A simple `git diff' will
123reveal if the current version of the submodule (i.e., the actual
124contents of the gnulib directory) and the current request from the
125subscriber (i.e., the reference of the version of gnulib that the
126Bison reporitory requests) differ. To upgrade the submodules (i.e.,
127to check out the version that is actually requested by the subscriber,
128run `git submodule update'.
129
130 $ git pull
131 $ git submodule update
132
133*** Updating a submodule
134To update a submodule, say gnulib, do as follows:
135
136Get the most recent version of the master branch from git.
137
138 $ cd gnulib
139 $ git fetch
140 $ git checkout -b master --track origin/master
141
142Make sure Bison can live with that version of gnulib.
143
144 $ cd ..
145 $ ./bootstrap
146 $ make distcheck
147
148Register your changes.
149
150 $ git checkin ...
151
a898435b
JD
152For a suggestion of what gnulib commit might be stable enough for a
153formal release, see the ChangeLog in the latest gnulib snapshot at:
154
155 http://erislabs.net/ianb/projects/gnulib/
156
157The autoconf files we use are currently:
158
159 m4/m4.m4
160 lib/m4sugar/m4sugar.m4
161 lib/m4sugar/foreach.m4
162
e2eb03d4 163These files don't change very often in Autoconf, so it should be
a898435b
JD
164relatively straight-forward to examine the differences in order to
165decide whether to update.
26fccd4d 166
2ed0e35f
AD
167* Test suite
168
169** make check
170Use liberally.
171
172** Release checks
173Try to run the test suite with more severe conditions before a
174release:
175
176- Configure the package with --enable-gcc-warnings, so that one checks
177 that 1. Bison compiles cleanly, 2. the parsers it produces compile
178 cleanly too.
179
fb9a3976
JD
180- Build with -DGNULIB_POSIXCHECK. It suggests gnulib modules that can
181 fix portability issues.
182
a0cd287e
AD
183- Check with `make syntax-check' if there are issues diagnosed by
184 gnulib.
185
f377f69f
AD
186- run `make maintainer-check' which:
187 - runs `valgrind -q bison' to run Bison under Valgrind.
1f1b791b 188 - runs the parsers under Valgrind.
f377f69f 189 - runs the test suite with G++ as C compiler...
2ed0e35f 190
9126263e
JD
191- run `make maintainer-push-check', which runs `make maintainer-check'
192 while activating the push implementation and its pull interface wrappers
193 in many test cases that were originally written to exercise only the
194 pull implementation. This makes certain the push implementation can
195 perform every task the pull implementation can.
196
197- run `make maintainer-xml-check', which runs `make maintainer-check'
198 while checking Bison's XML automaton report for every working grammar
199 passed to Bison in the test suite. The check just diffs the output of
200 Bison's included XSLT style sheets with the output of --report=all and
201 --graph.
202
a0cd287e
AD
203- running `make maintainer-release-check' takes care of running
204 maintainer-check, maintainer-push-check and maintainer-xml-check.
205
2ed0e35f
AD
206- Change tests/atlocal/CFLAGS to add your preferred options. For
207 instance, `-traditional' to check that the parsers are K&R. Note
208 that it does not make sense for glr.c, which should be ANSI,
548e2104 209 but currently is actually GNU C, nor for lalr1.cc.
2ed0e35f 210
2ed0e35f
AD
211
212* Release Procedure
213
a898435b
JD
214** Update the submodules. See above.
215
845346b3
JD
216** Update maintainer tools, such as Autoconf. See above.
217
d67e37a7
JD
218** Try to get the *.pot files to the Translation Project at least one
219week before a stable release, to give them time to translate them.
220Before generating the *.pot files, make sure that po/POTFILES.in and
221runtime-po/POTFILES.in list all files with translatable strings.
222This helps: grep -l '\<_(' *
ddc65ffd 223
2ed0e35f
AD
224** Tests
225See above.
226
227** Update the foreign files
e476c87d
PE
228Running `./bootstrap' in the top level should update them all for you.
229This covers PO files too. Sometimes a PO file contains problems that
230causes it to be rejected by recent Gettext releases; please report
231these to the Translation Project.
2ed0e35f 232
548e2104 233** Update README
8ebc7dca
JD
234Make sure the information in README is current. Most notably, make sure
235it recommends a version of GNU M4 that is compatible with the latest
236Bison sources.
237
238** Check copyright years.
239We update years in copyright statements throughout Bison once at the
240start of every year by running `make update-copyright'. However, before
241a release, it's good to verify that it's actually been run. Besides the
242copyright statement for each Bison file, check the copyright statements
243that the skeletons insert into generated parsers, and check all
244occurrences of PACKAGE_COPYRIGHT_YEAR in configure.ac.
548e2104 245
2ed0e35f
AD
246** Update NEWS
247The version number, *and* the date of the release (including for
248betas).
249
7de42e52 250** Mention the release name in a commit message
e2eb03d4 251Should have an entry similar to `Version 2.3b.'.
548e2104 252
548e2104 253** Tag the release
e2eb03d4
AD
254Before Bison will build with the right version number, you must tag
255the release in git. Do this after all other changes. The command is
256similar to:
548e2104
JD
257
258 git tag -a v2.3b
259
7de42e52 260The commit message can be simply:
548e2104
JD
261
262 Bison 2.3b
263
264** Push
265Once `make distcheck' passes, push your changes and the tag.
266`git push' without arguments will not push the tag.
2ed0e35f
AD
267
268** make alpha
548e2104
JD
269FIXME: `make alpha' is not maintained and is broken. These
270instructions need to be replaced or removed.
271
2ed0e35f
AD
272Running `make alpha' is absolutely perfect for beta releases: it makes
273the tarballs, the xdeltas, and prepares (in /tmp/) a proto
274announcement. It is so neat, that that's what I use anyway for
275genuine releases, but adjusting things by hand (e.g., the urls in the
276announcement file, the ChangeLog which is not needed etc.).
277
278If it fails, you're on your own...
279
280It requires GNU Make.
281
282** Upload
548e2104 283The generic GNU upload procedure is at:
2ed0e35f 284
548e2104
JD
285 http://www.gnu.org/prep/maintain/maintain.html#Automated-FTP-Uploads
286
015e86a7
JD
287Follow the instructions there to register your information so you're permitted
288to upload. Make sure your public key has been uploaded at least to
289keys.gnupg.net. You can upload it with:
290
291 gpg --keyserver keys.gnupg.net --send-keys F125BDF3
292
293where F125BDF3 should be replaced with your key ID.
294
295Here's a brief reminder of how to roll the tarballs and upload them:
548e2104
JD
296
297*** make distcheck
298*** gpg -b bison-2.3b.tar.gz
299*** In a file named `bison-2.3b.tar.gz.directive', type:
300
301 version: 1.1
302 directory: bison
303 filename: bison-2.3b.tar.gz
304
305*** gpg --clearsign bison-2.3b.tar.gz.directive
306*** ftp ftp-upload.gnu.org # Log in as anonymous.
307*** cd /incoming/alpha # cd /incoming/ftp for full release.
308*** put bison-2.3b.tar.gz # This can take a while.
309*** put bison-2.3b.tar.gz.sig
310*** put bison-2.3b.tar.gz.directive.asc
abd189e8 311*** Repeat all these steps for bison-2.3b.tar.xz.
2ed0e35f 312
5bc993d9
JD
313** Update Bison manual on www.gnu.org.
314
315*** You need a non-anonymous checkout of the web pages directory.
316
317 $ cvs -d YOUR_USERID@cvs.savannah.gnu.org:/web/bison checkout bison
318
319*** Get familiar with the instructions for web page maintainers.
320http://www.gnu.org/server/standards/readme_index.html
321http://www.gnu.org/server/standards/README.software.html
322especially the note about symlinks.
323
324*** Build the web pages.
325Assuming BISON_CHECKOUT refers to a checkout of the Bison dir, and
326BISON_WWW_CHECKOUT refers to the web directory created above, do:
327
328 $ cd $BISON_CHECKOUT/doc
329 $ make stamp-vti
330 $ ../build-aux/gendocs.sh -o "$BISON_WWW_CHECKOUT/manual" \
331 bison "Bison - GNU parser generator"
332 $ cd $BISON_WWW_CHECKOUT
333
334Verify that the result looks sane.
335
336*** Commit the modified and the new files.
337
338*** Remove old files.
339Find the files which have not been overwritten (because they belonged to
340sections that have been removed or renamed):
341
342 $ cd manual/html_node
343 $ ls -lt
344
345Remove these files and commit their removal to CVS. For each of these
346files, add a line to the file .symlinks. This will ensure that
347hyperlinks to the removed files will redirect to the entire manual; this
348is better than a 404 error.
349
350There is a problem with 'index.html' being written twice (once for POSIX
351function 'index', once for the table of contents); you can ignore this
352issue.
353
2ed0e35f 354** Announce
548e2104
JD
355To generate a template announcement file:
356
357 make RELEASE_TYPE=alpha gpg_key_ID=F125BDF3 announcement
358
5a3c69f1 359where alpha can be replaced by beta or stable and F125BDF3 should be
83e34842
JD
360replaced with your key ID.
361
362Complete/fix the announcement file. The generated list of recipients
363(info-gnu@gnu.org, bug-bison@gnu.org, help-bison@gnu.org,
364bison-patches@gnu.org, and coordinator@translationproject.org) is
365appropriate for a stable release or a ``serious beta''. For any other
366release, drop at least info-gnu@gnu.org. For an example of how to fill
367out the rest of the template, search the mailing list archives for the
368most recent release announcement.
369
370For a stable release, send the same announcement on the comp.compilers
371newsgroup by sending email to compilers@iecc.com. Do not make any Cc as
372the moderator will throw away anything cross-posted or Cc'ed. It really
373needs to be a separate message.
43a91d61 374
548e2104
JD
375** Bump the version number
376In configure.ac. Run `make'. So that developers don't accidentally add new
377items to the old NEWS entry, create a new empty NEWS entry something like:
378
379 Changes in version ?.? (????-??-??):
380
381Push these changes.
382
2ed0e35f
AD
383
384-----
385
c932d613 386Copyright (C) 2002-2005, 2007-2012 Free Software Foundation, Inc.
2ed0e35f
AD
387
388This file is part of GNU Bison.
389
f16b0819 390This program is free software: you can redistribute it and/or modify
2ed0e35f 391it under the terms of the GNU General Public License as published by
f16b0819
PE
392the Free Software Foundation, either version 3 of the License, or
393(at your option) any later version.
2ed0e35f 394
f16b0819 395This program is distributed in the hope that it will be useful,
2ed0e35f
AD
396but WITHOUT ANY WARRANTY; without even the implied warranty of
397MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
398GNU General Public License for more details.
399
400You should have received a copy of the GNU General Public License
f16b0819 401along with this program. If not, see <http://www.gnu.org/licenses/>.