mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
* [ptxdist] [PATCH v2 0/2] json-c: version bump 0.13.1 -> 0.14
@ 2020-05-05  8:28 Alexander Dahl
  2020-05-05  8:28 ` [ptxdist] [PATCH v2 1/2] json-c: Revert URL to upstream provided tarballs Alexander Dahl
  2020-05-05  8:28 ` [ptxdist] [PATCH v2 2/2] json-c: version bump 0.13.1 -> 0.14 Alexander Dahl
  0 siblings, 2 replies; 5+ messages in thread
From: Alexander Dahl @ 2020-05-05  8:28 UTC (permalink / raw)
  To: ptxdist

Hei hei,

second iteration of this small series, basically adding dependencies to
host-cmake and also covering host-json-c.

Curious question: The only package depending on host-json-c is
host-cryptsetup, but no other package depends on host-cryptsetup … why?

Greets
Alex

Alexander Dahl (2):
  json-c: Revert URL to upstream provided tarballs
  json-c: version bump 0.13.1 -> 0.14

 rules/host-json-c.in   |  1 +
 rules/host-json-c.make | 17 +++++++++--------
 rules/json-c.in        |  1 +
 rules/json-c.make      | 23 ++++++++++++-----------
 4 files changed, 23 insertions(+), 19 deletions(-)

-- 
2.20.1


_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

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

* [ptxdist] [PATCH v2 1/2] json-c: Revert URL to upstream provided tarballs
  2020-05-05  8:28 [ptxdist] [PATCH v2 0/2] json-c: version bump 0.13.1 -> 0.14 Alexander Dahl
@ 2020-05-05  8:28 ` Alexander Dahl
  2020-05-11  5:42   ` [ptxdist] [APPLIED] " Michael Olbrich
  2020-05-05  8:28 ` [ptxdist] [PATCH v2 2/2] json-c: version bump 0.13.1 -> 0.14 Alexander Dahl
  1 sibling, 1 reply; 5+ messages in thread
From: Alexander Dahl @ 2020-05-05  8:28 UTC (permalink / raw)
  To: ptxdist

With 815c24d3b783 ("json-c: version bump 0.12 -> 0.12.1") URL was
changed, because back then it seemed upstream moved the project to
GitHub and would not provide tarballs anymore. They did again later,
however.

There are checksums of old releases at
https://github.com/json-c/json-c/wiki and all release tarballs are
available from the old/new URL. This avoids the archives built by GitHub
from source.

Signed-off-by: Alexander Dahl <ada@thorsis.com>
---
 rules/json-c.make | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/rules/json-c.make b/rules/json-c.make
index d4b4fc468..f6df25dba 100644
--- a/rules/json-c.make
+++ b/rules/json-c.make
@@ -14,11 +14,11 @@ PACKAGES-$(PTXCONF_JSON_C) += json-c
 #
 # Paths and names
 #
-JSON_C_VERSION		:= 0.13.1-20180305
-JSON_C_MD5		:= 20dba7bf773599a0842745a2fe5b7cd3
+JSON_C_VERSION		:= 0.13.1
+JSON_C_MD5		:= 04969ad59cc37bddd83741a08b98f350
 JSON_C			:= json-c-$(JSON_C_VERSION)
 JSON_C_SUFFIX		:= tar.gz
-JSON_C_URL		:= https://github.com/json-c/json-c/archive/$(JSON_C).$(JSON_C_SUFFIX)
+JSON_C_URL		:= https://s3.amazonaws.com/json-c_releases/releases/$(JSON_C).$(JSON_C_SUFFIX)
 JSON_C_SOURCE		:= $(SRCDIR)/$(JSON_C).$(JSON_C_SUFFIX)
 JSON_C_DIR		:= $(BUILDDIR)/$(JSON_C)
 JSON_C_LICENSE		:= MIT
-- 
2.20.1


_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

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

* [ptxdist] [PATCH v2 2/2] json-c: version bump 0.13.1 -> 0.14
  2020-05-05  8:28 [ptxdist] [PATCH v2 0/2] json-c: version bump 0.13.1 -> 0.14 Alexander Dahl
  2020-05-05  8:28 ` [ptxdist] [PATCH v2 1/2] json-c: Revert URL to upstream provided tarballs Alexander Dahl
@ 2020-05-05  8:28 ` Alexander Dahl
  2020-05-11  5:42   ` [ptxdist] [APPLIED] " Michael Olbrich
  1 sibling, 1 reply; 5+ messages in thread
From: Alexander Dahl @ 2020-05-05  8:28 UTC (permalink / raw)
  To: ptxdist

Quoting from the file 'ChangeLog':

