Build fails because of alfred-json

[Adrian Schmutzler]

Hallo Fabian,

 

ich bin auch ein Tabs Fan, dann mach du gerne den Patch.

 

Grüße

 

Adrian

 

From: Fabian Bläse [mailto:fabian at blaese.de] 
Sent: Freitag, 1. Februar 2019 11:22
To: Adrian Schmutzler <mail at adrianschmutzler.de>; franken-dev at freifunk.net
Subject: Re: Build fails because of alfred-json

 

Hallo Adrian, 

On 01.02.19 10:44, Adrian Schmutzler wrote: 
> Make unterstützt keine leading spaces bei Kommandozeilen, das müssen Tabs sein! 
Richtig. 
Allerdings war ich gestern, als ich das als Problem identifiziert habe, sehr erstaunt darüber, dass das nur in der "compile" Definition bricht. In der "install" Definition tut das interessanterweise nicht weh. Vielleicht wird nur compile in ein Make Target umgewandelt? Oder die OpenWRT Make-Magie tut da noch Dinge...

> Entsprechend ist jetzt die Frage, ob ich einen Patch schicken soll, der nur für solche Kommandozeilen Tabs rein tut (und man dann Mischmasch hat) oder ob man gleich die ganzen Makefiles auf Tabs umstellt (= fast alle).

Ich bin ja sowieso fast immer für Tabs als Indentation, wenn ich gefragt werden. Das einzige, was dann unhübsch wird, sind die Multiline-Dependencies.

Da es sich bei den Dateien sowieso um Makefiles handelt (wenn auch hauptsächlich irgendwelche Definitions drin vorkommen) wäre ich für komplett auf Tabs umstellen.

Ich würde dafür auch einen Patch machen. 

Gruß 
Fabian 





-------------- nächster Teil --------------
Ein Dateianhang mit HTML-Daten wurde abgetrennt...
URL: <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20190201/a1067d4f/attachment.html>
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : nicht verfügbar
Dateityp    : application/pgp-signature
Dateigröße  : 834 bytes
Beschreibung: nicht verfügbar
URL         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20190201/a1067d4f/attachment.sig>