Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH v2 1/4] package/liburcu: introduce BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS symbol
@ 2015-12-13 18:47 Samuel Martin
  2015-12-13 18:47 ` [Buildroot] [PATCH v2 2/4] package/lttng-libust: use BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS dependency Samuel Martin
                   ` (4 more replies)
  0 siblings, 5 replies; 9+ messages in thread
From: Samuel Martin @ 2015-12-13 18:47 UTC (permalink / raw)
  To: buildroot

This new hidden symbol will allow simplifying the architecture dependencies
on package that depends on liburcu. These packages will be updated in follow-up
patches.

Signed-off-by: Samuel Martin <s.martin49@gmail.com>

---
changes v1->v2:
- new patch
---
 package/liburcu/Config.in | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/package/liburcu/Config.in b/package/liburcu/Config.in
index a4e8dc8..36d5353 100644
--- a/package/liburcu/Config.in
+++ b/package/liburcu/Config.in
@@ -1,6 +1,11 @@
+config BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS
+	bool
+	depends on BR2_arm || BR2_armeb || BR2_aarch64 || BR2_i386 || BR2_powerpc || BR2_x86_64
+	default y
+
 config BR2_PACKAGE_LIBURCU
 	bool "liburcu"
-	depends on BR2_arm || BR2_armeb || BR2_aarch64 || BR2_i386 || BR2_powerpc || BR2_x86_64
+	depends on BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS
 	depends on !BR2_TOOLCHAIN_HAS_GCC_BUG_58854 # liburcu/gcc-4.8.x bug
 	depends on BR2_TOOLCHAIN_HAS_THREADS
 	help
@@ -18,7 +23,7 @@ config BR2_PACKAGE_LIBURCU
 	  http://lttng.org/urcu
 
 comment "liburcu needs a toolchain w/ threads"
-	depends on BR2_arm || BR2_armeb || BR2_aarch64 || BR2_i386 || BR2_powerpc || BR2_x86_64
+	depends on BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS
 	depends on !BR2_TOOLCHAIN_HAS_GCC_BUG_58854 # liburcu/gcc-4.8.x bug
 	depends on !BR2_TOOLCHAIN_HAS_THREADS
 
-- 
2.6.4

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

* [Buildroot] [PATCH v2 2/4] package/lttng-libust: use BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS dependency
  2015-12-13 18:47 [Buildroot] [PATCH v2 1/4] package/liburcu: introduce BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS symbol Samuel Martin
@ 2015-12-13 18:47 ` Samuel Martin
  2015-12-13 20:09   ` Yann E. MORIN
  2015-12-13 18:47 ` [Buildroot] [PATCH v2 3/4] package/lttng-tools: " Samuel Martin
                   ` (3 subsequent siblings)
  4 siblings, 1 reply; 9+ messages in thread
From: Samuel Martin @ 2015-12-13 18:47 UTC (permalink / raw)
  To: buildroot

Using this newly introduced symbol lets to simplify the dependency code.

This change also adds the missing architecture dependencies on the second
comment.

Signed-off-by: Samuel Martin <s.martin49@gmail.com>

---
changes v1->v2:
- new patch
---
 package/lttng-libust/Config.in | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/package/lttng-libust/Config.in b/package/lttng-libust/Config.in
index 3c26055..26838bb 100644
--- a/package/lttng-libust/Config.in
+++ b/package/lttng-libust/Config.in
@@ -5,7 +5,7 @@ config BR2_PACKAGE_LTTNG_LIBUST
 	select BR2_PACKAGE_UTIL_LINUX_LIBUUID
 	depends on BR2_USE_WCHAR # util-linux
 	# liburcu only works on some architectures and requires threads support"
-	depends on BR2_arm || BR2_armeb || BR2_aarch64 || BR2_i386 || BR2_powerpc || BR2_x86_64
+	depends on BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS
 	depends on !BR2_TOOLCHAIN_HAS_GCC_BUG_58854 # liburcu/gcc-4.8.x bug
 	depends on BR2_TOOLCHAIN_HAS_THREADS
 	depends on !BR2_STATIC_LIBS
