mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
* [ptxdist] [PATCH] yajl: add new package
@ 2018-06-21 12:56 Denis OSTERLAND
  2018-06-22  7:19 ` Michael Olbrich
  2018-06-22  8:12 ` [ptxdist] [PATCH v2] " Denis OSTERLAND
  0 siblings, 2 replies; 10+ messages in thread
From: Denis OSTERLAND @ 2018-06-21 12:56 UTC (permalink / raw)
  To: ptxdist

Signed-off-by: Denis Osterland <Denis.Osterland@diehl.com>
---
 rules/yajl.in   |  9 ++++++++
 rules/yajl.make | 58 +++++++++++++++++++++++++++++++++++++++++++++++++
 2 files changed, 67 insertions(+)
 create mode 100644 rules/yajl.in
 create mode 100644 rules/yajl.make

diff --git a/rules/yajl.in b/rules/yajl.in
new file mode 100644
index 000000000..24c216d9d
--- /dev/null
+++ b/rules/yajl.in
@@ -0,0 +1,9 @@
+## SECTION=system_libraries
+
+config YAJL
+	tristate
+	prompt "yajl"
+	default ALLYES
+	select HOST_CMAKE
+	help
+	  A fast streaming JSON parsing library in C.
diff --git a/rules/yajl.make b/rules/yajl.make
new file mode 100644
index 000000000..84a0fca88
--- /dev/null
+++ b/rules/yajl.make
@@ -0,0 +1,58 @@
+# -*-makefile-*-
+#
+# Copyright (C) 2016 by Denis Osterland <Denis.Osterland@diehl.com>
+#
+# See CREDITS for details about who has contributed to this project.
+#
+# For further information about the PTXdist project and license conditions
+# see the README file.
+#
+
+#
+# We provide this package
+#
+PACKAGES-$(PTXCONF_YAJL) += yajl
+
+#
+# Paths and names
+#
+YAJL_VERSION       := 2.1.0
+YAJL_MD5           := 6887e0ed7479d2549761a4d284d3ecb0
+YAJL               := yajl-$(YAJL_VERSION)
+YAJL_SUFFIX        := orig.tar.gz
+YAJL_URL           := $(call ptx/mirror, DEB, pool/main/y/yajl)/yajl_$(YAJL_VERSION).$(YAJL_SUFFIX)
+YAJL_SOURCE        := $(SRCDIR)/$(YAJL).$(YAJL_SUFFIX)
+YAJL_DIR           := $(BUILDDIR)/$(YAJL)
+YAJL_LICENSE       := ISC
+YAJL_LICENSE_FILES := file://COPYING;md5=39af6eb42999852bdd3ea00ad120a36d
+
+
+# ----------------------------------------------------------------------------
+# Prepare
+# ----------------------------------------------------------------------------
+
+#
+# autoconf
+#
+YAJL_CONF_TOOL	:= cmake
+
+# ----------------------------------------------------------------------------
+# Target-Install
+# ----------------------------------------------------------------------------
+
+$(STATEDIR)/yajl.targetinstall:
+	@$(call targetinfo)
+
+	@$(call install_init, yajl)
+	@$(call install_fixup, yajl,PRIORITY,optional)
+	@$(call install_fixup, yajl,SECTION,base)
+	@$(call install_fixup, yajl,AUTHOR,"Denis Osterland <Denis.Osterland@diehl.com>")
+	@$(call install_fixup, yajl,DESCRIPTION,missing)
+
+	@$(call install_lib, yajl, 0, 0, 0755, libyajl)
+
+	@$(call install_finish, yajl)
+
+	@$(call touch)
+
+# vim: syntax=make
-- 
2.17.1


