[oe] [PATCH] eglibc-2.9: allow building with newer GNU make
Paul Menzel
paulepanter at users.sourceforge.net
Wed Sep 14 14:58:59 UTC 2011
Am Mittwoch, den 14.09.2011, 16:43 +0200 schrieb Steffen Sledz:
> On 14.09.2011 16:36, Paul Menzel wrote:
> >>>> The patch header is missing [1].
> >>>
> >>> Hmmm? After reading [1] i could not find what exactly is missing.
> >>
> >> Does the guidelines require that the patch files itself (make-3.82.patch in this case) need a header (in addition to the commit comment)?
> >
> > Yes, that is correct.
>
> It would be very helpful if [1] could be modified in a way that makes clear that it talk about to separate things: the commit message *and* the patch header.
>
> This was totally unclear for me after reading it.
I remarked that too when the guide lines were reviewed. And it is
explained in the second paragraph in the guideline [1].
This policy refers both to patches that are being applied by
recipes as well as commit messages that are part of the source
control system, usually git. A patch file needs a header in
order to describe the specific changes that are made within that
patch, while a commit message describes one or more changes to
the overall project or system. Both the patch headers and commit
messages require the same attention and basic details, however
the purposes of the messages are slightly different. A commit
message documents all of the changes made as part of a commit,
while a patch header documents items specific to a single patch.
In many cases the patch header can also be used as the commit
message.
Any suggestions are welcome though. And (the lame response) it is a Wiki
so everybody can improve the document.
Thanks,
Paul
> [1] http://openembedded.org/index.php?title=Commit_Patch_Message_Guidelines
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL: <http://lists.openembedded.org/pipermail/openembedded-devel/attachments/20110914/b0621049/attachment-0002.sig>
More information about the Openembedded-devel
mailing list