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

Fabian Bläse fabian at blaese.de
Do Dez 27 00:12:50 CET 2018


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 wrote:
> Hallo,
> 
> Reviewed-by: Adrian Schmutzler <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 Binärdaten wurde abgetrennt...
Dateiname   : signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 833 bytes
Beschreibung: OpenPGP digital signature
URL         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20181227/cff05f0f/attachment.sig>


Mehr Informationen über die Mailingliste franken-dev