Re: [PATCH] Include the git way into Documentation/SubmittingPatches

From: Randy Dunlap
Date: Sat Jan 21 2012 - 14:39:43 EST


On 01/04/2012 10:05 AM, Stefan Beller wrote:
> I was quite confused when doing my first patch for linux
> and the documentation told me to use "diff -up" instead of
> git add/commit.
>
> Signed-off-by: Stefan Beller <stefanbeller@xxxxxxxxxxxxxx>


Can anyone comment on how accurate this is?
Thanks.


> ---
> Documentation/SubmittingPatches | 26 ++++++++++++++++++++++----
> 1 files changed, 22 insertions(+), 4 deletions(-)
>
> diff --git a/Documentation/SubmittingPatches b/Documentation/SubmittingPatches
> index 4468ce2..43acf99 100644
> --- a/Documentation/SubmittingPatches
> +++ b/Documentation/SubmittingPatches
> @@ -20,9 +20,14 @@ Documentation/SubmittingDrivers.
> SECTION 1 - CREATING AND SENDING YOUR CHANGE
> --------------------------------------------
>
> +The steps 1) - 3) can be either performed using the diff utility or
> +using git, the distributed revision control used by the kernel developers.
> +Using git is easier, but you need it installed of course.
> +So either follow steps 1a), 2a) and 3a) utilising the diff method
> +or just 3b) using git.
>
>
> -1) "diff -up"
> +1a) "diff -up"
> ------------
>
> Use "diff -up" or "diff -uprN" to create patches.
> @@ -82,8 +87,7 @@ Instead of these scripts, quilt is the recommended
> patch management
> tool (see above).
>
>
> -
> -2) Describe your changes.
> +2a) Describe your changes.
>
> Describe the technical detail of the change(s) your patch includes.
>
> @@ -111,7 +115,7 @@ If the patch fixes a logged bug entry, refer to
> that bug entry by
> number and URL.
>
>
> -3) Separate your changes.
> +3a) Separate your changes.
>
> Separate _logical changes_ into a single patch file.
>
> @@ -131,6 +135,20 @@ in your patch description.
> If you cannot condense your patch set into a smaller set of patches,
> then only post say 15 or so at a time and wait for review and integration.
>
> +3b) Using git
> +
> +If you have followed 1a) 2a) and 3a), skip this step.
> +
> +Use "git add -i" to add all separate _logical_ changes (as described in 3a)
> +to the index of git.
> +
> +Use "git commit" to commit all added changes into the revision control.
> +A texteditor (vi, nano or graphical) will appear, which should be used to
> +describe the changes. Describe the technical detail of the change(s)
> +your patch includes. (see 2a for explanation of "technical changes")
> +
> +Use "git format-patch HEAD~1" to put your changes and your description
> +into a single textfile.
>
>
> 4) Style check your changes.


--
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/