All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 1/2] blktrace: add missing dependency on Config.in comment
@ 2016-08-06 22:11 Thomas Petazzoni
  2016-08-06 22:11 ` [Buildroot] [PATCH 2/2] blktrace: disable on PowerPC Thomas Petazzoni
                   ` (2 more replies)
  0 siblings, 3 replies; 5+ messages in thread
From: Thomas Petazzoni @ 2016-08-06 22:11 UTC (permalink / raw)
  To: buildroot

The BR2_PACKAGE_BLKTRACE option "depends on
BR2_PACKAGE_LIBAIO_ARCH_SUPPORTS", but this architecture dependency was
not replicated in the Config.in comment. This commit fixes this
inconsistency.

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
---
 package/blktrace/Config.in | 1 +
 1 file changed, 1 insertion(+)

diff --git a/package/blktrace/Config.in b/package/blktrace/Config.in
index 958f004..c17de80 100644
--- a/package/blktrace/Config.in
+++ b/package/blktrace/Config.in
@@ -12,4 +12,5 @@ config BR2_PACKAGE_BLKTRACE
 	  http://git.kernel.dk/?p=blktrace.git;a=summary
 
 comment "blktrace needs a glibc or musl toolchain"
+	depends on BR2_PACKAGE_LIBAIO_ARCH_SUPPORTS
 	depends on !(BR2_TOOLCHAIN_USES_GLIBC || BR2_TOOLCHAIN_USES_MUSL)
-- 
2.7.4

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

* [Buildroot] [PATCH 2/2] blktrace: disable on PowerPC
  2016-08-06 22:11 [Buildroot] [PATCH 1/2] blktrace: add missing dependency on Config.in comment Thomas Petazzoni
@ 2016-08-06 22:11 ` Thomas Petazzoni
  2016-08-07 20:06   ` Yann E. MORIN
  2016-08-07 20:04 ` [Buildroot] [PATCH 1/2] blktrace: add missing dependency on Config.in comment Yann E. MORIN
  2016-08-07 20:59 ` Thomas Petazzoni
  2 siblings, 1 reply; 5+ messages in thread
From: Thomas Petazzoni @ 2016-08-06 22:11 UTC (permalink / raw)
  To: buildroot

This package has been failing to build on PowerPC since March 2016, with
nobody taking care of the problem, so let's disable this package on
PowerPC for the time being. PowerPC is the only architecture on which
blktrace fails to build currently.

Fixes:

  http://autobuild.buildroot.net/results/5a673c692aeb957cd8beb2c2c10ce5fd882ba4b5/

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
---
 package/blktrace/Config.in | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/package/blktrace/Config.in b/package/blktrace/Config.in
index c17de80..1b8212d 100644
--- a/package/blktrace/Config.in
+++ b/package/blktrace/Config.in
@@ -3,6 +3,8 @@ config BR2_PACKAGE_BLKTRACE
 	# Uses posix_spawn()
 	depends on BR2_TOOLCHAIN_USES_GLIBC || BR2_TOOLCHAIN_USES_MUSL
 	depends on BR2_PACKAGE_LIBAIO_ARCH_SUPPORTS
+	# issue: hidden symbol `_rest32gpr_30_x` is referenced by DSO
+	depends on !BR2_powerpc
 	select BR2_PACKAGE_LIBAIO
 	help
 	  blktrace is a block layer IO tracing mechanism which provides
@@ -13,4 +15,5 @@ config BR2_PACKAGE_BLKTRACE
 
 comment "blktrace needs a glibc or musl toolchain"
 	depends on BR2_PACKAGE_LIBAIO_ARCH_SUPPORTS
+	depends on !BR2_powerpc
 	depends on !(BR2_TOOLCHAIN_USES_GLIBC || BR2_TOOLCHAIN_USES_MUSL)
-- 
2.7.4

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

* [Buildroot] [PATCH 1/2] blktrace: add missing dependency on Config.in comment
  2016-08-06 22:11 [Buildroot] [PATCH 1/2] blktrace: add missing dependency on Config.in comment Thomas Petazzoni
  2016-08-06 22:11 ` [Buildroot] [PATCH 2/2] blktrace: disable on PowerPC Thomas Petazzoni
@ 2016-08-07 20:04 ` Yann E. MORIN
  2016-08-07 20:59 ` Thomas Petazzoni
  2 siblings, 0 replies; 5+ messages in thread
From: Yann E. MORIN @ 2016-08-07 20:04 UTC (permalink / raw)
  To: buildroot

Thomas, All,

On 2016-08-07 00:11 +0200, Thomas Petazzoni spake thusly:
> The BR2_PACKAGE_BLKTRACE option "depends on
> BR2_PACKAGE_LIBAIO_ARCH_SUPPORTS", but this architecture dependency was
> not replicated in the Config.in comment. This commit fixes this
> inconsistency.
> 
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>

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

Regards,
Yann E. MORIN.

> ---
>  package/blktrace/Config.in | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/package/blktrace/Config.in b/package/blktrace/Config.in
> index 958f004..c17de80 100644
> --- a/package/blktrace/Config.in
> +++ b/package/blktrace/Config.in
> @@ -12,4 +12,5 @@ config BR2_PACKAGE_BLKTRACE
>  	  http://git.kernel.dk/?p=blktrace.git;a=summary
>  
>  comment "blktrace needs a glibc or musl toolchain"
> +	depends on BR2_PACKAGE_LIBAIO_ARCH_SUPPORTS
>  	depends on !(BR2_TOOLCHAIN_USES_GLIBC || BR2_TOOLCHAIN_USES_MUSL)
> -- 
> 2.7.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] 5+ messages in thread

* [Buildroot] [PATCH 2/2] blktrace: disable on PowerPC
  2016-08-06 22:11 ` [Buildroot] [PATCH 2/2] blktrace: disable on PowerPC Thomas Petazzoni