Diehl AKO Stiftung & Co. KG, Pfannerstraße 75-83, 88239 Wangen im Allgäu
Bereichsvorstand: Dipl.-Ing. Michael Siedentop (Sprecher), Josef Fellner (Mitglied)
Sitz der Gesellschaft: Wangen i.A. – Registergericht: Amtsgericht Ulm HRA 620609 – Persönlich haftende Gesellschafterin: Diehl Verwaltungs-Stiftung – Sitz: Nürnberg – Registergericht: Amtsgericht Nürnberg HRA 11756 –
Vorstand: Dr.-Ing. Karl Tragl (Vorsitzender), Herr Dipl.-Wirtsch.-Ing. Wolfgang Weggen (stellvertretender Vorsitzender), Dipl.-Kfm. Claus Günther, Dipl.-Kfm. Frank Gutzeit, Dr.-Ing. Heinrich Schunk, Dr.-Ing. Michael Siedentop , Dipl.-Kfm. Dr.-Ing. Martin Sommer, Dipl.-Ing. (FH) Rainer von Borstel, Vorsitzender des Aufsichtsrates: Dr. Klaus Maier
___________________________________________________________________________________________________
Der Inhalt der vorstehenden E-Mail ist nicht rechtlich bindend. Diese E-Mail enthaelt vertrauliche und/oder rechtlich geschuetzte Informationen.
Informieren Sie uns bitte, wenn Sie diese E-Mail faelschlicherweise erhalten haben. Bitte loeschen Sie in diesem Fall die Nachricht. Jede unerlaubte Form der Reproduktion, Bekanntgabe, Aenderung, Verteilung und/oder Publikation dieser E-Mail ist strengstens untersagt.
The contents of the above mentioned e-mail is not legally binding. This e-mail contains confidential and/or legally protected information. Please inform us if you have received this e-mail by mistake and delete it in such a case. Each unauthorized reproduction, disclosure, alteration, distribution and/or publication of this e-mail is strictly prohibited.

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [ptxdist] [PATCH] yajl: add new package
  2018-06-21 12:56 [ptxdist] [PATCH] yajl: add new package Denis OSTERLAND
@ 2018-06-22  7:19 ` Michael Olbrich
  2018-06-22  7:37   ` Denis OSTERLAND
  2018-06-22  8:12 ` [ptxdist] [PATCH v2] " Denis OSTERLAND
  1 sibling, 1 reply; 10+ messages in thread
From: Michael Olbrich @ 2018-06-22  7:19 UTC (permalink / raw)
  To: ptxdist

On Thu, Jun 21, 2018 at 12:56:36PM +0000, Denis OSTERLAND wrote:
> Signed-off-by: Denis Osterland <Denis.Osterland@diehl.com>
> ---
>  rules/yajl.in   |  9 ++++++++
>  rules/yajl.make | 58 +++++++++++++++++++++++++++++++++++++++++++++++++
>  2 files changed, 67 insertions(+)
>  create mode 100644 rules/yajl.in
>  create mode 100644 rules/yajl.make
> 
> diff --git a/rules/yajl.in b/rules/yajl.in
> new file mode 100644
> index 000000000..24c216d9d
> --- /dev/null
> +++ b/rules/yajl.in
> @@ -0,0 +1,9 @@
> +## SECTION=system_libraries
> +
> +config YAJL
> +	tristate
> +	prompt "yajl"
> +	default ALLYES

This default is only used for noprompt packages.

Michael

> +	select HOST_CMAKE
> +	help
> +	  A fast streaming JSON parsing library in C.
> diff --git a/rules/yajl.make b/rules/yajl.make
> new file mode 100644
> index 000000000..84a0fca88
> --- /dev/null
> +++ b/rules/yajl.make
> @@ -0,0 +1,58 @@
> +# -*-makefile-*-
> +#
> +# Copyright (C) 2016 by Denis Osterland <Denis.Osterland@diehl.com>
> +#
> +# See CREDITS for details about who has contributed to this project.
> +#
> +# For further information about the PTXdist project and license conditions
> +# see the README file.
> +#
> +
> +#
> +# We provide this package
> +#
> +PACKAGES-$(PTXCONF_YAJL) += yajl
> +
> +#
> +# Paths and names
> +#
> +YAJL_VERSION       := 2.1.0
> +YAJL_MD5           := 6887e0ed7479d2549761a4d284d3ecb0
> +YAJL               := yajl-$(YAJL_VERSION)
> +YAJL_SUFFIX        := orig.tar.gz
> +YAJL_URL           := $(call ptx/mirror, DEB, pool/main/y/yajl)/yajl_$(YAJL_VERSION).$(YAJL_SUFFIX)

Use the upstream github URL:

http://github.com/lloyd/yajl/tarball/2.1.0

Michael

> +YAJL_SOURCE        := $(SRCDIR)/$(YAJL).$(YAJL_SUFFIX)
> +YAJL_DIR           := $(BUILDDIR)/$(YAJL)
> +YAJL_LICENSE       := ISC
> +YAJL_LICENSE_FILES := file://COPYING;md5=39af6eb42999852bdd3ea00ad120a36d
> +
> +
> +# ----------------------------------------------------------------------------
> +# Prepare
> +# ----------------------------------------------------------------------------
> +
> +#
> +# autoconf
> +#
> +YAJL_CONF_TOOL	:= cmake
> +
> +# ----------------------------------------------------------------------------
> +# Target-Install
> +# ----------------------------------------------------------------------------
> +
> +$(STATEDIR)/yajl.targetinstall:
> +	@$(call targetinfo)
> +
> +	@$(call install_init, yajl)
> +	@$(call install_fixup, yajl,PRIORITY,optional)
> +	@$(call install_fixup, yajl,SECTION,base)
> +	@$(call install_fixup, yajl,AUTHOR,"Denis Osterland <Denis.Osterland@diehl.com>")
> +	@$(call install_fixup, yajl,DESCRIPTION,missing)
> +
> +	@$(call install_lib, yajl, 0, 0, 0755, libyajl)
> +
> +	@$(call install_finish, yajl)
> +
> +	@$(call touch)
> +
> +# vim: syntax=make
> -- 
> 2.17.1
> 
> 
> Diehl AKO Stiftung & Co. KG, Pfannerstraße 75-83, 88239 Wangen im Allgäu
> Bereichsvorstand: Dipl.-Ing. Michael Siedentop (Sprecher), Josef Fellner (Mitglied)
> Sitz der Gesellschaft: Wangen i.A. – Registergericht: Amtsgericht Ulm HRA 620609 – Persönlich haftende Gesellschafterin: Diehl Verwaltungs-Stiftung – Sitz: Nürnberg – Registergericht: Amtsgericht Nürnberg HRA 11756 –
> Vorstand: Dr.-Ing. Karl Tragl (Vorsitzender), Herr Dipl.-Wirtsch.-Ing. Wolfgang Weggen (stellvertretender Vorsitzender), Dipl.-Kfm. Claus Günther, Dipl.-Kfm. Frank Gutzeit, Dr.-Ing. Heinrich Schunk, Dr.-Ing. Michael Siedentop , Dipl.-Kfm. Dr.-Ing. Martin Sommer, Dipl.-Ing. (FH) Rainer von Borstel, Vorsitzender des Aufsichtsrates: Dr. Klaus Maier
> ___________________________________________________________________________________________________
> Der Inhalt der vorstehenden E-Mail ist nicht rechtlich bindend. Diese E-Mail enthaelt vertrauliche und/oder rechtlich geschuetzte Informationen.
> Informieren Sie uns bitte, wenn Sie diese E-Mail faelschlicherweise erhalten haben. Bitte loeschen Sie in diesem Fall die Nachricht. Jede unerlaubte Form der Reproduktion, Bekanntgabe, Aenderung, Verteilung und/oder Publikation dieser E-Mail ist strengstens untersagt.
> The contents of the above mentioned e-mail is not legally binding. This e-mail contains confidential and/or legally protected information. Please inform us if you have received this e-mail by mistake and delete it in such a case. Each unauthorized reproduction, disclosure, alteration, distribution and/or publication of this e-mail is strictly prohibited.
> 
> _______________________________________________
> ptxdist mailing list
> ptxdist@pengutronix.de

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [ptxdist] [PATCH] yajl: add new package
  2018-06-22  7:19 ` Michael Olbrich
