diff options
author | Darren Salt <linux@youmustbejoking.demon.co.uk> | 2009-01-15 15:29:13 +0000 |
---|---|---|
committer | Darren Salt <linux@youmustbejoking.demon.co.uk> | 2009-01-15 15:29:13 +0000 |
commit | 2c07c66cf1be3a8aad70f34fff32592a96c5df41 (patch) | |
tree | 214bb2b1e6812cdface1a39c89da252454fe93ef /doc/hackersguide | |
parent | 9776c5dad4e77cb84f27961a50cc24993785e54a (diff) | |
download | xine-lib-2c07c66cf1be3a8aad70f34fff32592a96c5df41.tar.gz xine-lib-2c07c66cf1be3a8aad70f34fff32592a96c5df41.tar.bz2 |
Document patch submission preferences.
Diffstat (limited to 'doc/hackersguide')
-rw-r--r-- | doc/hackersguide/overview.sgml | 26 |
1 files changed, 26 insertions, 0 deletions
diff --git a/doc/hackersguide/overview.sgml b/doc/hackersguide/overview.sgml index d405ac18c..c8d15a4f9 100644 --- a/doc/hackersguide/overview.sgml +++ b/doc/hackersguide/overview.sgml @@ -806,6 +806,32 @@ working on other parts of the code or are simply busy at the moment). </para> + <para> + We prefer to receive patches in a format which is immediately + committable. This normally means that we want your name and email address + (we're not keen on pseudonyms), a subject line which provides a + convenient summary of your changes, and body text which provides a longer + description (if needed). Patches must be generated using <command>hg + diff</command> or <command>cvs -z9 diff</command>, depending on which is + used for the repository, and always from within the top level of the + checked-out source. + </para> + <para> + If your patch is intended for a Mercurial-format repository, then + committing it locally and using + <command>hg export <parameter>.</parameter> ><filename>foo.patch</filename></command> + to create the file to be attached is acceptable (but note that the first + line - up to the first line feed - of the commit message is regarded as + the patch summary; if you're not sure, check with + <command>hg log -r<parameter>.</parameter></command>. + Alternatively, use <token>From:</token> and <token>Subject:</token> lines + in the patch file itself; this is useful when forwarding. + </para> + <para> + Patches should normally be included as attachments. Some mail clients and + webmail services are known to mangle whitespace or wrap lines, either of + which is enough to render a patch potentially useless. + </para> </sect1> </chapter> |