Working on a feature or bug fix? Great! This page tells you all you need to know to submit your patch.
First of all, these days development of collectd is organized on Github. If you don't have an account there already, we recomend you create one. It's free :-)
From within a Git repository
Of course the first step is "cloning" collectd's Git repository using the following command:
git clone git://github.com/collectd/collectd.git
After you "cloned" collectd's Git repository and made your changes you should send them in using the normal git-format-patch(1) and git-send-email(1) procedure. Please send them directly to collectd's mailing list at firstname.lastname@example.org. Mails sent by non-subscribers will be held for approval which will typically happen within 24 hours.
If you're new to Git you might want to read the following documents to get started:
Patching a certain branch
It's possible that you're asked to make your changes against a certain branch of the repository. To do this, you need to first fetch the branch into your local copy of the repository (here the local branch has the name "foo/branch-origin"):
git fetch origin foo/branch:foo/branch-origin
Then you create a copy of that branch (here it's named "foo/branch"), to which you will make your changes:
git checkout -b foo/branch foo/branch-origin
After making your changes to the "foo/branch"-branch you will need to create a (set of) patch(es) and send them in:
git format-patch -o output-directory -s foo/branch-origin..foo/branch git send-email --to email@example.com output-directory
Patching a release
If you don't want to work out the complexities of Git (or you read this page after you made the changes ;) you can take some release (at best the latest one) and make your changes there.
I'll outline the typical steps one does when making the changes:
# Get the tarball wget http://collectd.org/files/collectd-version.tar.bz2 # Unpack and rename the sources tar jxf collectd-version.tar.bz2 cp -r collectd-version collectd-version-mine # Do changes to collectd-version-mine # Create the patch diff -pur collectd-version collectd-version-mine >collectd-version-mine.patch
So the steps are typically:
- Download the latest release.
- Extract the source code and make a copy of the entire directory.
- Make your changes, a.k.a. *magic*.
- Test your changes.
- Do a
make distcleanto clean up the generated files.
- Generate the patch with
diff -pur, as outlined above.
- Test if the
.patchfile applies cleanly.
- Review the
- Submit the patch by sending an email to collectd's mailinglist at firstname.lastname@example.org. Mails sent by non-subscribers will be held for approval.
Other random notes
- If you patch a release, please don't submit changes to files such as
configureand other files that are generated by the autotools. A (incomplete) list of uninteresting files can be found in the .gitignore file.
- Send one patch for each change in functionality. If they're too fine-grained it's easy to merge them together. The other way around is a pain in the back.
- Write code that blends in well (i.e. (try to) stick to the coding style). I won't be too picky about that, but keep it in mind please.
- Please don't compress your patches in any way. If the moderation filter holds back your mail because it's too big, please be patient: Unless it's huge I'll let it through. If I don't seem to take any action please drop me a note: There's quite some spam in the moderation filter, so I may have missed your mail.
- While 99.9 % of the code is just plain ASCII, there are situations in which you need non-ASCII characters, such as adding your name to the copyright notice at the top of each file, to the AUTHORS file or to the ChangeLog file. In this case, please make sure that the character set used is UTF-8. At the same time: Please write your name as it is supposed to be written, not some ASCII approximation of it – we consider this to be more polite.