@ 2018-06-22  7:37   ` Denis OSTERLAND
  0 siblings, 0 replies; 10+ messages in thread
From: Denis OSTERLAND @ 2018-06-22  7:37 UTC (permalink / raw)
  To: ptxdist

Am Freitag, den 22.06.2018, 09:19 +0200 schrieb Michael Olbrich:
> > +	default ALLYES
> This default is only used for noprompt packages.
> 
> Michael
I will remove this in v2.
> 
> > +YAJL_SUFFIX        := orig.tar.gz
> > +YAJL_URL           := $(call ptx/mirror, DEB, pool/main/y/yajl)/yajl_$(YAJL_VERSION).$(YAJL_SUFFIX)
> Use the upstream github URL:
> 
> http://github.com/lloyd/yajl/tarball/2.1.0
> 
> Michael
I will change the URL in v2.

Regards Denis
> 
> > 
> > _______________________________________________
> > ptxdist mailing list
> > ptxdist@pengutronix.de
Diehl AKO Stiftung & Co. KG, Pfannerstraße 75-83, 88239 Wangen im Allgäu
Bereichsvorstand: Dipl.-Ing. Michael Siedentop (Sprecher), Josef Fellner (Mitglied)
Sitz der Gesellschaft: Wangen i.A. – Registergericht: Amtsgericht Ulm HRA 620609 – Persönlich haftende Gesellschafterin: Diehl Verwaltungs-Stiftung – Sitz: Nürnberg – Registergericht: Amtsgericht Nürnberg HRA 11756 –
Vorstand: Dr.-Ing. Karl Tragl (Vorsitzender), Herr Dipl.-Wirtsch.-Ing. Wolfgang Weggen (stellvertretender Vorsitzender), Dipl.-Kfm. Claus Günther, Dipl.-Kfm. Frank Gutzeit, Dr.-Ing. Heinrich Schunk, Dr.-Ing. Michael Siedentop , Dipl.-Kfm. Dr.-Ing. Martin Sommer, Dipl.-Ing. (FH) Rainer von Borstel, Vorsitzender des Aufsichtsrates: Dr. Klaus Maier
___________________________________________________________________________________________________
Der Inhalt der vorstehenden E-Mail ist nicht rechtlich bindend. Diese E-Mail enthaelt vertrauliche und/oder rechtlich geschuetzte Informationen.
Informieren Sie uns bitte, wenn Sie diese E-Mail faelschlicherweise erhalten haben. Bitte loeschen Sie in diesem Fall die Nachricht. Jede unerlaubte Form der Reproduktion, Bekanntgabe, Aenderung, Verteilung und/oder Publikation dieser E-Mail ist strengstens untersagt.
The contents of the above mentioned e-mail is not legally binding. This e-mail contains confidential and/or legally protected information. Please inform us if you have received this e-mail by mistake and delete it in such a case. Each unauthorized reproduction, disclosure, alteration, distribution and/or publication of this e-mail is strictly prohibited.
_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [ptxdist] [PATCH v2] yajl: add new package
  2018-06-21 12:56 [ptxdist] [PATCH] yajl: add new package Denis OSTERLAND
  2018-06-22  7:19 ` Michael Olbrich
