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