[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: |
Leo Famulari |
Subject: |
Re: [PATCH 4/6] gnu: linux-libre-beagle-bone-black: Remove kernel variant. |
Date: |
Mon, 26 Sep 2016 13:31:09 -0400 |
User-agent: |
Mutt/1.7.0 (2016-08-17) |
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.
signature.asc
Description: PGP signature
- Re: [PATCH 5/6] gnu: make-u-boot-package: Add files-to-install argument., (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 <=
- 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., 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