[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 3/5] build: Add 'emacs-build-system'
From: |
Ludovic Courtès |
Subject: |
Re: [PATCH 3/5] build: Add 'emacs-build-system' |
Date: |
Thu, 25 Jun 2015 13:57:09 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Federico Beffa <address@hidden> skribis:
> On Mon, Jun 22, 2015 at 9:40 PM, Thompson, David
> <address@hidden> wrote:
>> On Mon, Jun 22, 2015 at 3:33 PM, Federico Beffa <address@hidden> wrote:
>>>
>>> But, my question was NOT: how can I see white spaces. Rather: is there
>>> a Guix coding style "rule" which states that white spaces there are
>>> undesired.
>>>
>>> I personally prefer to have them, because then, if I use M-up/down, I
>>> move to the beginning/end of a whole top-level block, without stopping
>>> at internal points and that's what I want most of the time.
>>>
>>> So, these spaces are not just coding artifacts, but have some use.
>>
>> There should be *no* trailing whitespace in submitted patches, and we
>> should add a note about it to our contribution guidelines if it's not
>> already there.
+1
> OK, I will delete those spaces then. But, I'm curious about the
> rationale for such a rule.
It’s mostly that no-trailing-whitespace is a simple canonical form.
Having everyone follow it makes sure we don’t run into annoying patch
conflicts due to whitespace, nor “noisy patches” that remove trailing
spaces here and there.
Ludo’.
- [PATCH 3/5] build: Add 'emacs-build-system', Federico Beffa, 2015/06/21
- Re: [PATCH 3/5] build: Add 'emacs-build-system', Alex Kost, 2015/06/21
- Re: [PATCH 3/5] build: Add 'emacs-build-system', Federico Beffa, 2015/06/22
- Re: [PATCH 3/5] build: Add 'emacs-build-system', Mathieu Lirzin, 2015/06/22
- Re: [PATCH 3/5] build: Add 'emacs-build-system', Alex Kost, 2015/06/22
- Re: [PATCH 3/5] build: Add 'emacs-build-system', Federico Beffa, 2015/06/22
- Re: [PATCH 3/5] build: Add 'emacs-build-system', Thompson, David, 2015/06/22
- Re: [PATCH 3/5] build: Add 'emacs-build-system', Federico Beffa, 2015/06/23
- Re: [PATCH 3/5] build: Add 'emacs-build-system',
Ludovic Courtès <=
- Re: [PATCH 3/5] build: Add 'emacs-build-system', Federico Beffa, 2015/06/25
- Re: [PATCH 3/5] build: Add 'emacs-build-system', Alex Kost, 2015/06/23
- Re: [PATCH 3/5] build: Add 'emacs-build-system', Federico Beffa, 2015/06/24
- Re: [PATCH 3/5] build: Add 'emacs-build-system', Ludovic Courtès, 2015/06/25
- Re: [PATCH 3/5] build: Add 'emacs-build-system', Federico Beffa, 2015/06/25
- Re: [PATCH 3/5] build: Add 'emacs-build-system', Ludovic Courtès, 2015/06/27