> Build machinery has been switched to CMake.

The announcement is not clear about that however:

https://groups.google.com/forum/#!topic/json-c/yLwwAqiRf-4

Options are pinned like they were for the old autotools build (shared
lib, no threads, no rdrand) and what seemed to make sense in this build
environment.

Signed-off-by: Alexander Dahl <ada@thorsis.com>
---

Notes:
    v1 -> v2
    --------
    - select host_cmake
    - also update host-json-c

 rules/host-json-c.in   |  1 +
 rules/host-json-c.make | 17 +++++++++--------
 rules/json-c.in        |  1 +
 rules/json-c.make      | 21 +++++++++++----------
 4 files changed, 22 insertions(+), 18 deletions(-)

diff --git a/rules/host-json-c.in b/rules/host-json-c.in
index 569e21030..7354ae2d3 100644
--- a/rules/host-json-c.in
+++ b/rules/host-json-c.in
@@ -2,4 +2,5 @@
 
 config HOST_JSON_C
 	tristate
+	select HOST_CMAKE
 	default y if ALLYES
diff --git a/rules/host-json-c.make b/rules/host-json-c.make
index f96a27fa1..8e607f783 100644
--- a/rules/host-json-c.make
+++ b/rules/host-json-c.make
@@ -15,14 +15,15 @@ HOST_PACKAGES-$(PTXCONF_HOST_JSON_C) += host-json-c
 # Prepare
 # ----------------------------------------------------------------------------
 
-#
-# autoconf
-#
-HOST_JSON_C_CONF_TOOL	:= autoconf
+HOST_JSON_C_CONF_TOOL	:= cmake
 HOST_JSON_C_CONF_OPT	:= \
-	$(HOST_AUTOCONF) \
-	--disable-threading \
-	--disable-rdrand \
-	--disable-static
+	$(HOST_CMAKE_OPT) \
+	-DBUILD_DOCUMENTATION:BOOL=OFF \
+	-DBUILD_SHARED_LIBS:BOOL=ON \
+	-DBUILD_TESTING:BOOL=OFF \
+	-DDISABLE_BSYMBOLIC:BOOL=ON \
+	-DDISABLE_WERROR:BOOL=ON \
+	-DENABLE_RDRAND:BOOL=OFF \
+	-DENABLE_THREADING:BOOL=OFF
 
 # vim: syntax=make
diff --git a/rules/json-c.in b/rules/json-c.in
index ca4f1d500..a12fac01a 100644
--- a/rules/json-c.in
+++ b/rules/json-c.in
@@ -3,5 +3,6 @@
 config JSON_C
 	tristate
 	prompt "json-c"
+	select HOST_CMAKE
 	help
 	  json bindings in C
diff --git a/rules/json-c.make b/rules/json-c.make
index f6df25dba..f2a990bf4 100644
--- a/rules/json-c.make
+++ b/rules/json-c.make
@@ -14,8 +14,8 @@ PACKAGES-$(PTXCONF_JSON_C) += json-c
 #
 # Paths and names
 #
-JSON_C_VERSION		:= 0.13.1
-JSON_C_MD5		:= 04969ad59cc37bddd83741a08b98f350
+JSON_C_VERSION		:= 0.14
+JSON_C_MD5		:= 72cbb065b43376d825cd521d115ae1f6
 JSON_C			:= json-c-$(JSON_C_VERSION)
 JSON_C_SUFFIX		:= tar.gz
 JSON_C_URL		:= https://s3.amazonaws.com/json-c_releases/releases/$(JSON_C).$(JSON_C_SUFFIX)
@@ -28,15 +28,16 @@ JSON_C_LICENSE_FILES	:= file://COPYING;md5=de54b60fbbc35123ba193fea8ee216f2
 # Prepare
 # ----------------------------------------------------------------------------
 
-#
-# autoconf
-#
-JSON_C_CONF_TOOL	:= autoconf
+JSON_C_CONF_TOOL	:= cmake
 JSON_C_CONF_OPT		:= \
-	$(CROSS_AUTOCONF_USR) \
-	--disable-threading \
-	--disable-rdrand \
-	--disable-static
+	$(CROSS_CMAKE_USR) \
+	-DBUILD_DOCUMENTATION:BOOL=OFF \
+	-DBUILD_SHARED_LIBS:BOOL=ON \
+	-DBUILD_TESTING:BOOL=OFF \
+	-DDISABLE_BSYMBOLIC:BOOL=ON \
+	-DDISABLE_WERROR:BOOL=ON \
+	-DENABLE_RDRAND:BOOL=OFF \
+	-DENABLE_THREADING:BOOL=OFF
 
 # ----------------------------------------------------------------------------
 # Install
