https://bugzilla.kernel.org/show_bug.cgi?id=43641





--- Comment #13 from Jonathan Nieder <jrnie...@gmail.com>  2012-07-10 05:33:02 
---
Lin Ming wrote:

> Created an attachment (id=75161)
 --> (https://bugzilla.kernel.org/attachment.cgi?id=75161)
>  --> (https://bugzilla.kernel.org/attachment.cgi?id=75161)
> patch from Jonathan and updated by Bob

Thanks.  The new commit message is worse in almost every way than the
original:

 - the shortlog line focuses on removal of code instead of the effect
   of that removal of code

 - it does not explain the context and how we know that this will work
   and will not cause a regression

 - it does not mention that this fixes a regression

 - it does not mention which commit introduced the regression, so
   distro maintainers will have more trouble seeing whether they need
   the patch

 - it does not cc: stable@

-- 
Configure bugmail: https://bugzilla.kernel.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
acpi-bugzilla mailing list
acpi-bugzilla@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla

Reply via email to