@ 2018-06-22  8:12 ` Denis OSTERLAND
  2018-06-22  8:25   ` Alexander Dahl
  2018-06-22  9:42   ` [ptxdist] [PATCH v3] " Denis OSTERLAND
  1 sibling, 2 replies; 10+ messages in thread
From: Denis OSTERLAND @ 2018-06-22  8:12 UTC (permalink / raw)
  To: ptxdist

Signed-off-by: Denis Osterland <Denis.Osterland@diehl.com>
---
 rules/yajl.in   |  8 +++++++
 rules/yajl.make | 58 +++++++++++++++++++++++++++++++++++++++++++++++++
 2 files changed, 66 insertions(+)
 create mode 100644 rules/yajl.in
 create mode 100644 rules/yajl.make

diff --git a/rules/yajl.in b/rules/yajl.in
new file mode 100644
index 000000000..39c53e1df
--- /dev/null
+++ b/rules/yajl.in
@@ -0,0 +1,8 @@
+## SECTION=system_libraries
+
+config YAJL
+	tristate
+	prompt "yajl"
+	select HOST_CMAKE
+	help
+	  A fast streaming JSON parsing library in C.
diff --git a/rules/yajl.make b/rules/yajl.make
new file mode 100644
index 000000000..92bfe49ad
--- /dev/null
+++ b/rules/yajl.make
@@ -0,0 +1,58 @@
+# -*-makefile-*-
+#
+# Copyright (C) 2016 by Denis Osterland <Denis.Osterland@diehl.com>
+#
+# See CREDITS for details about who has contributed to this project.
+#
+# For further information about the PTXdist project and license conditions
+# see the README file.
+#
+
+#
+# We provide this package
+#
+PACKAGES-$(PTXCONF_YAJL) += yajl
+
+#
+# Paths and names
+#
+YAJL_VERSION       := 2.1.0
+YAJL_MD5           := 8df8a92a2799bc949577e8e7a9f43670
+YAJL               := yajl-$(YAJL_VERSION)
+YAJL_SUFFIX        := tar.gz
+YAJL_URL           := http://github.com/lloyd/yajl/tarball/$(YAJL_VERSION)
+YAJL_SOURCE        := $(SRCDIR)/$(YAJL).$(YAJL_SUFFIX)
+YAJL_DIR           := $(BUILDDIR)/$(YAJL)
+YAJL_LICENSE       := ISC
+YAJL_LICENSE_FILES := file://COPYING;md5=39af6eb42999852bdd3ea00ad120a36d
+
+
+# ----------------------------------------------------------------------------
+# Prepare
+# ----------------------------------------------------------------------------
+
+#
+# autoconf
+#
+YAJL_CONF_TOOL	:= cmake
+
+# ----------------------------------------------------------------------------
+# Target-Install
+# ----------------------------------------------------------------------------
+
+$(STATEDIR)/yajl.targetinstall:
+	@$(call targetinfo)
+
+	@$(call install_init, yajl)
+	@$(call install_fixup, yajl,PRIORITY,optional)
+	@$(call install_fixup, yajl,SECTION,base)
+	@$(call install_fixup, yajl,AUTHOR,"Denis Osterland <Denis.Osterland@diehl.com>")
+	@$(call install_fixup, yajl,DESCRIPTION,missing)
+
+	@$(call install_lib, yajl, 0, 0, 0755, libyajl)
+
+	@$(call install_finish, yajl)
+
+	@$(call touch)
+
+# vim: syntax=make
-- 
2.18.0


Diehl AKO Stiftung & Co. KG, Pfannerstraße 75-83, 88239 Wangen im Allgäu
Bereichsvorstand: Dipl.-Ing. Michael Siedentop (Sprecher), Josef Fellner (Mitglied)
Sitz der Gesellschaft: Wangen i.A. – Registergericht: Amtsgericht Ulm HRA 620609 – Persönlich haftende Gesellschafterin: Diehl Verwaltungs-Stiftung – Sitz: Nürnberg – Registergericht: Amtsgericht Nürnberg HRA 11756 –
Vorstand: Dr.-Ing. Karl Tragl (Vorsitzender), Herr Dipl.-Wirtsch.-Ing. Wolfgang Weggen (stellvertretender Vorsitzender), Dipl.-Kfm. Claus Günther, Dipl.-Kfm. Frank Gutzeit, Dr.-Ing. Heinrich Schunk, Dr.-Ing. Michael Siedentop , Dipl.-Kfm. Dr.-Ing. Martin Sommer, Dipl.-Ing. (FH) Rainer von Borstel, Vorsitzender des Aufsichtsrates: Dr. Klaus Maier
___________________________________________________________________________________________________
Der Inhalt der vorstehenden E-Mail ist nicht rechtlich bindend. Diese E-Mail enthaelt vertrauliche und/oder rechtlich geschuetzte Informationen.
Informieren Sie uns bitte, wenn Sie diese E-Mail faelschlicherweise erhalten haben. Bitte loeschen Sie in diesem Fall die Nachricht. Jede unerlaubte Form der Reproduktion, Bekanntgabe, Aenderung, Verteilung und/oder Publikation dieser E-Mail ist strengstens untersagt.
The contents of the above mentioned e-mail is not legally binding. This e-mail contains confidential and/or legally protected information. Please inform us if you have received this e-mail by mistake and delete it in such a case. Each unauthorized reproduction, disclosure, alteration, distribution and/or publication of this e-mail is strictly prohibited.

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [ptxdist] [PATCH v2] yajl: add new package
  2018-06-22  8:12 ` [ptxdist] [PATCH v2] " Denis OSTERLAND