-- 
2.20.1


_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

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

* Re: [ptxdist] [APPLIED] json-c: Revert URL to upstream provided tarballs
  2020-05-05  8:28 ` [ptxdist] [PATCH v2 1/2] json-c: Revert URL to upstream provided tarballs Alexander Dahl
@ 2020-05-11  5:42   ` Michael Olbrich
  0 siblings, 0 replies; 5+ messages in thread
From: Michael Olbrich @ 2020-05-11  5:42 UTC (permalink / raw)
  To: ptxdist; +Cc: Alexander Dahl

Thanks, applied as 9633ce1284fdc6f87dfc3ee2e1393780b4d86070.

Michael

[sent from post-receive hook]

On Mon, 11 May 2020 07:42:44 +0200, Alexander Dahl <ada@thorsis.com> wrote:
> With 815c24d3b783 ("json-c: version bump 0.12 -> 0.12.1") URL was
> changed, because back then it seemed upstream moved the project to
> GitHub and would not provide tarballs anymore. They did again later,
> however.
> 
> There are checksums of old releases at
> https://github.com/json-c/json-c/wiki and all release tarballs are
> available from the old/new URL. This avoids the archives built by GitHub
> from source.
> 
> Signed-off-by: Alexander Dahl <ada@thorsis.com>
> Message-Id: <20200505082812.22335-2-ada@thorsis.com>
> Signed-off-by: Michael Olbrich <m.olbrich@pengutronix.de>
> 
> diff --git a/rules/json-c.make b/rules/json-c.make
> index d4b4fc468b47..f6df25dba80d 100644
> --- a/rules/json-c.make
> +++ b/rules/json-c.make
> @@ -14,11 +14,11 @@ PACKAGES-$(PTXCONF_JSON_C) += json-c
>  #
>  # Paths and names
>  #
> -JSON_C_VERSION		:= 0.13.1-20180305
> -JSON_C_MD5		:= 20dba7bf773599a0842745a2fe5b7cd3
> +JSON_C_VERSION		:= 0.13.1
> +JSON_C_MD5		:= 04969ad59cc37bddd83741a08b98f350
>  JSON_C			:= json-c-$(JSON_C_VERSION)
>  JSON_C_SUFFIX		:= tar.gz
> -JSON_C_URL		:= https://github.com/json-c/json-c/archive/$(JSON_C).$(JSON_C_SUFFIX)
> +JSON_C_URL		:= https://s3.amazonaws.com/json-c_releases/releases/$(JSON_C).$(JSON_C_SUFFIX)
>  JSON_C_SOURCE		:= $(SRCDIR)/$(JSON_C).$(JSON_C_SUFFIX)
>  JSON_C_DIR		:= $(BUILDDIR)/$(JSON_C)
>  JSON_C_LICENSE		:= MIT

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

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

* Re: [ptxdist] [APPLIED] json-c: version bump 0.13.1 -> 0.14
  2020-05-05  8:28 ` [ptxdist] [PATCH v2 2/2] json-c: version bump 0.13.1 -> 0.14 Alexander Dahl
@ 2020-05-11  5:42   ` Michael Olbrich
  0 siblings, 0 replies; 5+ messages in thread
From: Michael Olbrich @ 2020-05-11  5:42 UTC (permalink / raw)
  To: ptxdist; +Cc: Alexander Dahl

Thanks, applied as 879f2c25a5a48791905a7b54642bff409a3d780b.

Michael

[sent from post-receive hook]

