]> git.saurik.com Git - bison.git/blame_incremental - README-hacking
tests: save/restore Autotest special files when checking XML support.
[bison.git] / README-hacking
... / ...
CommitLineData
1-*- outline -*-
2
3This file attempts to describe the rules to use when hacking Bison.
4Don't put this file into the distribution.
5
6Everything related to the development of Bison is on Savannah:
7
8 http://savannah.gnu.org/projects/bison/
9
10
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
18** If a change fixes a test, mention the test in the commit message.
19
20** Bug reports
21If somebody reports a new bug, mention his name in the commit message
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
28
29* Hacking
30
31** Visible changes
32Which include serious bug fixes, must be mentioned in NEWS.
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.
39
40
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
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
163These files don't change very often in Autoconf, so it should be
164relatively straight-forward to examine the differences in order to
165decide whether to update.
166
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
180- Build with -DGNULIB_POSIXCHECK. It suggests gnulib modules that can
181 fix portability issues.
182
183- Check with `make syntax-check' if there are issues diagnosed by
184 gnulib.
185
186- run `make maintainer-check' which:
187 - runs `valgrind -q bison' to run Bison under Valgrind.
188 - runs the parsers under Valgrind.
189 - runs the test suite with G++ as C compiler...
190
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
203- running `make maintainer-release-check' takes care of running
204 maintainer-check, maintainer-push-check and maintainer-xml-check.
205
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,
209 but currently is actually GNU C, nor for lalr1.cc.
210
211
212* Release Procedure
213
214** Update the submodules. See above.
215
216** Update maintainer tools, such as Autoconf. See above.
217
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 '\<_(' *
223
224** Tests
225See above.
226
227** Update the foreign files
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.
232
233** Update README
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.
245
246** Update NEWS
247The version number, *and* the date of the release (including for
248betas).
249
250** Mention the release name in a commit message
251Should have an entry similar to `Version 2.3b.'.
252
253** Tag the release
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:
257
258 git tag -a v2.3b
259
260The commit message can be simply:
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.
267
268** make alpha
269FIXME: `make alpha' is not maintained and is broken. These
270instructions need to be replaced or removed.
271
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
283The generic GNU upload procedure is at:
284
285 http://www.gnu.org/prep/maintain/maintain.html#Automated-FTP-Uploads
286
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:
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
311*** Repeat all these steps for bison-2.3b.tar.xz.
312
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
354** Announce
355To generate a template announcement file:
356
357 make RELEASE_TYPE=alpha gpg_key_ID=F125BDF3 announcement
358
359where alpha can be replaced by beta or stable and F125BDF3 should be
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.
374
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
383
384-----
385
386Copyright (C) 2002-2005, 2007-2012 Free Software Foundation, Inc.
387
388This file is part of GNU Bison.
389
390This program is free software: you can redistribute it and/or modify
391it under the terms of the GNU General Public License as published by
392the Free Software Foundation, either version 3 of the License, or
393(at your option) any later version.
394
395This program is distributed in the hope that it will be useful,
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
401along with this program. If not, see <http://www.gnu.org/licenses/>.