@ 2018-06-22  8:25   ` Alexander Dahl
  2018-06-22  8:32     ` Denis OSTERLAND
  2018-06-22  9:42   ` [ptxdist] [PATCH v3] " Denis OSTERLAND
  1 sibling, 1 reply; 10+ messages in thread
From: Alexander Dahl @ 2018-06-22  8:25 UTC (permalink / raw)
  To: ptxdist; +Cc: Denis OSTERLAND

Hei hei,

Am Freitag, 22. Juni 2018, 08:12:10 CEST schrieb Denis OSTERLAND:
> +#
> +# autoconf
> +#
> +YAJL_CONF_TOOL	:= cmake

Comment does not match assignment. ;-)

Greets
Alex


_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [ptxdist] [PATCH v2] yajl: add new package
  2018-06-22  8:25   ` Alexander Dahl
@ 2018-06-22  8:32     ` Denis OSTERLAND
  2018-06-22  8:52       ` Alexander Dahl
  0 siblings, 1 reply; 10+ messages in thread
From: Denis OSTERLAND @ 2018-06-22  8:32 UTC (permalink / raw)
  To: ptxdist, ada

Am Freitag, den 22.06.2018, 10:25 +0200 schrieb Alexander Dahl:
> Hei hei,
> 
> Am Freitag, 22. Juni 2018, 08:12:10 CEST schrieb Denis OSTERLAND:
> > 
> > +#
> > +# autoconf
> > +#
> > +YAJL_CONF_TOOL	:= cmake
> Comment does not match assignment. ;-)
You are guess configtool as comment would be better.
Shall we patch it in the template?
> 
> Greets
> Alex
> 
Diehl AKO Stiftung & Co. KG, Pfannerstraße 75-83, 88239 Wangen im Allgäu
Bereichsvorstand: Dipl.-Ing. Michael Siedentop (Sprecher), Josef Fellner (Mitglied)
Sitz der Gesellschaft: Wangen i.A. – Registergericht: Amtsgericht Ulm HRA 620609 – Persönlich haftende Gesellschafterin: Diehl Verwaltungs-Stiftung – Sitz: Nürnberg – Registergericht: Amtsgericht Nürnberg HRA 11756 –
Vorstand: Dr.-Ing. Karl Tragl (Vorsitzender), Herr Dipl.-Wirtsch.-Ing. Wolfgang Weggen (stellvertretender Vorsitzender), Dipl.-Kfm. Claus Günther, Dipl.-Kfm. Frank Gutzeit, Dr.-Ing. Heinrich Schunk, Dr.-Ing. Michael Siedentop , Dipl.-Kfm. Dr.-Ing. Martin Sommer, Dipl.-Ing. (FH) Rainer von Borstel, Vorsitzender des Aufsichtsrates: Dr. Klaus Maier
___________________________________________________________________________________________________
Der Inhalt der vorstehenden E-Mail ist nicht rechtlich bindend. Diese E-Mail enthaelt vertrauliche und/oder rechtlich geschuetzte Informationen.
Informieren Sie uns bitte, wenn Sie diese E-Mail faelschlicherweise erhalten haben. Bitte loeschen Sie in diesem Fall die Nachricht. Jede unerlaubte Form der Reproduktion, Bekanntgabe, Aenderung, Verteilung und/oder Publikation dieser E-Mail ist strengstens untersagt.
The contents of the above mentioned e-mail is not legally binding. This e-mail contains confidential and/or legally protected information. Please inform us if you have received this e-mail by mistake and delete it in such a case. Each unauthorized reproduction, disclosure, alteration, distribution and/or publication of this e-mail is strictly prohibited.
_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [ptxdist] [PATCH v2] yajl: add new package
  2018-06-22  8:32     ` Denis OSTERLAND
@ 2018-06-22  8:52       ` Alexander Dahl
  2018-06-22  9:07         ` Denis OSTERLAND
  0 siblings, 1 reply; 10+ messages in thread
From: Alexander Dahl @ 2018-06-22  8:52 UTC (permalink / raw)
  To: ptxdist; +Cc: Denis OSTERLAND

Hello Denis,

> > > +#
> > > +# autoconf
> > > +#
> > > +YAJL_CONF_TOOL	:= cmake
> > 
> > Comment does not match assignment. ;-)
> 
> You are guess configtool as comment would be better.

From the 'src-cmake-prog' template you would get:

#                                                                                                                                                                                    
# cmake                                                                                                                                                                              
#                                                                                                                                                                                    
FOO_CONF_TOOL     := cmake                                                                                                                                                     

> Shall we patch it in the template?

I guess you used the 'target' template, which got you this:

#                                                                                                                                                                                    
# autoconf                                                                                                                                                                           
#                                                                                                                                                                                    
FOO_CONF_TOOL     := autoconf                                                                                                                                                  

I would not touch this in the template, it is consistent.

So I would say, just change the comment to 'cmake' or remove those comment 
lines. The line assigning the conf tool speaks for itself.

Greets
Alex


_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [ptxdist] [PATCH v2] yajl: add new package
  2018-06-22  8:52       ` Alexander Dahl
