[oe] meta-oe: Patch review/quality and new and new commit policy

[Paul Menzel]

Dear OE folks,


looking at the commit log of meta-oe, I see two issues.

1. A lot of these patches were never sent to *this* list like [2].
2. The commit message are not adhering to the guide lines [1] and are
therefore not as useful as they could be.

If I remember correctly one reason for the new structure was to also
improve the quality of OpenEmbedded. And as there is nobody having any
time to update documentation commit messages are the only means right
now to document things correctly. I know that a lot of you are probably
annoyed by me saying guide lines, guide lines, guide lines all over
again, but I have not seen any other suggestions yet and looking at the
current meta-oe state I doubt that the quality is that much better than
before. The only benefit I can see is that there are not so many
versions of the same recipe.

Since nobody else did so already I propose the following policy to get
patches into meta-oe. They should be in effect immediately and can be
improved/adapted over time.

1. Commits have to adhere to the Commit Patch Message Guidelines.
2. It is advisable to add to the commit message how the patch was tested
(build, run time, build system, distribution, machine).
3. Patches have to be send for review (preferably with a pull request)
to openembedded-devel. After one week without any objections they can be
pulled into the repository.
4. The state in the patch queue Patchwork¹ has to be updated.

Thanks,

Paul


¹ As soon as it works correctly again.

[1] http://openembedded.org/index.php?title=Commit_Patch_Message_Guidelines
[2] http://www.mail-archive.com/openembedded-core@lists.openembedded.org/msg08136.html
-------------- 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/20110915/86aee10b/attachment-0002.sig>