]> xenbits.xensource.com Git - libvirt.git/commitdiff
docs: remove outdated suggestion to make patches with "diff -urp"/"git diff"
authorLaine Stump <laine@laine.org>
Wed, 29 Jun 2016 19:36:31 +0000 (15:36 -0400)
committerLaine Stump <laine@laine.org>
Fri, 1 Jul 2016 16:41:10 +0000 (12:41 -0400)
I can't think of any good reason to do either of those, and having the
examples there will just lead to unusable patch emails from people who
can't be bothered to read the entire page.

HACKING
docs/hacking.html.in

diff --git a/HACKING b/HACKING
index 0aba279bf73e89fcb6125161129a63292565d891..36564118e58e675eb618ec958a85afaa73499cb3 100644 (file)
--- a/HACKING
+++ b/HACKING
@@ -23,19 +23,11 @@ libvirt-python) online <http://libvirt.org/git/>.
 join the appropriate language team. The libvirt release process automatically
 pulls the latest version of each translation file from zanata.
 
-(4) Post patches in unified diff format, with git rename detection enabled. You
+(4) Post patches using "git send-email", with git rename detection enabled. You
 need a one-time setup of:
 
   git config diff.renames true
 
-After that, a command similar to this should work:
-
-  diff -urp libvirt.orig/ libvirt.modified/ > libvirt-myfeature.patch
-
-or:
-
-  git diff > libvirt-myfeature.patch
-
 Also, for code motion patches, you may find that "git diff --patience"
 provides an easier-to-read patch. However, the usual workflow of libvirt
 developer is:
index a471d88552f91dd20afdfc911ebdef9a3035503f..5f19143408d555bc39a52ee7b13eedd854057331 100644 (file)
         automatically pulls the latest version of each translation
         file from zanata.</li>
 
-      <li><p>Post patches in unified diff format, with git rename
+      <li><p>Post patches using "git send-email", with git rename
         detection enabled.  You need a one-time setup of:</p>
 <pre>
   git config diff.renames true
-</pre>
-        <p>After that, a command similar to this should work:</p>
-<pre>
-  diff -urp libvirt.orig/ libvirt.modified/ &gt; libvirt-myfeature.patch
-</pre>
-        <p>
-          or:
-        </p>
-<pre>
-  git diff &gt; libvirt-myfeature.patch
 </pre>
         <p>Also, for code motion patches, you may find that <code>git
         diff --patience</code> provides an easier-to-read patch.