Submitting Patches

From GeeklogWiki
Revision as of 18:03, 3 May 2010 by Dirk (talk | contribs) (to get a patch file, you need to redirect the output of hg export ...)

(diff) ← Older revision | Latest revision (diff) | Newer revision → (diff)
Jump to: navigation, search

If you want to submit a patch for Geeklog, please check out the current development version from our Mercurial repository. You can also submit patches against released versions by checking out that version's code using the ..._stable tag.

Checklist

  • check out the source code
  • make the necessary changes to the source
  • commit changes into your local repository
  • Exporting:
    • export the changeset using hg export REV where REV is the revision number of the changeset
      • to export the latest changeset from your repository, simply use
        hg export tip > patch.diff
      • The TortoiseHg GUI equivalence is: Hg Repository Explorer=>choose Changeset=>Export Patch
  • if your patch consists of more than one changeset, please export them individually into separate files with consecutive numbers so we know in which order they need to be applied (Note: while hg export can export multiple changesets into one file, the equivalent hg import currently only accepts one changeset per file, so we're asking for separate files.)
  • Diff files:
    • if you are not using hg export please send unified diff files and make sure you list the files in the correct order, i.e.
      diff -u original-file modified-file > patch.diff
    • The TortoiseHg GUI equivalence is: Hg Commit=>Diff (?)
  • add the patch (or zipped/gzipped set of patches) as an attachment to the bug or feature request in question on our bugtracker
    • if there is no bug or feature request for your patch, send the patch (with an explanation) to the geeklog-devel mailing list (if the patch is really huge, you may want to upload it somewhere and only send the URL)

Thank you!

diff vs. hg export

You may be wondering why we're asking you to use hg export instead of the more common diff files. That is because export files

  • include the checkin comment
  • give proper attribution to the person who created the patch (i.e. you)