Re: [PATCH] Checklist clarifications: added "Commits", "Patch:" headings

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

 



Hi,

On Mon, 30 Apr 2007, Julian Phillips wrote:

> On Mon, 30 Apr 2007, Jari Aalto wrote:
> 
> > Is this workding okay?
> > 
> >        - send your patch to <git@xxxxxxxxxxxxxxx> by using
> >          git-format-patch(1) with option "-M". If you use
> >          git-send-email(1), please test it first by sending
> >          email to yourself.
> 
> I don't want to seem picky, but that wording suggests to me that the -M
> option to format-patch magically sends patches to the list.  Why not just
> have:
> 
> 	- use "git format-patch -M" to create the patch
> 	- send your patch to <git@xxxxxxxxxxxxxxx>. If you use
> 	  git-send-email(1), please test it first by sending
> 	  email to yourself.

Yes, I like that much better!

Thank you,
Dscho

-
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]