[PATCH 1/4] buildscript: Add variant variable, store variant in release-file

Fabian Bläse fabian at blaese.de
Mi Nov 20 17:56:37 CET 2019


Wow. Ich hätte nicht gedacht, dass es in meinem Umfeld noch Leute mit relevant höheren Ansprüchen an Commit Messages gibt, als mich. ;-D

Dieser Patch tut ja jetzt nichts wahnsinnig komplexes, die Commit Message beschreibt das ja eigentlich auch vollständig.
Man könnte jetzt noch dazu schreiben, wofür. Aber das ist dann ja eigentlich der nächste Patch.

Generell stelle ich fest, dass unsere Commit Messages eh schon recht umfangreicher sind, teilweise viel umfangreicher als in anderen Projekte, die ich kenne.
Ich bin aber auf jeden Fall auch für Messages, die das Problem und die Lösung ausreichend beschreiben.

Gruß
Fabian

On 20.11.19 17:31, Adrian Schmutzler wrote:
> Hi,
> 
>> buildscript: add variant information to firmware_release 
>> This also introduces a variant variable in our buildscript, as 
>> it is necessery multiple times in the build process. 
> 
> Gefällt mir besser.
> 
> Wenn es noch hinpasst, würde ich sogar explizit "/etc/firmware_release" schreiben.
> 
> Ich würde generell deutlich umfangreichere Commit Messages befürworten, aber das ist eine Grundsatzdiskussion und dafür ist dieser Patch sicher auch schlecht als Exempel geeignet.
> 
> Sollte man aber vll. mal diskutieren, wenn man sich Gedanken über Commit-/Pushregeln macht.
> 
> Grüße
> 
> Adrian
> 

-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 833 bytes
Beschreibung: OpenPGP digital signature
URL         : <https://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20191120/f0e560fe/attachment.sig>


Mehr Informationen über die Mailingliste franken-dev