[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 4/6] gnu: linux-libre-beagle-bone-black: Remove kernel varian
From: |
John Darrington |
Subject: |
Re: [PATCH 4/6] gnu: linux-libre-beagle-bone-black: Remove kernel variant. |
Date: |
Mon, 26 Sep 2016 20:00:17 +0200 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On Mon, Sep 26, 2016 at 01:31:09PM -0400, Leo Famulari wrote:
On Mon, Sep 26, 2016 at 06:49:34PM +0200, John Darrington wrote:
> Our current convention for commit messages is aesthetically pleasing,
> but useless. It is redundant to put in the message WHAT has changed.
> That can be easily determined by running "git show". What is much
> more usefull is the reason WHY we changed it.
I find the GNU changelog style to be very useful when reading the Git
log.
There are certain types of commits that we make over and over again,
especially with packages (updates, adding inputs, etc), and I find it
helpful when their commit messages follow a common format.
I agree that commits that don't conform to one of those types should
have some explanation. This is why I often rewrite commit messages when
I am committing on behalf of somebody else.
But there is a value to "aesthetic" consistency. It's just like having
standards for code style in my opinion. It helps a lot when reading
others' commits.
I don't object to a having a common style. I just wish that the information
that we put in our messages a) gave the rationale for the change; and b) didn't
merely repeat information which git show could have told me.
J'
--
Avoid eavesdropping. Send strong encrypted email.
PGP Public key ID: 1024D/2DE827B3
fingerprint = 8797 A26D 0854 2EAB 0285 A290 8A67 719C 2DE8 27B3
See http://sks-keyservers.net or any PGP keyserver for public key.
signature.asc
Description: Digital signature
- [PATCH 2/6] gnu: %default-extra-linux-options: Remove redundant options., (continued)
- [PATCH 2/6] gnu: %default-extra-linux-options: Remove redundant options., David Craven, 2016/09/26
- [PATCH 6/6] gnu: make-u-boot-package: Don't cross compile on arm., David Craven, 2016/09/26
- [PATCH 4/6] gnu: linux-libre-beagle-bone-black: Remove kernel variant., David Craven, 2016/09/26
- Re: [PATCH 4/6] gnu: linux-libre-beagle-bone-black: Remove kernel variant., John Darrington, 2016/09/26
- Re: [PATCH 4/6] gnu: linux-libre-beagle-bone-black: Remove kernel variant., David Craven, 2016/09/26
- Re: [PATCH 4/6] gnu: linux-libre-beagle-bone-black: Remove kernel variant., Vincent Legoll, 2016/09/26
- Re: [PATCH 4/6] gnu: linux-libre-beagle-bone-black: Remove kernel variant., Ricardo Wurmus, 2016/09/26
- Re: [PATCH 4/6] gnu: linux-libre-beagle-bone-black: Remove kernel variant., John Darrington, 2016/09/26
- Re: [PATCH 4/6] gnu: linux-libre-beagle-bone-black: Remove kernel variant., Vincent Legoll, 2016/09/26
- Re: [PATCH 4/6] gnu: linux-libre-beagle-bone-black: Remove kernel variant., Leo Famulari, 2016/09/26
- Re: [PATCH 4/6] gnu: linux-libre-beagle-bone-black: Remove kernel variant.,
John Darrington <=
- Re: [PATCH 4/6] gnu: linux-libre-beagle-bone-black: Remove kernel variant., David Craven, 2016/09/26
- Re: [PATCH 4/6] gnu: linux-libre-beagle-bone-black: Remove kernel variant., Vincent Legoll, 2016/09/26
- Re: [PATCH 4/6] gnu: linux-libre-beagle-bone-black: Remove kernel variant., David Craven, 2016/09/26
- Re: [PATCH 4/6] gnu: linux-libre-beagle-bone-black: Remove kernel variant., Leo Famulari, 2016/09/26
- Re: [PATCH 4/6] gnu: linux-libre-beagle-bone-black: Remove kernel variant., Ludovic Courtès, 2016/09/28
- Re: [PATCH 4/6] gnu: linux-libre-beagle-bone-black: Remove kernel variant., Tobias Geerinckx-Rice, 2016/09/27
- Re: [PATCH 4/6] gnu: linux-libre-beagle-bone-black: Remove kernel variant., Mark H Weaver, 2016/09/27
- Re: [PATCH 4/6] gnu: linux-libre-beagle-bone-black: Remove kernel variant., John Darrington, 2016/09/28
- Re: [PATCH 4/6] gnu: linux-libre-beagle-bone-black: Remove kernel variant., Ludovic Courtès, 2016/09/28
- Re: [PATCH 4/6] gnu: linux-libre-beagle-bone-black: Remove kernel variant., Ludovic Courtès, 2016/09/28