[PATCH]: Optional explanation for options

Magnus Therning magnus at therning.org
Fri Sep 16 10:29:12 BST 2005


On Fri, Sep 16, 2005 at 09:28:33AM +0200, Jan Hudec wrote:
>On Thu, Sep 15, 2005 at 12:58:27 -0400, John A Meinel wrote:
>> In the future, can you attach patches, rather than putting them inline?
>> It means that patchwork will find them, as well as making it easier to
>> just save the attachment, rather than the cut and paste dance.
>
>Just as a side note:
>1) Many others (most remarkably Linus) require patches inline -- and
>   than everyone needs to keep track of who wants them which way.
>2) Many mail clients won't properly detect text/x-diff format, so that's
>   not reliable for patchwork anyway. So it can as well look for patch
>   headers everywhere (meaning including in the "body").
>3) You don't need to do any cut-and-paste dance. Just save the body.
>   patch will skip the text around the diff proper.

And I'd like to add that patchwork seems to find inline patches just
fine as well:

 http://patchwork.ozlabs.org/bazaar-ng/patch?id=2401

However, I'm not religious about this. I'll be attaching rather than
inlining patches in the future since that seems to be preferred by
others on the list as well...

/M

-- 
Magnus Therning                    (OpenPGP: 0xAB4DFBA4)
magnus at therning.org
http://therning.org/magnus

Software is not manufactured, it is something you write and publish.
Keep Europe free from software patents, we do not want censorship
by patent law on written works.

The sort of person who uses phrases like `incompletely socialized'
usually thinks hackers are. Hackers regard such people with contempt
when they notice them at all.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20050916/0a06591f/attachment.pgp 


More information about the bazaar mailing list