kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #31324
Re: [PATCH 00/12] Minor cleanups and improvements
-
To:
<kicad-developers@xxxxxxxxxxxxxxxxxxx>
-
From:
Maciej Sumiński <maciej.suminski@xxxxxxx>
-
Date:
Fri, 3 Nov 2017 14:08:40 +0100
-
Authentication-results:
spf=pass (sender IP is 188.184.36.48) smtp.mailfrom=cern.ch; lists.launchpad.net; dkim=none (message not signed) header.d=none;lists.launchpad.net; dmarc=bestguesspass action=none header.from=cern.ch;
-
In-reply-to:
<9b0688ba-8a64-3d74-4b93-4fdad17bd6a7@t-online.de>
-
Spamdiagnosticmetadata:
NSPM
-
Spamdiagnosticoutput:
1:99
-
User-agent:
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0
On 11/03/2017 01:38 PM, Carsten Schoenert wrote:
> Hello Marvin,
>
> Am 02.11.2017 um 21:58 schrieb Marvin Schmidt:
>> Just a set of small code cleanups to fix a bunch of warnings
>> and remove dead code.
>> Other than that the 'build:' commits avoid/remove/fix the
>> installation of certain files:
>> - it doesn't make much sense to install the INSTALL.txt
>> - *.bat scripts shouldn't be installed on UNIX systems
>> - use CMAKE_INSTALL_* consistently to install files in the
>> correct places
> wouldn't it make more sense to write the specific reason for the commits
> *into* the specific commit?
> This email with your explanation why you have done all that work will go
> down under in the typical noise of the mailing list and nobody but you
> will later know why the single commit was made.
I think the e-mail subject contains the first line of commit message
that explains the changes and most likely is handled correctly by
git-am. Personally, I prefer patches sent as attachments as I do not use
a client that is compatible with git-am. I also wonder whether git-am
chokes on the lines add by launchpad in every message.
Apart from that, I think the proposed changes are reasonable and could
be merged.
Regards,
Orson
Attachment:
signature.asc
Description: OpenPGP digital signature
Follow ups
References