diff options
author | gdt <gdt> | 2003-12-19 20:20:25 +0100 |
---|---|---|
committer | gdt <gdt> | 2003-12-19 20:20:25 +0100 |
commit | d9fd04c24bb6e6fc9aaca6daf5c062beced2605f (patch) | |
tree | 72499a69f58e24a14e085614b9928825b6e17e06 /HACKING | |
parent | {lib,ospfd,ospfapi}/Makefile.am: (diff) | |
download | frr-d9fd04c24bb6e6fc9aaca6daf5c062beced2605f.tar.xz frr-d9fd04c24bb6e6fc9aaca6daf5c062beced2605f.zip |
rough cut at committing guidelines
Diffstat (limited to 'HACKING')
-rw-r--r-- | HACKING | 69 |
1 files changed, 69 insertions, 0 deletions
diff --git a/HACKING b/HACKING new file mode 100644 index 000000000..d215824fc --- /dev/null +++ b/HACKING @@ -0,0 +1,69 @@ +-*- mode: text; -*- + +$Id: HACKING,v 1.1 2003/12/19 19:20:25 gdt Exp $ + +GUIDELINES FOR HACKING ON QUAGGA + +[this is a strawman on which consensus has been neither tested nor reached] + +[this is a draft in progress] + +Generally, GNU coding standards apply. The indentation style is a bit +different from standard GNU style, and the existing style should be +maintained and used for new code. + +PATCH SUBMISSION + +* Send a clean diff against the head of CVS. + +* Include ChangeLog and NEWS entries as appropriate before the patch + (or in it if you are 100% up to date). + +* Inclue only one semantic change or group of changes per patch.p + +* Do not make gratuitous changes to whitespace. + +* State on which platforms and with what daemons the patch has been + tested. Understand that if the set of testing locations is small, + and the patch might have unforeseen or hard to fix consequences that + there may be a call for testers on quagga-dev, and that the patch + may be blocked until test results appear. + + If there are no users for a platform on quagga-dev who are able and + willing to verify -current occasionally, that platform may be + dropped from the "should be checked" list. + +PATCH APPLICATION TO CVS + +* Only apply patches that meet the submission guidelines. + +* If a patch is large (perhaps more than 100 new/changed lines), tag + the repository before and after the change with e.g. before-foo-fix + and after-foo-fix. + +* If the patch might break something, issue a call for testing on the + mailinglist. + +* By committing a patch, you are responsible for fixing problems + resulting from it (or backing it out). + +STABLE PLATFORMS AND DAEMONS + +The list of platforms that should be tested follow. This is a list +derived from what quagga is thought to run on and for which +maintainers can test or there are people on quagga-dev who are able +and willing to verify that -current does or does not work correctly. + + BSD (Free, Net or Open, any platform) # without capabilities + GNU/Linux (any distribution, i386) + [future: some 64-bit machine, e.g. NetBSD/sparc64] + [Solaris? (could address 64-bit issue)] + +The list of daemons that are thought to be stable and that should be +tested are: + + zebra + bgpd + ripd + ospfd + ripngd |