@ 2018-06-22  9:07         ` Denis OSTERLAND
  2018-06-22  9:15           ` Alexander Dahl
  0 siblings, 1 reply; 10+ messages in thread
From: Denis OSTERLAND @ 2018-06-22  9:07 UTC (permalink / raw)
  To: ptxdist, ada

Am Freitag, den 22.06.2018, 10:52 +0200 schrieb Alexander Dahl:
> Hello Denis,
> 
> > 
> > > 
> > > > 
> > > > +#
> > > > +# autoconf
> > > > +#
> > > > +YAJL_CONF_TOOL	:= cmake
> > > Comment does not match assignment. ;-)
> > You are guess configtool as comment would be better.
> From the 'src-cmake-prog' template you would get:
> 
> #                                                                                                                                                                                    
> # cmake                                                                                                                                                                              
> #                                                                                                                                                                                    
> FOO_CONF_TOOL     := cmake                                                                                                                                                     
> 
> > 
> > Shall we patch it in the template?
> I guess you used the 'target' template, which got you this:
> 
> #                                                                                                                                                                                    
> # autoconf                                                                                                                                                                           
> #                                                                                                                                                                                    
> FOO_CONF_TOOL     := autoconf                                                                                                                                                  
> 
> I would not touch this in the template, it is consistent.
> 
> So I would say, just change the comment to 'cmake' or remove those comment 
> lines. The line assigning the conf tool speaks for itself.
> 
> Greets
> Alex
> 
So I just used the wrong template.
Thanks for that hint.
I will remove the useless comment.