@ 2016-08-07 20:06   ` Yann E. MORIN
  0 siblings, 0 replies; 5+ messages in thread
From: Yann E. MORIN @ 2016-08-07 20:06 UTC (permalink / raw)
  To: buildroot

Thomas, All,

On 2016-08-07 00:11 +0200, Thomas Petazzoni spake thusly:
> This package has been failing to build on PowerPC since March 2016, with
> nobody taking care of the problem, so let's disable this package on
> PowerPC for the time being. PowerPC is the only architecture on which
> blktrace fails to build currently.
> 
> Fixes:
> 
>   http://autobuild.buildroot.net/results/5a673c692aeb957cd8beb2c2c10ce5fd882ba4b5/
> 
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>

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

Regards,
Yann E. MORIN.

> ---
>  package/blktrace/Config.in | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/package/blktrace/Config.in b/package/blktrace/Config.in
> index c17de80..1b8212d 100644
> --- a/package/blktrace/Config.in
> +++ b/package/blktrace/Config.in
> @@ -3,6 +3,8 @@ config BR2_PACKAGE_BLKTRACE
>  	# Uses posix_spawn()
>  	depends on BR2_TOOLCHAIN_USES_GLIBC || BR2_TOOLCHAIN_USES_MUSL
>  	depends on BR2_PACKAGE_LIBAIO_ARCH_SUPPORTS
> +	# issue: hidden symbol `_rest32gpr_30_x` is referenced by DSO
> +	depends on !BR2_powerpc
>  	select BR2_PACKAGE_LIBAIO
>  	help
>  	  blktrace is a block layer IO tracing mechanism which provides
> @@ -13,4 +15,5 @@ config BR2_PACKAGE_BLKTRACE
>  
>  comment "blktrace needs a glibc or musl toolchain"
>  	depends on BR2_PACKAGE_LIBAIO_ARCH_SUPPORTS
> +	depends on !BR2_powerpc
>  	depends on !(BR2_TOOLCHAIN_USES_GLIBC || BR2_TOOLCHAIN_USES_MUSL)
> -- 
> 2.7.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] 5+ messages in thread

* [Buildroot] [PATCH 1/2] blktrace: add missing dependency on Config.in comment
  2016-08-06 22:11 [Buildroot] [PATCH 1/2] blktrace: add missing dependency on Config.in comment Thomas Petazzoni
  2016-08-06 22:11 ` [Buildroot] [PATCH 2/2] blktrace: disable on PowerPC Thomas Petazzoni
  2016-08-07 20:04 ` [Buildroot] [PATCH 1/2] blktrace: add missing dependency on Config.in comment Yann E. MORIN
@ 2016-08-07 20:59 ` Thomas Petazzoni
  2 siblings, 0 replies; 5+ messages in thread
From: Thomas Petazzoni @ 2016-08-07 20:59 UTC (permalink / raw)
  To: buildroot

Hello,

On Sun,  7 Aug 2016 00:11:13 +0200, Thomas Petazzoni wrote:
> The BR2_PACKAGE_BLKTRACE option "depends on
> BR2_PACKAGE_LIBAIO_ARCH_SUPPORTS", but this architecture dependency was
> not replicated in the Config.in comment. This commit fixes this
> inconsistency.
> 
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> ---
>  package/blktrace/Config.in | 1 +
>  1 file changed, 1 insertion(+)

Both applied.

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

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

end of thread, other threads:[~2016-08-07 20:59 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-08-06 22:11 [Buildroot] [PATCH 1/2] blktrace: add missing dependency on Config.in comment Thomas Petazzoni
2016-08-06 22:11 ` [Buildroot] [PATCH 2/2] blktrace: disable on PowerPC Thomas Petazzoni
2016-08-07 20:06   ` Yann E. MORIN
2016-08-07 20:04 ` [Buildroot] [PATCH 1/2] blktrace: add missing dependency on Config.in comment Yann E. MORIN
2016-08-07 20:59 ` Thomas Petazzoni

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.