[PATCH libpciaccess] config: provide a makefile for scanpci dev tool

Gaetan Nadon memsize at videotron.ca
Sun Sep 26 12:31:39 PDT 2010


On Sun, 2010-09-26 at 13:42 +0300, Vignatti Tiago (Nokia-MS/Helsinki)
wrote:

> "A commit should contain exactly one logical change. A logical change
> includes
> adding a new feature, fixing a specific bug, etc. If it's not possible
> to
> describe the high level change in a few words, it is most likely too
> complex
> for a single commit. The diff itself should be as concise as
> reasonably
> possibly and it's almost always better to err on the side of too many
> patches
> than too few. As a rule of thumb, given only the commit message,
> another
> developer should be able to implement the same patch in a reasonable
> amount of
> time." - from
> http://who-t.blogspot.com/2009/12/on-commit-messages.html.
> 
>              Tiago
> 

I agree. The more code changes in the patch, the higher the likelihood
of both the author 
and the reviewer of overlooking an issue.

Just like "good programming practices", "good patching practices" is not
an exact science
and will always be the subject of debate.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.x.org/archives/xorg-devel/attachments/20100926/7681bcc8/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part
URL: <http://lists.x.org/archives/xorg-devel/attachments/20100926/7681bcc8/attachment.pgp>


More information about the xorg-devel mailing list