Regards Denis
Diehl AKO Stiftung & Co. KG, Pfannerstraße 75-83, 88239 Wangen im Allgäu
Bereichsvorstand: Dipl.-Ing. Michael Siedentop (Sprecher), Josef Fellner (Mitglied)
Sitz der Gesellschaft: Wangen i.A. – Registergericht: Amtsgericht Ulm HRA 620609 – Persönlich haftende Gesellschafterin: Diehl Verwaltungs-Stiftung – Sitz: Nürnberg – Registergericht: Amtsgericht Nürnberg HRA 11756 –
Vorstand: Dr.-Ing. Karl Tragl (Vorsitzender), Herr Dipl.-Wirtsch.-Ing. Wolfgang Weggen (stellvertretender Vorsitzender), Dipl.-Kfm. Claus Günther, Dipl.-Kfm. Frank Gutzeit, Dr.-Ing. Heinrich Schunk, Dr.-Ing. Michael Siedentop , Dipl.-Kfm. Dr.-Ing. Martin Sommer, Dipl.-Ing. (FH) Rainer von Borstel, Vorsitzender des Aufsichtsrates: Dr. Klaus Maier
___________________________________________________________________________________________________
Der Inhalt der vorstehenden E-Mail ist nicht rechtlich bindend. Diese E-Mail enthaelt vertrauliche und/oder rechtlich geschuetzte Informationen.
Informieren Sie uns bitte, wenn Sie diese E-Mail faelschlicherweise erhalten haben. Bitte loeschen Sie in diesem Fall die Nachricht. Jede unerlaubte Form der Reproduktion, Bekanntgabe, Aenderung, Verteilung und/oder Publikation dieser E-Mail ist strengstens untersagt.
The contents of the above mentioned e-mail is not legally binding. This e-mail contains confidential and/or legally protected information. Please inform us if you have received this e-mail by mistake and delete it in such a case. Each unauthorized reproduction, disclosure, alteration, distribution and/or publication of this e-mail is strictly prohibited.
_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [ptxdist] [PATCH v2] yajl: add new package
  2018-06-22  9:07         ` Denis OSTERLAND
@ 2018-06-22  9:15           ` Alexander Dahl
  0 siblings, 0 replies; 10+ messages in thread
From: Alexander Dahl @ 2018-06-22  9:15 UTC (permalink / raw)
  To: ptxdist; +Cc: Denis OSTERLAND

Hello Denis,

Am Freitag, 22. Juni 2018, 09:07:58 CEST schrieb Denis OSTERLAND:
> So I just used the wrong template.

I would not say so. You wanted a target package with external sources and you 
picked the right template for that. I guess it just makes no sense to have 
more target templates for all those different build systems used out there?

The 'src-cmake-prog' package would generate some source files including a 
basic CMakeLists.txt below your folder 'local_src'. In my opinion it makes 
sense to have different src templates, because those add some more than a 
single line to select the conf tool. But that template had a matching comment, 
I didn't mean to confuse anyone. ;-)

Maybe it gets more clear now? Sorry to bother everyone with those minor 
things. :-/

Greets
Alex


_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [ptxdist] [PATCH v3] yajl: add new package
  2018-06-22  8:12 ` [ptxdist] [PATCH v2] " Denis OSTERLAND
  2018-06-22  8:25   ` Alexander Dahl