@@ -18,9 +18,10 @@ config BR2_PACKAGE_LTTNG_LIBUST
 	  http://lttng.org
 
 comment "lttng-libust needs a toolchain w/ dynamic library, wchar, threads"
-	depends on BR2_arm || BR2_armeb || BR2_aarch64 || BR2_i386 || BR2_powerpc || BR2_x86_64
+	depends on BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS
 	depends on BR2_STATIC_LIBS || !BR2_USE_WCHAR || !BR2_TOOLCHAIN_HAS_THREADS
 	depends on !BR2_TOOLCHAIN_HAS_GCC_BUG_58854 # liburcu/gcc-4.8.x bug
 
 comment "lttng-libust needs a toolchain not affected by GCC bug 58854"
+	depends on BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS
 	depends on BR2_TOOLCHAIN_HAS_GCC_BUG_58854
-- 
2.6.4

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

* [Buildroot] [PATCH v2 3/4] package/lttng-tools: use BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS dependency
  2015-12-13 18:47 [Buildroot] [PATCH v2 1/4] package/liburcu: introduce BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS symbol Samuel Martin
  2015-12-13 18:47 ` [Buildroot] [PATCH v2 2/4] package/lttng-libust: use BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS dependency Samuel Martin
@ 2015-12-13 18:47 ` Samuel Martin
  2015-12-13 20:15   ` Yann E. MORIN
  2015-12-13 18:47 ` [Buildroot] [PATCH v2 4/4] package/netsniff: fix missing depend on statements Samuel Martin
                   ` (2 subsequent siblings)
  4 siblings, 1 reply; 9+ messages in thread
From: Samuel Martin @ 2015-12-13 18:47 UTC (permalink / raw)
  To: buildroot

Using this newly introduced symbol lets to simplify the dependency code.

This change also adds the missing architecture dependencies on the second
comment.

Signed-off-by: Samuel Martin <s.martin49@gmail.com>

---
changes v1-> v2:
- new patch
---
 package/lttng-tools/Config.in | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/package/lttng-tools/Config.in b/package/lttng-tools/Config.in
index 4c5b5f6..599095d 100644
--- a/package/lttng-tools/Config.in
+++ b/package/lttng-tools/Config.in
@@ -6,7 +6,7 @@ config BR2_PACKAGE_LTTNG_TOOLS
 	select BR2_PACKAGE_UTIL_LINUX
 	select BR2_PACKAGE_UTIL_LINUX_LIBUUID
 	# liburcu only works on some architectures and requires thread support
-	depends on BR2_arm || BR2_armeb || BR2_aarch64 || BR2_i386 || BR2_powerpc || BR2_x86_64
+	depends on BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS
 	depends on !BR2_TOOLCHAIN_HAS_GCC_BUG_58595 # lttng-tools/gcc-4.{7,8}.x bug
 	depends on !BR2_TOOLCHAIN_HAS_GCC_BUG_58854 # liburcu/gcc-4.8.x bug
 	depends on BR2_USE_WCHAR # util-linux
@@ -30,10 +30,11 @@ config BR2_PACKAGE_LTTNG_TOOLS
 	  http://lttng.org
 
 comment "lttng-tools needs a toolchain w/ threads, wchar"
-	depends on BR2_arm || BR2_armeb || BR2_aarch64 || BR2_i386 || BR2_powerpc || BR2_x86_64
+	depends on BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS
 	depends on !BR2_TOOLCHAIN_HAS_THREADS || !BR2_USE_WCHAR
 	depends on !BR2_TOOLCHAIN_HAS_GCC_BUG_58595 # lttng-tools/gcc-4.{7,8}.x bug
 	depends on !BR2_TOOLCHAIN_HAS_GCC_BUG_58854 # liburcu/gcc-4.8.x bug
 
 comment "lttng-tools needs a toolchain not affected by GCC bug 58854 and 58595"
+	depends on BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS
 	depends on BR2_TOOLCHAIN_HAS_GCC_BUG_58854 || BR2_TOOLCHAIN_HAS_GCC_BUG_58595
-- 
2.6.4

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

* [Buildroot] [PATCH v2 4/4] package/netsniff: fix missing depend on statements
  2015-12-13 18:47 [Buildroot] [PATCH v2 1/4] package/liburcu: introduce BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS symbol Samuel Martin
  2015-12-13 18:47 ` [Buildroot] [PATCH v2 2/4] package/lttng-libust: use BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS dependency Samuel Martin
  2015-12-13 18:47 ` [Buildroot] [PATCH v2 3/4] package/lttng-tools: " Samuel Martin
@ 2015-12-13 18:47 ` Samuel Martin
  2015-12-13 20:16   ` Yann E. MORIN
  2015-12-13 20:05 ` [Buildroot] [PATCH v2 1/4] package/liburcu: introduce BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS symbol Yann E. MORIN
  2015-12-13 21:37 ` Thomas Petazzoni
  4 siblings, 1 reply; 9+ messages in thread
From: Samuel Martin @ 2015-12-13 18:47 UTC (permalink / raw)
  To: buildroot

Add missing architecture dependencies inherited from liburcu.

Fixes:
  http://autobuild.buildroot.org/results/0a9/0a90d7c8f523b90b8a4ca6312ab4835547590fb2/

Signed-off-by: Samuel Martin <s.martin49@gmail.com>

---
changes v1->v2:
- use BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS (ThomasP)
---
 package/netsniff-ng/Config.in | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/package/netsniff-ng/Config.in b/package/netsniff-ng/Config.in
index 8c48f39..b39ec96 100644
--- a/package/netsniff-ng/Config.in
+++ b/package/netsniff-ng/Config.in
@@ -11,14 +11,17 @@ config BR2_PACKAGE_NETSNIFF_NG
 	depends on BR2_TOOLCHAIN_USES_GLIBC
 	depends on BR2_TOOLCHAIN_HAS_THREADS
 	depends on !BR2_TOOLCHAIN_HAS_GCC_BUG_58854 # liburcu
+	depends on BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS
 	help
 	  netsniff-ng is a free, performant Linux network analyzer and
 	  networking toolkit. If you will, the Swiss army knife for
 	  network packets.
 
 comment "netsniff-ng needs an (e)glibc toolchain w/ threads"
+	depends on BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS
 	depends on !BR2_TOOLCHAIN_HAS_THREADS || !BR2_TOOLCHAIN_USES_GLIBC
 	depends on !BR2_TOOLCHAIN_HAS_GCC_BUG_58854
 
 comment "netsniff-ng needs a toolchain not affected by GCC bug 58854"
+	depends on BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS
 	depends on BR2_TOOLCHAIN_HAS_GCC_BUG_58854
-- 
2.6.4

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

* [Buildroot] [PATCH v2 1/4] package/liburcu: introduce BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS symbol
  2015-12-13 18:47 [Buildroot] [PATCH v2 1/4] package/liburcu: introduce BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS symbol Samuel Martin
                   ` (2 preceding siblings ...)
  2015-12-13 18:47 ` [Buildroot] [PATCH v2 4/4] package/netsniff: fix missing depend on statements Samuel Martin
@ 2015-12-13 20:05 ` Yann E. MORIN
  2015-12-13 21:37 ` Thomas Petazzoni
  4 siblings, 0 replies; 9+ messages in thread
From: Yann E. MORIN @ 2015-12-13 20:05 UTC (permalink / raw)
  To: buildroot

Samuel, All,

On 2015-12-13 19:47 +0100, Samuel Martin spake thusly:
> This new hidden symbol will allow simplifying the architecture dependencies
> on package that depends on liburcu. These packages will be updated in follow-up
> patches.
> 
> Signed-off-by: Samuel Martin <s.martin49@gmail.com>

Reviewed-by: "Yann E. MORIN" <yann.morin.1998@free.fr>

Regards,
Yann E. MORIN.

> ---
> changes v1->v2:
> - new patch
> ---
>  package/liburcu/Config.in | 9 +++++++--
>  1 file changed, 7 insertions(+), 2 deletions(-)
> 
> diff --git a/package/liburcu/Config.in b/package/liburcu/Config.in
> index a4e8dc8..36d5353 100644
> --- a/package/liburcu/Config.in
> +++ b/package/liburcu/Config.in
> @@ -1,6 +1,11 @@
> +config BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS
> +	bool
> +	depends on BR2_arm || BR2_armeb || BR2_aarch64 || BR2_i386 || BR2_powerpc || BR2_x86_64
> +	default y
> +
>  config BR2_PACKAGE_LIBURCU
>  	bool "liburcu"
> -	depends on BR2_arm || BR2_armeb || BR2_aarch64 || BR2_i386 || BR2_powerpc || BR2_x86_64
> +	depends on BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS
>  	depends on !BR2_TOOLCHAIN_HAS_GCC_BUG_58854 # liburcu/gcc-4.8.x bug
>  	depends on BR2_TOOLCHAIN_HAS_THREADS
>  	help
> @@ -18,7 +23,7 @@ config BR2_PACKAGE_LIBURCU
>  	  http://lttng.org/urcu
>  
>  comment "liburcu needs a toolchain w/ threads"
> -	depends on BR2_arm || BR2_armeb || BR2_aarch64 || BR2_i386 || BR2_powerpc || BR2_x86_64
> +	depends on BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS
>  	depends on !BR2_TOOLCHAIN_HAS_GCC_BUG_58854 # liburcu/gcc-4.8.x bug
>  	depends on !BR2_TOOLCHAIN_HAS_THREADS
>  
> -- 
> 2.6.4
> 
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

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

* [Buildroot] [PATCH v2 2/4] package/lttng-libust: use BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS dependency
  2015-12-13 18:47 ` [Buildroot] [PATCH v2 2/4] package/lttng-libust: use BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS dependency Samuel Martin
