diff mbox

luvi: build for the right MIPS endianness

Message ID 1453133375-54831-1-git-send-email-Vincent.Riera@imgtec.com
State Accepted
Headers show

Commit Message

Vicente Olivert Riera Jan. 18, 2016, 4:09 p.m. UTC
Otherwise it will cause build failures like this one:

[100%] Linking C executable luvi
/home/buildroot/autobuild/run/instance-2/output/host/opt/ext-toolchain/bin/../lib/gcc/mipsel-buildroot-linux-gnu/5.2.0/../../../../mipsel-buildroot-linux-gnu/bin/ld:
jitted_tmp/src/lua/init.lua_luvi_generated.o: compiled for a big endian
system and target is little endian

Signed-off-by: Vicente Olivert Riera <Vincent.Riera@imgtec.com>
---
 package/luvi/luvi.mk | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

Comments

Thomas Petazzoni Jan. 19, 2016, 9 p.m. UTC | #1
Dear Vicente Olivert Riera,

On Mon, 18 Jan 2016 16:09:35 +0000, Vicente Olivert Riera wrote:
> Otherwise it will cause build failures like this one:
> 
> [100%] Linking C executable luvi
> /home/buildroot/autobuild/run/instance-2/output/host/opt/ext-toolchain/bin/../lib/gcc/mipsel-buildroot-linux-gnu/5.2.0/../../../../mipsel-buildroot-linux-gnu/bin/ld:
> jitted_tmp/src/lua/init.lua_luvi_generated.o: compiled for a big endian
> system and target is little endian
> 
> Signed-off-by: Vicente Olivert Riera <Vincent.Riera@imgtec.com>
> ---
>  package/luvi/luvi.mk | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)

Applied, thanks.

Thomas
Arnout Vandecappelle Jan. 19, 2016, 11:09 p.m. UTC | #2
On 18-01-16 17:09, Vicente Olivert Riera wrote:
> Otherwise it will cause build failures like this one:
> 
> [100%] Linking C executable luvi
> /home/buildroot/autobuild/run/instance-2/output/host/opt/ext-toolchain/bin/../lib/gcc/mipsel-buildroot-linux-gnu/5.2.0/../../../../mipsel-buildroot-linux-gnu/bin/ld:
> jitted_tmp/src/lua/init.lua_luvi_generated.o: compiled for a big endian
> system and target is little endian
> 
> Signed-off-by: Vicente Olivert Riera <Vincent.Riera@imgtec.com>
> ---
>  package/luvi/luvi.mk | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/package/luvi/luvi.mk b/package/luvi/luvi.mk
> index ef1c2e1..d83933e 100644
> --- a/package/luvi/luvi.mk
> +++ b/package/luvi/luvi.mk
> @@ -20,8 +20,10 @@ else ifeq ($(BR2_powerpc),y)
>  LUVI_TARGET_ARCH = ppc
>  else ifeq ($(BR2_arm)$(BR2_armeb),y)
>  LUVI_TARGET_ARCH = arm
> -else ifeq ($(BR2_mips)$(BR2_mipsel),y)
> +else ifeq ($(BR2_mips),y)
>  LUVI_TARGET_ARCH = mips
> +else ifeq ($(BR2_mipsel),y)
> +LUVI_TARGET_ARCH = mipsel

 This could just fall into the BR2_ARCH situation below, no?

 Regards,
 Arnout

>  else
>  LUVI_TARGET_ARCH = $(BR2_ARCH)
>  endif
>
diff mbox

Patch

diff --git a/package/luvi/luvi.mk b/package/luvi/luvi.mk
index ef1c2e1..d83933e 100644
--- a/package/luvi/luvi.mk
+++ b/package/luvi/luvi.mk
@@ -20,8 +20,10 @@  else ifeq ($(BR2_powerpc),y)
 LUVI_TARGET_ARCH = ppc
 else ifeq ($(BR2_arm)$(BR2_armeb),y)
 LUVI_TARGET_ARCH = arm
-else ifeq ($(BR2_mips)$(BR2_mipsel),y)
+else ifeq ($(BR2_mips),y)
 LUVI_TARGET_ARCH = mips
+else ifeq ($(BR2_mipsel),y)
+LUVI_TARGET_ARCH = mipsel
 else
 LUVI_TARGET_ARCH = $(BR2_ARCH)
 endif