diff mbox

ltrace: disable on mips/mipsel

Message ID 1408038104-9440-1-git-send-email-thomas.petazzoni@free-electrons.com
State Accepted
Commit dfaa18af00dfcff71a0159adaafb0880e58fa6a1
Headers show

Commit Message

Thomas Petazzoni Aug. 14, 2014, 5:41 p.m. UTC
ltrace currently doesn't build on mips/mipsel, and it's an upstream
issue that has been reported. Until it get fixed, let's disable ltrace
for mips/mipsel.

Fixes:

  http://autobuild.buildroot.org/results/43a/43a8fc7075f52eab74ebfee4c9f25dd2b886e75e/

Cc: Vicente Olivert Riera <Vincent.Riera@imgtec.com>
Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
---
 package/ltrace/Config.in | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

Comments

Vicente Olivert Riera Aug. 14, 2014, 5:42 p.m. UTC | #1
On 08/14/2014 06:41 PM, Thomas Petazzoni wrote:
> ltrace currently doesn't build on mips/mipsel, and it's an upstream
> issue that has been reported. Until it get fixed, let's disable ltrace
> for mips/mipsel.
>
> Fixes:
>
>    http://autobuild.buildroot.org/results/43a/43a8fc7075f52eab74ebfee4c9f25dd2b886e75e/
>
> Cc: Vicente Olivert Riera <Vincent.Riera@imgtec.com>
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> ---
>   package/ltrace/Config.in | 6 +++++-
>   1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/package/ltrace/Config.in b/package/ltrace/Config.in
> index 4d1acfa..8cabfa2 100644
> --- a/package/ltrace/Config.in
> +++ b/package/ltrace/Config.in
> @@ -1,6 +1,10 @@
>   config BR2_PACKAGE_LTRACE
>   	bool "ltrace"
> -	depends on (BR2_i386 || BR2_arm || BR2_mips || BR2_mipsel \
> +	# ltrace normally has mips/mipsel support, but it's currently
> +	# broken (error: 'struct ltelf' has no member named
> +	# 'relplt_count'). Issue reported upstream at
> +	# https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=756764.
> +	depends on (BR2_i386 || BR2_arm \
>   		|| BR2_powerpc || BR2_sparc || BR2_x86_64)
>   	select BR2_PACKAGE_LIBELF
>   	select BR2_PACKAGE_ELFUTILS
>

Thanks Thomas.
Peter Korsgaard Aug. 15, 2014, 8:14 a.m. UTC | #2
>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni@free-electrons.com> writes:

 > ltrace currently doesn't build on mips/mipsel, and it's an upstream
 > issue that has been reported. Until it get fixed, let's disable ltrace
 > for mips/mipsel.

 > Fixes:

 >   http://autobuild.buildroot.org/results/43a/43a8fc7075f52eab74ebfee4c9f25dd2b886e75e/

 > Cc: Vicente Olivert Riera <Vincent.Riera@imgtec.com>
 > Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>

Committed, thanks.
diff mbox

Patch

diff --git a/package/ltrace/Config.in b/package/ltrace/Config.in
index 4d1acfa..8cabfa2 100644
--- a/package/ltrace/Config.in
+++ b/package/ltrace/Config.in
@@ -1,6 +1,10 @@ 
 config BR2_PACKAGE_LTRACE
 	bool "ltrace"
-	depends on (BR2_i386 || BR2_arm || BR2_mips || BR2_mipsel \
+	# ltrace normally has mips/mipsel support, but it's currently
+	# broken (error: 'struct ltelf' has no member named
+	# 'relplt_count'). Issue reported upstream at
+	# https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=756764.
+	depends on (BR2_i386 || BR2_arm \
 		|| BR2_powerpc || BR2_sparc || BR2_x86_64)
 	select BR2_PACKAGE_LIBELF
 	select BR2_PACKAGE_ELFUTILS