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

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

Details

Message ID 20180811171953.14822-1-tim@tn-x.org
State Superseded
Headers show

Commit Message

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

Signed-off-by: Tim Niemeyer <tim@tn-x.org>
- Changed subject
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

 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..94549ce
--- /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/kratz00/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. 11, 2018, 5:28 p.m. 
Hallo,

nur kurz vom Handy:


On August 11, 2018 7:19:52 PM GMT+02:00, Tim Niemeyer <tim@tn-x.org> wrote:
>From: Steffen Pankratz <kratz00@gmx.de>
>
>Signed-off-by: Tim Niemeyer <tim@tn-x.org>
>- Changed subject

Brauchen wir das? Schaut so unfertig aus.

>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
>
>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..94549ce
>--- /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/kratz00/alfred-json

Der Pfad bleibt alt?

Grüße

Adrian

>+    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))
Tim Niemeyer Aug. 11, 2018, 5:32 p.m. 
Am Samstag, den 11.08.2018, 19:28 +0200 schrieb Adrian Schmutzler:
> Hallo,
> 
> nur kurz vom Handy:
> 
> 
> On August 11, 2018 7:19:52 PM GMT+02:00, Tim Niemeyer <tim@tn-x.org>
> wrote:
> > From: Steffen Pankratz <kratz00@gmx.de>
> > 
> > Signed-off-by: Tim Niemeyer <tim@tn-x.org>
> > - Changed subject
> 
> Brauchen wir das? Schaut so unfertig aus.
Ich kann da nichts unfertiges erkennen. Eigentlich sollten hier sogar
die Änderungen von unten auftauchen, weil ja Kratz der Autor ist und
die Änderung nicht selbst gemacht hat.

> 
> > 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
> > 
> > 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..94549ce
> > --- /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/kratz00/alfred-json
> 
> Der Pfad bleibt alt?
Ops.

Tim

> 
> Grüße
> 
> Adrian
> 
> > +    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))
Adrian Schmutzler Aug. 11, 2018, 5:38 p.m. 
Hallo Tim,

ich finde es nicht so relevant, wer was geändert hat und würde da einfach nur die signed-offs hin machen.
Insbesondere da nicht erklärt wird, was der patch tut.
Aber das sind Kleinigkeiten...

Grüße

Adrian

On August 11, 2018 7:32:07 PM GMT+02:00, Tim Niemeyer <tim@tn-x.org> wrote:
>Am Samstag, den 11.08.2018, 19:28 +0200 schrieb Adrian Schmutzler:
>> Hallo,
>> 
>> nur kurz vom Handy:
>> 
>> 
>> On August 11, 2018 7:19:52 PM GMT+02:00, Tim Niemeyer <tim@tn-x.org>
>> wrote:
>> > From: Steffen Pankratz <kratz00@gmx.de>
>> > 
>> > Signed-off-by: Tim Niemeyer <tim@tn-x.org>
>> > - Changed subject
>> 
>> Brauchen wir das? Schaut so unfertig aus.
>Ich kann da nichts unfertiges erkennen. Eigentlich sollten hier sogar
>die Änderungen von unten auftauchen, weil ja Kratz der Autor ist und
>die Änderung nicht selbst gemacht hat.
>
>> 
>> > 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
>> > 
>> > 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..94549ce
>> > --- /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/kratz00/alfred-json
>> 
>> Der Pfad bleibt alt?
>Ops.
>
>Tim
>
>> 
>> Grüße
>> 
>> Adrian
>> 
>> > +    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))
Tim Niemeyer Aug. 11, 2018, 5:43 p.m. 
Am Samstag, den 11.08.2018, 19:38 +0200 schrieb Adrian Schmutzler:
> Hallo Tim,
> 
> ich finde es nicht so relevant, wer was geändert hat und würde da
> einfach nur die signed-offs hin machen.
> Insbesondere da nicht erklärt wird, was der patch tut.
Meinst du das jetzt erst oder ist das ein Spaß?

> Aber das sind Kleinigkeiten...
Das kann ich nicht entscheiden! Es ist nicht mein Patch! Der ist nunmal
von Kratz, also gehören die Änderungen dokumentiert.

Tim