@ 2015-12-13 20:09   ` Yann E. MORIN
  0 siblings, 0 replies; 9+ messages in thread
From: Yann E. MORIN @ 2015-12-13 20:09 UTC (permalink / raw)
  To: buildroot

Samuel, All,

On 2015-12-13 19:47 +0100, Samuel Martin spake thusly:
> Using this newly introduced symbol lets to simplify the dependency code.
> 
> This change also adds the missing architecture dependencies on the second
> comment.
> 
> Signed-off-by: Samuel Martin <s.martin49@gmail.com>

Reviewed-by: "Yann E. MORIN" <yann.morin.1998@free.fr>

Regards,
Yann E. MORIN.

> ---
> changes v1->v2:
> - new patch
> ---
>  package/lttng-libust/Config.in | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/package/lttng-libust/Config.in b/package/lttng-libust/Config.in
> index 3c26055..26838bb 100644
> --- a/package/lttng-libust/Config.in
> +++ b/package/lttng-libust/Config.in
> @@ -5,7 +5,7 @@ config BR2_PACKAGE_LTTNG_LIBUST
>  	select BR2_PACKAGE_UTIL_LINUX_LIBUUID
>  	depends on BR2_USE_WCHAR # util-linux
>  	# liburcu only works on some architectures and requires threads support"
> -	depends on BR2_arm || BR2_armeb || BR2_aarch64 || BR2_i386 || BR2_powerpc || BR2_x86_64
> +	depends on BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS
>  	depends on !BR2_TOOLCHAIN_HAS_GCC_BUG_58854 # liburcu/gcc-4.8.x bug
>  	depends on BR2_TOOLCHAIN_HAS_THREADS
>  	depends on !BR2_STATIC_LIBS
> @@ -18,9 +18,10 @@ config BR2_PACKAGE_LTTNG_LIBUST
>  	  http://lttng.org
>  
>  comment "lttng-libust needs a toolchain w/ dynamic library, wchar, threads"
> -	depends on BR2_arm || BR2_armeb || BR2_aarch64 || BR2_i386 || BR2_powerpc || BR2_x86_64
> +	depends on BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS
>  	depends on BR2_STATIC_LIBS || !BR2_USE_WCHAR || !BR2_TOOLCHAIN_HAS_THREADS
>  	depends on !BR2_TOOLCHAIN_HAS_GCC_BUG_58854 # liburcu/gcc-4.8.x bug
>  
>  comment "lttng-libust needs a toolchain not affected by GCC bug 58854"
> +	depends on BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS
>  	depends on BR2_TOOLCHAIN_HAS_GCC_BUG_58854
> -- 
> 2.6.4
> 
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

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

* [Buildroot] [PATCH v2 3/4] package/lttng-tools: use BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS dependency
  2015-12-13 18:47 ` [Buildroot] [PATCH v2 3/4] package/lttng-tools: " Samuel Martin
