* [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 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 ` 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 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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox