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

Submitted by Tim Niemeyer on Aug. 11, 2018, 5:53 p.m.

Details

Message ID 20180811175339.27545-1-tim@tn-x.org
State Accepted
Headers show

Commit Message

Tim Niemeyer Aug. 11, 2018, 5:53 p.m. 
From: Steffen Pankratz <kratz00@gmx.de>

Signed-off-by: Tim Niemeyer <tim@tn-x.org>
- Changed subject
- Move alfred-json to own repo
Reviewed-by: Fabian Bläse <fabian@blaese.de>
Tested-by: Fabian Bläse <fabian@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

Patch hide | download patch | download mbox 

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
+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
+    DEPENDS:=+jansson +zlib
+endef
+
+define Package/alfred-json/install
+	$(INSTALL_DIR) $(1)/bin/
+	$(CP) $(PKG_INSTALL_DIR)/usr/bin/alfred-json $(1)/bin/
+endef
+
+$(eval $(call BuildPackage,alfred-json))

Comments

Adrian Schmutzler Aug. 12, 2018, 4:42 p.m. 
Hallo,

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

Bitte Kommentare unten beachten und ggf. noch anpassen.

> -----Original Message-----
> From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf
> Of Tim Niemeyer
> Sent: Samstag, 11. August 2018 19:54
> To: franken-dev@freifunk.net
> Subject: [PATCH v3 1/2] packages/alfred-json: added package
> 
> From: Steffen Pankratz <kratz00@gmx.de>
> 
> Signed-off-by: Tim Niemeyer <tim@tn-x.org>
> - Changed subject
> - Move alfred-json to own repo
> Reviewed-by: Fabian Bläse <fabian@blaese.de>
> Tested-by: Fabian Bläse <fabian@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
Robert Langhammer Sept. 9, 2018, 11:57 p.m. 
Hallo,

Reviewed-by: Robert Langhammer <rlanghammer@web.de>

Tested-by: Robert Langhammer <rlanghammer@web.de>



Am 11.08.2018 um 19:53 schrieb Tim Niemeyer:
> From: Steffen Pankratz <kratz00@gmx.de>
>
> Signed-off-by: Tim Niemeyer <tim@tn-x.org>
> - Changed subject
> - Move alfred-json to own repo
> Reviewed-by: Fabian Bläse <fabian@blaese.de>
> Tested-by: Fabian Bläse <fabian@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
> +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
> +    DEPENDS:=+jansson +zlib
> +endef
> +
> +define Package/alfred-json/install
> +	$(INSTALL_DIR) $(1)/bin/
> +	$(CP) $(PKG_INSTALL_DIR)/usr/bin/alfred-json $(1)/bin/
> +endef
> +
> +$(eval $(call BuildPackage,alfred-json))
Fabian Blaese Dec. 26, 2018, 11:12 p.m. 
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@adrianschmutzler.de wrote:
> Hallo,
> 
> Reviewed-by: Adrian Schmutzler <freifunk@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
Adrian Schmutzler Jan. 21, 2019, 1:09 p.m. 
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@blaese.de] 
Sent: Donnerstag, 27. Dezember 2018 00:13
To: mail@adrianschmutzler.de; franken-dev@freifunk.net; 'Tim Niemeyer' <tim@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@adrianschmutzler.de <mailto:mail@adrianschmutzler.de>  wrote: 
> Hallo, 
> 
> Reviewed-by: Adrian Schmutzler <freifunk@adrianschmutzler.de <mailto:freifunk@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
Adrian Schmutzler Jan. 27, 2019, 2:41 p.m. 
Hallo,

 

applied ohne die unten genannte Änderung.

Die schicke ich auch nochmal extra.

 

Grüße

 

Adrian

 

From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf Of Adrian Schmutzler
Sent: Montag, 21. Januar 2019 14:10
To: 'Fabian Bläse' <fabian@blaese.de>; franken-dev@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@blaese.de] 
Sent: Donnerstag, 27. Dezember 2018 00:13
To: mail@adrianschmutzler.de <mailto:mail@adrianschmutzler.de> ; franken-dev@freifunk.net <mailto:franken-dev@freifunk.net> ; 'Tim Niemeyer' <tim@tn-x.org <mailto:tim@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@adrianschmutzler.de <mailto:mail@adrianschmutzler.de>  wrote: 
> Hallo, 
> 
> Reviewed-by: Adrian Schmutzler <freifunk@adrianschmutzler.de <mailto:freifunk@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