@ 2015-12-13 20:15   ` Yann E. MORIN
  0 siblings, 0 replies; 9+ messages in thread
From: Yann E. MORIN @ 2015-12-13 20:15 UTC (permalink / raw)
  To: buildroot

Samuel, All,

On 2015-12-13 19:47 +0100, Samuel Martin spake thusly:
> Using this newly introduced symbol lets to simplify the dependency code.
> 
> This change also adds the missing architecture dependencies on the second
> comment.
> 
> Signed-off-by: Samuel Martin <s.martin49@gmail.com>

Reviewed-by: "Yann E. MORIN" <yann.morin.1998@free.fr>

Regards,
Yann E. MORIN.

> ---
> changes v1-> v2:
> - new patch
> ---
>  package/lttng-tools/Config.in | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/package/lttng-tools/Config.in b/package/lttng-tools/Config.in
> index 4c5b5f6..599095d 100644
> --- a/package/lttng-tools/Config.in
> +++ b/package/lttng-tools/Config.in
> @@ -6,7 +6,7 @@ config BR2_PACKAGE_LTTNG_TOOLS
>  	select BR2_PACKAGE_UTIL_LINUX
>  	select BR2_PACKAGE_UTIL_LINUX_LIBUUID
>  	# liburcu only works on some architectures and requires thread support
> -	depends on BR2_arm || BR2_armeb || BR2_aarch64 || BR2_i386 || BR2_powerpc || BR2_x86_64
> +	depends on BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS
>  	depends on !BR2_TOOLCHAIN_HAS_GCC_BUG_58595 # lttng-tools/gcc-4.{7,8}.x bug
>  	depends on !BR2_TOOLCHAIN_HAS_GCC_BUG_58854 # liburcu/gcc-4.8.x bug
>  	depends on BR2_USE_WCHAR # util-linux
> @@ -30,10 +30,11 @@ config BR2_PACKAGE_LTTNG_TOOLS
>  	  http://lttng.org
>  
>  comment "lttng-tools needs a toolchain w/ threads, wchar"
> -	depends on BR2_arm || BR2_armeb || BR2_aarch64 || BR2_i386 || BR2_powerpc || BR2_x86_64
> +	depends on BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS
>  	depends on !BR2_TOOLCHAIN_HAS_THREADS || !BR2_USE_WCHAR
>  	depends on !BR2_TOOLCHAIN_HAS_GCC_BUG_58595 # lttng-tools/gcc-4.{7,8}.x bug
>  	depends on !BR2_TOOLCHAIN_HAS_GCC_BUG_58854 # liburcu/gcc-4.8.x bug
>  
>  comment "lttng-tools needs a toolchain not affected by GCC bug 58854 and 58595"
> +	depends on BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS
>  	depends on BR2_TOOLCHAIN_HAS_GCC_BUG_58854 || BR2_TOOLCHAIN_HAS_GCC_BUG_58595
> -- 
> 2.6.4
> 
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

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

* [Buildroot] [PATCH v2 4/4] package/netsniff: fix missing depend on statements
  2015-12-13 18:47 ` [Buildroot] [PATCH v2 4/4] package/netsniff: fix missing depend on statements Samuel Martin
