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