On Mon, 11 May 2020 07:42:44 +0200, Alexander Dahl <ada@thorsis.com> wrote:
> Quoting from the file 'ChangeLog':
> 
> > Build machinery has been switched to CMake.
> 
> The announcement is not clear about that however:
> 
> https://groups.google.com/forum/#!topic/json-c/yLwwAqiRf-4
> 
> Options are pinned like they were for the old autotools build (shared
> lib, no threads, no rdrand) and what seemed to make sense in this build
> environment.
> 
> Signed-off-by: Alexander Dahl <ada@thorsis.com>
> Message-Id: <20200505082812.22335-3-ada@thorsis.com>
> Signed-off-by: Michael Olbrich <m.olbrich@pengutronix.de>
> 
> diff --git a/rules/host-json-c.in b/rules/host-json-c.in
> index 569e210305b6..7354ae2d3aa9 100644
> --- a/rules/host-json-c.in
> +++ b/rules/host-json-c.in
> @@ -2,4 +2,5 @@
>  
>  config HOST_JSON_C
>  	tristate
> +	select HOST_CMAKE
>  	default y if ALLYES
> diff --git a/rules/host-json-c.make b/rules/host-json-c.make
> index f96a27fa1c13..8e607f78346d 100644
> --- a/rules/host-json-c.make
> +++ b/rules/host-json-c.make
> @@ -15,14 +15,15 @@ HOST_PACKAGES-$(PTXCONF_HOST_JSON_C) += host-json-c
>  # Prepare
>  # ----------------------------------------------------------------------------
>  
> -#
> -# autoconf
> -#
> -HOST_JSON_C_CONF_TOOL	:= autoconf
> +HOST_JSON_C_CONF_TOOL	:= cmake
>  HOST_JSON_C_CONF_OPT	:= \
> -	$(HOST_AUTOCONF) \
> -	--disable-threading \
> -	--disable-rdrand \
> -	--disable-static
> +	$(HOST_CMAKE_OPT) \
> +	-DBUILD_DOCUMENTATION:BOOL=OFF \
> +	-DBUILD_SHARED_LIBS:BOOL=ON \
> +	-DBUILD_TESTING:BOOL=OFF \
> +	-DDISABLE_BSYMBOLIC:BOOL=ON \
> +	-DDISABLE_WERROR:BOOL=ON \
> +	-DENABLE_RDRAND:BOOL=OFF \
> +	-DENABLE_THREADING:BOOL=OFF
>  
>  # vim: syntax=make
> diff --git a/rules/json-c.in b/rules/json-c.in
> index ca4f1d5004b3..a12fac01adb4 100644
> --- a/rules/json-c.in
> +++ b/rules/json-c.in
> @@ -3,5 +3,6 @@
>  config JSON_C
>  	tristate
>  	prompt "json-c"
> +	select HOST_CMAKE
>  	help
>  	  json bindings in C
> diff --git a/rules/json-c.make b/rules/json-c.make
> index f6df25dba80d..f2a990bf4aa9 100644
> --- a/rules/json-c.make
> +++ b/rules/json-c.make
> @@ -14,8 +14,8 @@ PACKAGES-$(PTXCONF_JSON_C) += json-c
>  #
>  # Paths and names
>  #
> -JSON_C_VERSION		:= 0.13.1
> -JSON_C_MD5		:= 04969ad59cc37bddd83741a08b98f350
> +JSON_C_VERSION		:= 0.14
> +JSON_C_MD5		:= 72cbb065b43376d825cd521d115ae1f6
>  JSON_C			:= json-c-$(JSON_C_VERSION)
>  JSON_C_SUFFIX		:= tar.gz
>  JSON_C_URL		:= https://s3.amazonaws.com/json-c_releases/releases/$(JSON_C).$(JSON_C_SUFFIX)
> @@ -28,15 +28,16 @@ JSON_C_LICENSE_FILES	:= file://COPYING;md5=de54b60fbbc35123ba193fea8ee216f2
>  # Prepare
>  # ----------------------------------------------------------------------------
>  
> -#
> -# autoconf
> -#
> -JSON_C_CONF_TOOL	:= autoconf
> +JSON_C_CONF_TOOL	:= cmake
>  JSON_C_CONF_OPT		:= \
> -	$(CROSS_AUTOCONF_USR) \
> -	--disable-threading \
> -	--disable-rdrand \
> -	--disable-static
> +	$(CROSS_CMAKE_USR) \
> +	-DBUILD_DOCUMENTATION:BOOL=OFF \
> +	-DBUILD_SHARED_LIBS:BOOL=ON \
> +	-DBUILD_TESTING:BOOL=OFF \
> +	-DDISABLE_BSYMBOLIC:BOOL=ON \
> +	-DDISABLE_WERROR:BOOL=ON \
> +	-DENABLE_RDRAND:BOOL=OFF \
> +	-DENABLE_THREADING:BOOL=OFF
>  
>  # ----------------------------------------------------------------------------
>  # Install

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

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

end of thread, other threads:[~2020-05-11  5:42 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-05-05  8:28 [ptxdist] [PATCH v2 0/2] json-c: version bump 0.13.1 -> 0.14 Alexander Dahl
2020-05-05  8:28 ` [ptxdist] [PATCH v2 1/2] json-c: Revert URL to upstream provided tarballs Alexander Dahl
2020-05-11  5:42   ` [ptxdist] [APPLIED] " Michael Olbrich
2020-05-05  8:28 ` [ptxdist] [PATCH v2 2/2] json-c: version bump 0.13.1 -> 0.14 Alexander Dahl
2020-05-11  5:42   ` [ptxdist] [APPLIED] " Michael Olbrich

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