Re: Creating a patch set with git-format-patch

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, 10 Apr 2008 11:51:59 +0300
"Ian Brown" <ianbrn@xxxxxxxxx> wrote:

Hi Ian,

> I intend to  make a change in two files: net/ipv4/route.c and
> include/net/dst.h and send a patch set of two patches for review.
> Now I want to create this  patch set of two pathces correctly;
> First patch for net/ipv4/route.c and second for include/net/dst.h.
> 
> what should I do ?

The first step is to edit route.c and commit your change to Git with a complete
description[1].  Then edit the dst.h file and commit that to Git as a separate
change.  Afterward you can use git format-patch to prepare these commits to
be emailed:

$  git format-patch HEAD^^

This tells Git to create a	 text file in the current directory for each commit made
since the reference you pass.   You can then configure git send-email[2] to send
these text files out, or send them manually with your regular email client.  If you
do send them manually, make sure to include the text file inline in your email,
don't send it as an attachment.  You will find a subject line inside the file you can
cut-and-paste into place, and  a few other headers that you'll need to delete.

HTH,
Sean

[1]  Remember that each commit log entry should be made as a one-line overview,
followed by a blank line, followed by as much prose as it takes to fully explain the
change.

[2]  There is a small section about configuring git-send-email for use with gmail
using msmtp here:  http://git.or.cz/gitwiki/GitTips

--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux