documentation/patches.sgml

   1   <chapter id="patches">
   2     <title>Submitting Patches</title>
   3
   4     <para>
   5       Written by &name-albert-den-haan; <email>&email-albert-den-haan;</email>
   6     </para>
   7     <sect1 id="patch-format">
   8       <title>Patch Format</title>
   9
  10       <para>
  11         Your patch should include:
  12       </para>
  13
  14       <itemizedlist>
  15         <listitem>
  16           <para>
  17             a description of what was wrong and what is now better
  18             (and now broken :).
  19           </para>
  20         </listitem>
  21         <listitem>
  22           <para>
  23             your contact information ( Name/Handle and e-mail )
  24           </para>
  25         </listitem>
  26         <listitem>
  27           <para>
  28             the patch in <command>diff -u</command> format (it happens...)
  29           </para>
  30         </listitem>
  31       </itemizedlist>
  32
  33       <para>
  34         <command>cvs diff -u</command> works great for the common case
  35         where a file is edited.  However, if you add or remove a file
  36         <command>cvs diff</command> will not report that correctly so
  37         make sure you explicitly take care of this rare case.
  38       </para>
  39       <para>
  40         For additions: mention that you have some new files and
  41         include them as either separate attachments or by appending
  42         <command>diff -Nu</command> of them to any <command>cvs diff
  43           -u</command> output you may have.
  44       </para>
  45       <para>
  46         For removals, list the files.
  47       </para>
  48     </sect1>
  49
  50     <sect1 id="patch-quality">
  51       <title>Quality Assurance</title>
  52
  53       <para>
  54         (Or, "How do I get Alexandre to apply my patch quickly so I
  55         can build on it and it will not go stale?")
  56       </para>
  57       <para>
  58         Make sure your patch applies to the current CVS head
  59         revisions.  If a bunch of patches are commited to CVS that may
  60         affect whether your patch will apply cleanly then verify that
  61         your patch does apply!   <command>cvs update</command> is your
  62         friend!
  63       </para>
  64       <para>
  65         Save yourself some embarasment and run your patched code
  66         against more than just your current test example.  Experience
  67         will tell you how much effort to apply here.
  68       </para>
  69
  70     </sect1>
  71   </chapter>
  72
  73 <!-- Keep this comment at the end of the file
  74 Local variables:
  75 mode: sgml
  76 sgml-parent-document:("wine-doc.sgml" "set" "book" "part" "chapter" "")
  77 End:
  78 -->