> 
> Grüße
> 
> Adrian
> 
> On August 11, 2018 7:32:07 PM GMT+02:00, Tim Niemeyer <tim@tn-x.org>
> wrote:
> > Am Samstag, den 11.08.2018, 19:28 +0200 schrieb Adrian Schmutzler:
> > > Hallo,
> > > 
> > > nur kurz vom Handy:
> > > 
> > > 
> > > On August 11, 2018 7:19:52 PM GMT+02:00, Tim Niemeyer <tim@tn-x.o
> > > rg>
> > > wrote:
> > > > From: Steffen Pankratz <kratz00@gmx.de>
> > > > 
> > > > Signed-off-by: Tim Niemeyer <tim@tn-x.org>
> > > > - Changed subject
> > > 
> > > Brauchen wir das? Schaut so unfertig aus.
> > 
> > Ich kann da nichts unfertiges erkennen. Eigentlich sollten hier
> > sogar
> > die Änderungen von unten auftauchen, weil ja Kratz der Autor ist
> > und
> > die Änderung nicht selbst gemacht hat.
> > 
> > > 
> > > > 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
> > > > 
> > > > 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..94549ce
> > > > --- /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/kratz00/alfred-json
> > > 
> > > Der Pfad bleibt alt?
> > 
> > Ops.
> > 
> > Tim
> > 
> > > 
> > > Grüße
> > > 
> > > Adrian
> > > 
> > > > +    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))
Adrian Schmutzler Aug. 11, 2018, 5:45 p.m. 
Hallo nochmal,

On August 11, 2018 7:28:35 PM GMT+02:00, Adrian Schmutzler <mail@adrianschmutzler.de> wrote:
>Hallo,
>
>nur kurz vom Handy:
>
>
>On August 11, 2018 7:19:52 PM GMT+02:00, Tim Niemeyer <tim@tn-x.org>
>wrote:
>>From: Steffen Pankratz <kratz00@gmx.de>
>>
>>Signed-off-by: Tim Niemeyer <tim@tn-x.org>
>>- Changed subject
>
>Brauchen wir das? Schaut so unfertig aus.
>
>>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
>>
>>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..94549ce
>>--- /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/kratz00/alfred-json
>
>Der Pfad bleibt alt?

Brauchen wir die URL überhaupt doppelt?

Und wenn ja, ggf. die Variable PKGSOURCODINGS benutzen, dann steht der Pfad nur einmal explizit da.

Grüße

Adriat
>
>Grüße
>
>Adrian
>
>>+    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))
Tim Niemeyer Aug. 11, 2018, 5:48 p.m. 
Am Samstag, den 11.08.2018, 19:45 +0200 schrieb Adrian Schmutzler:
> Hallo nochmal,
> 
> On August 11, 2018 7:28:35 PM GMT+02:00, Adrian Schmutzler <mail@adri
> anschmutzler.de> wrote:
> > Hallo,
> > 
> > nur kurz vom Handy:
> > 
> > 
> > On August 11, 2018 7:19:52 PM GMT+02:00, Tim Niemeyer <tim@tn-x.org
> > >
> > wrote:
> > > From: Steffen Pankratz <kratz00@gmx.de>
> > > 
> > > Signed-off-by: Tim Niemeyer <tim@tn-x.org>
> > > - Changed subject
> > 
> > Brauchen wir das? Schaut so unfertig aus.
> > 
> > > 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
> > > 
> > > 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..94549ce
> > > --- /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.g
> > > it
> > > +
> > > +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/kratz00/alfred-json
> > 
> > Der Pfad bleibt alt?
> 
> Brauchen wir die URL überhaupt doppelt?
> 
> Und wenn ja, ggf. die Variable PKGSOURCODINGS benutzen, dann steht
> der Pfad nur einmal explizit da.
Ich habs jetzt schon angepasst. Wenn überhaupt macht es wohl eher Sinn
die URL zu entfernen. Aber ich will mich aber nicht mit dieser dämlich
URL beschäftigen, ich will mit der Firmware weiter kommen.

Tim

> 
> Grüße
> 
> Adriat
> > 
> > Grüße
> > 
> > Adrian
> > 
> > > +    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))