[PATCH v3 1/2] packages/alfred-json: added package

[mail at adrianschmutzler.de]

Hallo,

 

applied ohne die unten genannte Änderung.

Die schicke ich auch nochmal extra.

 

Grüße

 

Adrian

 

From: franken-dev [mailto:franken-dev-bounces at freifunk.net] On Behalf Of Adrian Schmutzler
Sent: Montag, 21. Januar 2019 14:10
To: 'Fabian Bläse' <fabian at blaese.de>; franken-dev at freifunk.net
Subject: RE: [PATCH v3 1/2] packages/alfred-json: added package

 

Hallo,

 

dieser Patch liegt ja auch noch rum.

 

Reviews hat er ja genug, ich würde folgende Zeilen entfernen und ihn ansonsten applien, wie er ist:

 

+PKG_SOURCE:=$(PKG_NAME)-$(PKG_VERSION).tar.bz2

+PKG_SOURCE_SUBDIR:=$(PKG_NAME)-$(PKG_VERSION)

 

Bitte kurz ACK hierfür.

 

Die Description würde ich dann ggf. auch nochmal separat reinpatchen.

 

Grüße

 

Adrian

 

From: Fabian Bläse [mailto:fabian at blaese.de] 
Sent: Donnerstag, 27. Dezember 2018 00:13
To: mail at adrianschmutzler.de <mailto:mail at adrianschmutzler.de> ; franken-dev at freifunk.net <mailto:franken-dev at freifunk.net> ; 'Tim Niemeyer' <tim at tn-x.org <mailto:tim at tn-x.org> >
Subject: Re: [PATCH v3 1/2] packages/alfred-json: added package

 

Hallo, 

@Tim: Möchtest du dieses Paket weiter pflegen (siehe Anmerkungen von Adrian und Kommentare unten) oder soll ich das übernehmen?

On 12.08.18 18:42, mail at adrianschmutzler.de <mailto:mail at adrianschmutzler.de>  wrote: 
> Hallo, 
> 
> Reviewed-by: Adrian Schmutzler <freifunk at adrianschmutzler.de <mailto:freifunk at adrianschmutzler.de> > 
> 
> Bitte Kommentare unten beachten und ggf. noch anpassen. 
> 
> [..] 
> 
>> +PKG_SOURCE_VERSION:=e4cacfc791092389f63c36a435d3f1d069f8a13e 
>> +PKG_SOURCE:=$(PKG_NAME)-$(PKG_VERSION).tar.bz2 
> 
> Brauchen wir das, wenn wir eine git source haben? Ich glaube, das kann weg... (Wenn ja, dann nicht nur hier...) 
Richtig, das brauchts eigentlich nicht. 

>> +PKG_SOURCE_SUBDIR:=$(PKG_NAME)-$(PKG_VERSION) 
Könnte man eigentlich auch weglassen, ist default. 

>> +PKG_SOURCE_URL:=https://github.com/FreifunkFranken/alfred-json.git 
>> + 
>> +include $(INCLUDE_DIR)/package.mk 
>> +include $(INCLUDE_DIR)/cmake.mk 
>> + 
>> +define Package/alfred-json 
>> +    SECTION:=utils 
>> +    CATEGORY:=Utilities 
>> +    TITLE:= A.L.F.R.E.D - JSON Client 
>> +    URL:=https://github.com/FreifunkFranken/alfred-json 
> 
> Das ist scheinbar nur ein Info-Link. Insofern ist das so i.O. und mein Kommentar mit der Variable erübrigt sich. 
>> +    DEPENDS:=+jansson +zlib 
>> +endef 
> 
> Hier fehlt der description Block, wäre schon schön, ggf. Copy/Paste vom git repo. 
Könnte man noch einbauen. 

>> + 
>> +define Package/alfred-json/install 
>> +    $(INSTALL_DIR) $(1)/bin/ 
> 
> Warum liegt das jetzt ausgerechnet im /bin ? (Ich glaube, dass muss mir mal jemand außerhalb der Diskussion dieses Patches erklären.)

Dass es sowohl /(s)bin, als auch /usr/(s)bin gibt, ist eine UNIX Eigenheit (wie so oft) aus historischen Gründen. 
Die Besonderheit von /usr ist, dass es erst später eingehängt werden kann. (z.B. über NFS, oder einfach auf einer eigenen Partition/Platte)

Einige Software braucht man aber zwingend, bevor man mounten kann. (z.B. "mount") 
Wo etwas jetzt tatsächlich reingehört, darüber kann man vermutlich streiten. Wenn du mich fragst, müsste das hier eher nach /usr/bin...

> 
>> +    $(CP) $(PKG_INSTALL_DIR)/usr/bin/alfred-json $(1)/bin/ endef 
> 
> Keine Ahnung, warum der source-Pfad so heißen muss. Vielleicht kann mich jemand erleuchten. Da ich aber weiß, dass es funktioniert, ignoriere ich das mal für den Review.

make install packt alfred-json nach /usr/bin/alfred-json, $(PKG_INSTALL_DIR) verweist auf das Prefix des Install-Dirs. Daher muss das Binary von dort kopiert werden.

Fabian 






-------------- nächster Teil --------------
Ein Dateianhang mit HTML-Daten wurde abgetrennt...
URL: <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20190127/9d9c0613/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/20190127/9d9c0613/attachment.sig>