Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] google-breakpad: disable on AArch64
@ 2015-05-06 19:28 Thomas Petazzoni
  2015-05-06 19:46 ` Romain Naour
  0 siblings, 1 reply; 3+ messages in thread
From: Thomas Petazzoni @ 2015-05-06 19:28 UTC (permalink / raw)
  To: buildroot

Currently doesn't build, will be fixed upstream in a later version.

Fixes:

  http://autobuild.buildroot.org/results/f45/f45925e63dd6ce09e41e1ce9a449d8fd57575314/

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
---
 package/google-breakpad/Config.in      | 4 +++-
 package/lua-periphery/lua-periphery.mk | 1 +
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/package/google-breakpad/Config.in b/package/google-breakpad/Config.in
index a95bf63..401a983 100644
--- a/package/google-breakpad/Config.in
+++ b/package/google-breakpad/Config.in
@@ -1,6 +1,8 @@
 config BR2_PACKAGE_GOOGLE_BREAKPAD_ARCH_SUPPORTS
 	bool
-	default y if BR2_i386 || BR2_x86_64 || BR2_arm || BR2_aarch64 || \
+	# Even though the source code pretends to support aarch64, it
+	# doesn't build as of Google Breakpad r1373.
+	default y if BR2_i386 || BR2_x86_64 || BR2_arm || \
 		BR2_mips || BR2_mipsel
 
 config BR2_PACKAGE_GOOGLE_BREAKPAD
diff --git a/package/lua-periphery/lua-periphery.mk b/package/lua-periphery/lua-periphery.mk
index 0717a27..9bf38de 100644
--- a/package/lua-periphery/lua-periphery.mk
+++ b/package/lua-periphery/lua-periphery.mk
@@ -9,5 +9,6 @@ LUA_PERIPHERY_VERSION = $(LUA_PERIPHERY_VERSION_UPSTREAM)-1
 LUA_PERIPHERY_SUBDIR = lua-periphery
 LUA_PERIPHERY_LICENSE = MIT
 LUA_PERIPHERY_LICENSE_FILES = $(LUA_PERIPHERY_SUBDIR)/LICENSE
+LUA_PERIPHERY_DEPENDENCIES = c-periphery
 
 $(eval $(luarocks-package))
-- 
2.1.0

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

* [Buildroot] [PATCH] google-breakpad: disable on AArch64
  2015-05-06 19:28 [Buildroot] [PATCH] google-breakpad: disable on AArch64 Thomas Petazzoni
@ 2015-05-06 19:46 ` Romain Naour
  2015-05-06 20:43   ` Thomas Petazzoni
  0 siblings, 1 reply; 3+ messages in thread
From: Romain Naour @ 2015-05-06 19:46 UTC (permalink / raw)
  To: buildroot

Hi Thomas,

Le 06/05/2015 21:28, Thomas Petazzoni a ?crit :
> Currently doesn't build, will be fixed upstream in a later version.

If you try with the aarch64 cs toolchain and the patch [1], google-breakpad
build fine.
But I don't have a aarch64 cpu to do a runtime test through.

[1] http://patchwork.ozlabs.org/patch/467857/

> 
> Fixes:
> 
>   http://autobuild.buildroot.org/results/f45/f45925e63dd6ce09e41e1ce9a449d8fd57575314/
> 
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> ---
>  package/google-breakpad/Config.in      | 4 +++-
>  package/lua-periphery/lua-periphery.mk | 1 +
>  2 files changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/package/google-breakpad/Config.in b/package/google-breakpad/Config.in
> index a95bf63..401a983 100644
> --- a/package/google-breakpad/Config.in
> +++ b/package/google-breakpad/Config.in
> @@ -1,6 +1,8 @@
>  config BR2_PACKAGE_GOOGLE_BREAKPAD_ARCH_SUPPORTS
>  	bool
> -	default y if BR2_i386 || BR2_x86_64 || BR2_arm || BR2_aarch64 || \
> +	# Even though the source code pretends to support aarch64, it
> +	# doesn't build as of Google Breakpad r1373.
> +	default y if BR2_i386 || BR2_x86_64 || BR2_arm || \
>  		BR2_mips || BR2_mipsel
>  
>  config BR2_PACKAGE_GOOGLE_BREAKPAD
> diff --git a/package/lua-periphery/lua-periphery.mk b/package/lua-periphery/lua-periphery.mk
> index 0717a27..9bf38de 100644
> --- a/package/lua-periphery/lua-periphery.mk
> +++ b/package/lua-periphery/lua-periphery.mk
> @@ -9,5 +9,6 @@ LUA_PERIPHERY_VERSION = $(LUA_PERIPHERY_VERSION_UPSTREAM)-1
>  LUA_PERIPHERY_SUBDIR = lua-periphery
>  LUA_PERIPHERY_LICENSE = MIT
>  LUA_PERIPHERY_LICENSE_FILES = $(LUA_PERIPHERY_SUBDIR)/LICENSE
> +LUA_PERIPHERY_DEPENDENCIES = c-periphery

This is not related to google-breakpad ;-)

Best regards,
Romain

>  
>  $(eval $(luarocks-package))
> 

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

* [Buildroot] [PATCH] google-breakpad: disable on AArch64
  2015-05-06 19:46 ` Romain Naour
@ 2015-05-06 20:43   ` Thomas Petazzoni
  0 siblings, 0 replies; 3+ messages in thread
From: Thomas Petazzoni @ 2015-05-06 20:43 UTC (permalink / raw)
  To: buildroot

Dear Romain Naour,

On Wed, 06 May 2015 21:46:33 +0200, Romain Naour wrote:
> Hi Thomas,
> 
> Le 06/05/2015 21:28, Thomas Petazzoni a ?crit :
> > Currently doesn't build, will be fixed upstream in a later version.
> 
> If you try with the aarch64 cs toolchain and the patch [1], google-breakpad
> build fine.
> But I don't have a aarch64 cpu to do a runtime test through.
> 
> [1] http://patchwork.ozlabs.org/patch/467857/

With the Linaro toolchain, it fails with:

  http://autobuild.buildroot.org/results/f45/f45925e63dd6ce09e41e1ce9a449d8fd57575314/build-end.log

But if it builds with the CS AArch64 toolchain, then the issue is
different.

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

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

end of thread, other threads:[~2015-05-06 20:43 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-05-06 19:28 [Buildroot] [PATCH] google-breakpad: disable on AArch64 Thomas Petazzoni
2015-05-06 19:46 ` Romain Naour
2015-05-06 20:43   ` Thomas Petazzoni

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