@ 2015-12-13 20:16   ` Yann E. MORIN
  0 siblings, 0 replies; 9+ messages in thread
From: Yann E. MORIN @ 2015-12-13 20:16 UTC (permalink / raw)
  To: buildroot

Samuel, All,

On 2015-12-13 19:47 +0100, Samuel Martin spake thusly:
> Add missing architecture dependencies inherited from liburcu.
> 
> Fixes:
>   http://autobuild.buildroot.org/results/0a9/0a90d7c8f523b90b8a4ca6312ab4835547590fb2/
> 
> Signed-off-by: Samuel Martin <s.martin49@gmail.com>

Reviewed-by: "Yann E. MORIN" <yann.morin.1998@free.fr>

Regards,
Yann E. MORIN.

> ---
> changes v1->v2:
> - use BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS (ThomasP)
> ---
>  package/netsniff-ng/Config.in | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/package/netsniff-ng/Config.in b/package/netsniff-ng/Config.in
> index 8c48f39..b39ec96 100644
> --- a/package/netsniff-ng/Config.in
> +++ b/package/netsniff-ng/Config.in
> @@ -11,14 +11,17 @@ config BR2_PACKAGE_NETSNIFF_NG
>  	depends on BR2_TOOLCHAIN_USES_GLIBC
>  	depends on BR2_TOOLCHAIN_HAS_THREADS
>  	depends on !BR2_TOOLCHAIN_HAS_GCC_BUG_58854 # liburcu
> +	depends on BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS
>  	help
>  	  netsniff-ng is a free, performant Linux network analyzer and
>  	  networking toolkit. If you will, the Swiss army knife for
>  	  network packets.
>  
>  comment "netsniff-ng needs an (e)glibc toolchain w/ threads"
> +	depends on BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS
>  	depends on !BR2_TOOLCHAIN_HAS_THREADS || !BR2_TOOLCHAIN_USES_GLIBC
>  	depends on !BR2_TOOLCHAIN_HAS_GCC_BUG_58854
>  
>  comment "netsniff-ng needs a toolchain not affected by GCC bug 58854"
> +	depends on BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS
>  	depends on BR2_TOOLCHAIN_HAS_GCC_BUG_58854
> -- 
> 2.6.4
> 
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

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

* [Buildroot] [PATCH v2 1/4] package/liburcu: introduce BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS symbol
  2015-12-13 18:47 [Buildroot] [PATCH v2 1/4] package/liburcu: introduce BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS symbol Samuel Martin
                   ` (3 preceding siblings ...)
  2015-12-13 20:05 ` [Buildroot] [PATCH v2 1/4] package/liburcu: introduce BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS symbol Yann E. MORIN
@ 2015-12-13 21:37 ` Thomas Petazzoni
  4 siblings, 0 replies; 9+ messages in thread
