5 This file is intended to provide a few tips for anyone doing development on nmh.
6 Developers who learn things "the hard way" about the nmh codebase (as opposed to
7 local info best encoded in a comment) are encouraged to share their wisdom here.
9 Following a commit checklist, the topics are organized alphabetically.
17 3. make distcheck passed?
18 4. man page and other documentation updated?
19 5. docs/pending-release-notes updated?
20 6. should commit message reference bug report?
21 7. update/close bug report (with commit id)?
24 A buildbot at http://orthanc.ca:8010/waterfall polls for new commits and
25 builds them on a few platforms. Keep an eye on its progress in case
26 you've committed something non-portable. (If you can provide another
27 platform, contact the nmh-workers list.)
30 ---------------------------------
31 C library/system call usage notes
32 ---------------------------------
33 * Use m_mktemp2() or m_mktemp() instead of mkstemp(3) (see section on
34 nmh temporary files below).
35 * Use m_unlink() instead of unlink(3).
36 * Use done() instead of _exit(3) except after a fork(3).
39 -------------------------
40 autoconf & automake files
41 -------------------------
43 If you wish to change the `configure' script, the generated Makefile
44 or other related files, you'll need to first install GNU m4, available
45 from <ftp://ftp.gnu.org/pub/gnu/m4/>, then GNU autoconf
46 (<ftp://ftp.gnu.org/pub/gnu/autoconf/>) and GNU automake
47 (<ftp://ftp.gnu.org/pub/gnu/automake/>). Nmh is currently using a
48 minimum of autoconf 2.68 and automake 1.12.
50 Most of the configure-related files are automatically generated.
51 The only files you should need to manually edit are configure.ac
52 and any autoconf macros in the m4 directory. Don't, for instance,
53 edit config.h.in. Though it is an input file from the point of
54 view of the users (and the configure script) it is an output file
55 from the point of view of the developers (and the autoconf script).
57 If you wish to add a new autoconf macro, it should be placed in it's
58 own file and put in the m4 directory; aclocal will automatically pick
59 it up and automake will add it to the distribution target automatically.
61 If you wish to make changes to the Makefile, you will need to edit
62 Makefile.am. See the automake documentation if you need further help.
63 You should always check changes to Makefile.am by using "make distcheck".
65 Note that the automatically generated autotools files (such as config.h.in,
66 Makefile.in, and configure), are NOT kept in git. Thus, when you check out
67 a git tree, you need to run the autogen.sh script before you can build
77 Following is a list of nmh's directories along with a brief description of the
78 purpose of each one. Meanings are given for the abbreviations, but note that
79 these meanings are just informed guesses as to what the MH developers were
83 The top-level directory. Contains files like README and INSTALL.
86 Contains utility files for the `configure' process. Ordinarily nothing in
87 here needs to be messed with.
90 Contains more specialized documentation, such as this file and
94 Contains files, file templates, and scripts to generate files that will be
95 installed in the ${prefix}/etc directory. Stuff like replcomps.
98 Most of nmh's header (.h) files are kept not in the individual source
99 directories, but in this central location.
102 Contains all the input files that are processed to generate nmh's manual
106 "mts" stands for "Message Transfer Service". Source files specific to the
107 different MTSs go in the subdirectories.
110 When nmh is configured to just talk to an SMTP server over TCP/IP, the
111 source in this directory is compiled.
114 "sbr" stands for "subroutine(s)". For the most part, each source file in
115 this directory contains a single function with the same name as the source
116 file. These functions are of general use and are called from throughout
120 Contains files such as RPM specs.
123 The num unit test suite.
126 "tools" contains tools, scripts, and supporting files used by the
127 developers while writing, debugging, and testing the code.
130 "uip" stands for "User Interface Programs". Most nmh commands have a file
131 in this directory named <command>.c containing the code for that command
132 (e.g. repl.c). In some cases there is also an auxiliary file called
133 <command>sbr.c which contains additional subroutines called from <command>.c
134 (which would contain not much else besides main()).
141 As of December 2010, nmh has switched to using git for revision control
142 instead of CVS. While the topic of git is beyond the scope of this FAQ,
143 to get started with git & nmh, you can run the following command to checkout
144 the nmh repository (with read-only access to it):
146 % git clone git://git.savannah.nongnu.org/nmh.git
148 That will create a workspace called nmh. To update that workspace
149 with changes to the master, cd to it and run:
153 If you are a project member and want write access to the repository,
154 you'll have to checkout with the following command instead of the one
157 % git clone <username>@git.sv.nongnu.org:/srv/git/nmh.git
159 We suggest using git pull --rebase instead of the default merge for
160 git pull. If you don't want to add the --rebase option every time,
161 you can tell git pull to always rebase in your nmh workspace by
162 cd'ing to it and running the following command:
164 % git config --bool branch.master.rebase true
166 And you'll probably want the following, also, so that --rebase applies
167 to any new branches that you create:
169 % git config branch.autosetuprebase always
172 -------------------------------------------------------
173 nmh-local functions to use in preference to OS versions
174 -------------------------------------------------------
176 For some system functions whose availability or behavior varies from OS to OS,
177 nmh conditionally uses a local definition with the same name as the OS function
178 (e.g. snprintf()). For other functions, developers need to avoid the OS
179 versions and always use the nmh-supplied function. Here is a list of such
182 OS function nmh-local version to use instead
183 =========== ================================
184 getpass() nmh_getpass()
191 To create a temporary file, use m_mktemp2() or m_mktemp(). They use
192 mkstemp(3), but they also register the temporary file for removal on
193 program termination. So, do not use mkstemp() directly.
195 To further support this, nmh_init() must be called at the beginning of
196 main(). And, if a child process is not going to immediately call one
197 of the exec(3) functions or _exit(3) after a fork(3), it should call
198 unregister_for_removal(0). Finally, nmh_init() sets up signal handlers
199 for several signals: these signal handlers should not be disabled.
206 The nmh test suite is run through the Makefile, with "make check"
209 In the nmh test suite, nmh programs to be tested should be invoked
210 through the run_test or run_prog shell functions defined in
213 Instead of echoing test progress, use start_test()/finish_test()
214 from tests/common.sh. These will report the particular test name,
215 within the test, only if there is a failure.
217 To enable the use of valgrind, where available, set the environment
218 variable NMH_VALGRIND to a non-null value. However, a separate
219 environment variable, VALGRIND_ME, triggers the use of valgrind in
220 test/inc/test-eom-align because it greatly extends the duration of
223 If valgrind complains about "serious error when reading debuginfo"
224 from a library, either update or remove the debuginfo package for
225 the offending library.
232 To make a public release of nmh (we'll use version 1.5 as the example
233 here; the convention for release candidates is to use something like
236 1. Create a release branch. The convention is to name release branches
237 with the name "<version>-release".
239 % git branch 1.5-release
241 Note you are still on the master branch at this point. Mark the
242 current revision as the branchpoint for the new release branch:
244 % git tag -a -m "This tag marks the point where we started the branch for 1.5" 1.5-branchpoint
246 Now mark the master branch with a post-release version number (the
247 convention here is to use VERSION+dev as the version number).
249 % echo 1.5+dev > VERSION
256 % git checkout 1.5-release
258 You are now on the 1.5 release branch.
260 2. % echo 1.5 > VERSION
261 % date +"%e %B %Y" > DATE
262 (DATE should contain something like "30 December 2000")
264 3. % git commit VERSION DATE; git push
266 4. % git tag -a 1.5 -m 'Releasing nmh-1.5.'
269 Note that the new convention for tagging is to simply tag with the
270 version number (tag formats in the past have varied).
274 If you want to check the distribution build with some particular
275 configure options, set the DISTCHECK_CONFIGURE_FLAGS variable.
278 % make distcheck DISTCHECK_CONFIGURE_FLAGS=--with-cyrus-sasl
280 6. Upload the distribution file to savannah. You can automate this process
283 % make upload SAVANNAH_USERNAME=username
285 This will automatically call gpg to sign the release. You can bypass
286 this step by setting the SKIP_GPG_SIG variable.
288 7. Update the http://www.nongnu.org/nmh/ homepage. (It lives in the CVS
289 'webpages repository'; see https://savannah.nongnu.org/cvs/?group=nmh)
291 8. Add a news item to the savannah nmh page. You'll have to submit it first
292 and then separately approve it (under News->Manage).
294 9. Send the release announcement email to the following places:
295 nmh-workers@nongnu.org
296 nmh-announce@nongnu.org
297 exmh-users@redhat.com
298 exmh-workers@redhat.com
299 mh-e-users@lists.sourceforge.net
301 If the release fixes significant security holes, also send an announcement
302 to bugtraq@securityfocus.com. The exmh lists require you to be subscribed
303 in order to post. Note that you don't need to post separately to
304 comp.mail.mh, as the mh-users mailing list is apparently bidirectionally
307 Preferably, the announcement should contain the MD5 hash generated above,
308 and should be PGP-signed. It should include the URL for the tarball as
309 well as the URL of the website. It should contain a brief summary of
310 visible changes, as well as the URL of the git diff page that would show
311 a detailed list of changes. The changes between 1.5 and 1.4 would be
312 shown by [this is just a guess, I don't know anything about cgit, and
313 it assumes that we tag with nmh-x_x-release from now on]:
315 http://git.savannah.gnu.org/cgit/nmh.git/diff/?h=nmh-1_5-release?h=nmh-1_4-release