@ 2018-06-22  9:42   ` Denis OSTERLAND
  1 sibling, 0 replies; 10+ messages in thread
From: Denis OSTERLAND @ 2018-06-22  9:42 UTC (permalink / raw)
  To: ptxdist

Signed-off-by: Denis Osterland <Denis.Osterland@diehl.com>
---
 rules/yajl.in   |  8 ++++++++
 rules/yajl.make | 49 +++++++++++++++++++++++++++++++++++++++++++++++++
 2 files changed, 57 insertions(+)
 create mode 100644 rules/yajl.in
 create mode 100644 rules/yajl.make

diff --git a/rules/yajl.in b/rules/yajl.in
new file mode 100644
index 000000000..39c53e1df
--- /dev/null
+++ b/rules/yajl.in
@@ -0,0 +1,8 @@
+## SECTION=system_libraries
+
+config YAJL
+	tristate
+	prompt "yajl"
+	select HOST_CMAKE
+	help
+	  A fast streaming JSON parsing library in C.
diff --git a/rules/yajl.make b/rules/yajl.make
new file mode 100644
index 000000000..3113a8cd0
--- /dev/null
+++ b/rules/yajl.make
@@ -0,0 +1,49 @@
+# -*-makefile-*-
+#
+# Copyright (C) 2016 by Denis Osterland <Denis.Osterland@diehl.com>
+#
+# See CREDITS for details about who has contributed to this project.
+#
+# For further information about the PTXdist project and license conditions
+# see the README file.
+#
+
+PACKAGES-$(PTXCONF_YAJL) += yajl
+
+YAJL_VERSION       := 2.1.0
+YAJL_MD5           := 8df8a92a2799bc949577e8e7a9f43670
+YAJL               := yajl-$(YAJL_VERSION)
+YAJL_SUFFIX        := tar.gz
+YAJL_URL           := http://github.com/lloyd/yajl/tarball/$(YAJL_VERSION)
+YAJL_SOURCE        := $(SRCDIR)/$(YAJL).$(YAJL_SUFFIX)
+YAJL_DIR           := $(BUILDDIR)/$(YAJL)
+YAJL_LICENSE       := ISC
+YAJL_LICENSE_FILES := file://COPYING;md5=39af6eb42999852bdd3ea00ad120a36d
+
+
+# ----------------------------------------------------------------------------
+# Prepare
+# ----------------------------------------------------------------------------
+
+YAJL_CONF_TOOL	:= cmake
+
+# ----------------------------------------------------------------------------
+# Target-Install
+# ----------------------------------------------------------------------------
+
+$(STATEDIR)/yajl.targetinstall:
+	@$(call targetinfo)
+
+	@$(call install_init, yajl)
+	@$(call install_fixup, yajl,PRIORITY,optional)
+	@$(call install_fixup, yajl,SECTION,base)
+	@$(call install_fixup, yajl,AUTHOR,"Denis Osterland <Denis.Osterland@diehl.com>")
+	@$(call install_fixup, yajl,DESCRIPTION,missing)
+
+	@$(call install_lib, yajl, 0, 0, 0755, libyajl)
+
+	@$(call install_finish, yajl)
+
+	@$(call touch)
+
+# vim: syntax=make
-- 
2.18.0


Diehl AKO Stiftung & Co. KG, Pfannerstraße 75-83, 88239 Wangen im Allgäu
Bereichsvorstand: Dipl.-Ing. Michael Siedentop (Sprecher), Josef Fellner (Mitglied)
Sitz der Gesellschaft: Wangen i.A. – Registergericht: Amtsgericht Ulm HRA 620609 – Persönlich haftende Gesellschafterin: Diehl Verwaltungs-Stiftung – Sitz: Nürnberg – Registergericht: Amtsgericht Nürnberg HRA 11756 –
Vorstand: Dr.-Ing. Karl Tragl (Vorsitzender), Herr Dipl.-Wirtsch.-Ing. Wolfgang Weggen (stellvertretender Vorsitzender), Dipl.-Kfm. Claus Günther, Dipl.-Kfm. Frank Gutzeit, Dr.-Ing. Heinrich Schunk, Dr.-Ing. Michael Siedentop , Dipl.-Kfm. Dr.-Ing. Martin Sommer, Dipl.-Ing. (FH) Rainer von Borstel, Vorsitzender des Aufsichtsrates: Dr. Klaus Maier
___________________________________________________________________________________________________
Der Inhalt der vorstehenden E-Mail ist nicht rechtlich bindend. Diese E-Mail enthaelt vertrauliche und/oder rechtlich geschuetzte Informationen.
Informieren Sie uns bitte, wenn Sie diese E-Mail faelschlicherweise erhalten haben. Bitte loeschen Sie in diesem Fall die Nachricht. Jede unerlaubte Form der Reproduktion, Bekanntgabe, Aenderung, Verteilung und/oder Publikation dieser E-Mail ist strengstens untersagt.
The contents of the above mentioned e-mail is not legally binding. This e-mail contains confidential and/or legally protected information. Please inform us if you have received this e-mail by mistake and delete it in such a case. Each unauthorized reproduction, disclosure, alteration, distribution and/or publication of this e-mail is strictly prohibited.

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2018-06-22  9:42 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-06-21 12:56 [ptxdist] [PATCH] yajl: add new package Denis OSTERLAND
2018-06-22  7:19 ` Michael Olbrich
2018-06-22  7:37   ` Denis OSTERLAND
2018-06-22  8:12 ` [ptxdist] [PATCH v2] " Denis OSTERLAND
2018-06-22  8:25   ` Alexander Dahl
2018-06-22  8:32     ` Denis OSTERLAND
2018-06-22  8:52       ` Alexander Dahl
2018-06-22  9:07         ` Denis OSTERLAND
2018-06-22  9:15           ` Alexander Dahl
2018-06-22  9:42   ` [ptxdist] [PATCH v3] " Denis OSTERLAND

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox