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

[mail at adrianschmutzler.de]

Hallo,

Reviewed-by: Adrian Schmutzler <freifunk at adrianschmutzler.de>

Bitte Kommentare unten beachten und ggf. noch anpassen.

> -----Original Message-----
> From: franken-dev [mailto:franken-dev-bounces at freifunk.net] On Behalf
> Of Tim Niemeyer
> Sent: Samstag, 11. August 2018 19:54
> To: franken-dev at freifunk.net
> Subject: [PATCH v3 1/2] packages/alfred-json: added package
> 
> From: Steffen Pankratz <kratz00 at gmx.de>
> 
> Signed-off-by: Tim Niemeyer <tim at tn-x.org>
> - Changed subject
> - Move alfred-json to own repo
> Reviewed-by: Fabian Bläse <fabian at blaese.de>
> Tested-by: Fabian Bläse <fabian at blaese.de>
> 
> ---
> 
> Changes in v2:
> - move alfred-json to own repo
> Series-changes: 3
> - fix url
> 
>  src/packages/fff/alfred-json/Makefile | 29
> +++++++++++++++++++++++++++++
>  1 file changed, 29 insertions(+)
>  create mode 100644 src/packages/fff/alfred-json/Makefile
> 
> diff --git a/src/packages/fff/alfred-json/Makefile b/src/packages/fff/alfred-
> json/Makefile
> new file mode 100644
> index 0000000..dfca763
> --- /dev/null
> +++ b/src/packages/fff/alfred-json/Makefile
> @@ -0,0 +1,29 @@
> +include $(TOPDIR)/rules.mk
> +
> +PKG_NAME:=alfred-json
> +PKG_VERSION:=0.0.1
> +PKG_RELEASE:=1
> +PKG_LICENSE:=GPL-2.0
> +PKG_SOURCE_PROTO:=git
> +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...)

> +PKG_SOURCE_SUBDIR:=$(PKG_NAME)-$(PKG_VERSION)
> +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.

> +
> +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.)

> +	$(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.

Grüße

Adrian



> +
> +$(eval $(call BuildPackage,alfred-json))
> --
> 2.11.0