From: Thomas Petazzoni @ 2015-12-13 21:37 UTC (permalink / raw)
  To: buildroot

Dear Samuel Martin,

On Sun, 13 Dec 2015 19:47:48 +0100, Samuel Martin wrote:
> This new hidden symbol will allow simplifying the architecture dependencies
> on package that depends on liburcu. These packages will be updated in follow-up
> patches.
> 
> Signed-off-by: Samuel Martin <s.martin49@gmail.com>
> 
> ---
> changes v1->v2:
> - new patch
> ---
>  package/liburcu/Config.in | 9 +++++++--
>  1 file changed, 7 insertions(+), 2 deletions(-)

All four patches applied, thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

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

end of thread, other threads:[~2015-12-13 21:37 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-12-13 18:47 [Buildroot] [PATCH v2 1/4] package/liburcu: introduce BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS symbol Samuel Martin
2015-12-13 18:47 ` [Buildroot] [PATCH v2 2/4] package/lttng-libust: use BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS dependency Samuel Martin
2015-12-13 20:09   ` Yann E. MORIN
2015-12-13 18:47 ` [Buildroot] [PATCH v2 3/4] package/lttng-tools: " Samuel Martin
2015-12-13 20:15   ` Yann E. MORIN
2015-12-13 18:47 ` [Buildroot] [PATCH v2 4/4] package/netsniff: fix missing depend on statements Samuel Martin
2015-12-13 20:16   ` Yann E. MORIN
2015-12-13 20:05 ` [Buildroot] [PATCH v2 1/4] package/liburcu: introduce BR2_PACKAGE_LIBURCU_ARCH_SUPPORTS symbol Yann E. MORIN
2015-12-13 21:37 ` Thomas Petazzoni

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