Opened 9 years ago

Closed 9 years ago

Last modified 7 months ago

#4757 closed defect (fixed)

BUILD: angstrom+non-arm and configure

Reported by: SF/gnutoo Owned by: djwillis
Priority: normal Component: Ports
Keywords: script Cc:
Game:

Description

hi,
The configure script of the 1.0.0 release(I was told that older images had it too) has that:
arm-linux|arm*-linux-gnueabi|arm-*-linux|*-angstrom-linux)
echo "Cross-compiling to $_host, forcing endianness, alignment and type sizes"
DEFINES="$DEFINES -DUNIX -DUSE_ARM_SMUSH_ASM"
http://www.angstrom-distribution.org/repo/?pkgname=libc6 shows that angstrom is also built for ppc and x86
So if someone has a non-arm angstrom compiler it fails when compiling the arm assembly for the audio:
sound/rate_arm_asm.s:1: Error: junk at end of line, first unrecognized character is `@'
Denis.

Ticket imported from: #2941762. Ticket imported from: bugs/4757.

Change History (9)

comment:1 Changed 9 years ago by djwillis

Owner: set to djwillis

comment:2 Changed 9 years ago by djwillis

Happy to pick this up as I actively develop packages for the Angstrom distribution and OpenEmbedded.

It should be a simple tweak to configure, I'll get this patched in trunk and we can look at the merits of a backport to 1.0.0.

comment:3 Changed 9 years ago by djwillis

Proposed fix (simple). Tested on local Angstrom builds.

Change configure line 1353 in trunk from

arm-linux|arm*-linux-gnueabi|arm-*-linux|*-angstrom-linux)

to

arm-linux|arm*-linux-gnueabi|arm-*-linux)

ARM Angstrom will be picked up by the arm*-linux-gnueabi rule when being built on ARM and will just work fine when non-ARM Angstrom is used to build.

As trunk will branch for the new release very soon I propose to only make the change to trunk and update the mainline Angstrom recipes to build the new release in due time, I'll get this committed shortly.

I also have patches to extend the ARM ASM support for ARM Angstrom builds but that is another patch.

John

comment:4 Changed 9 years ago by fingolfin

Summary: angstrom+non-arm and configureBUILD: angstrom+non-arm and configure

comment:5 Changed 9 years ago by fingolfin

Owner: changed from djwillis to jvprat
Resolution: fixed
Status: newclosed

comment:6 Changed 9 years ago by fingolfin

Seems jordi committed the fix proposed by djwillis in rev 48116 on Feb 27.

I assume that means the issue if fixed. If not, please reopen.

comment:7 Changed 9 years ago by jvprat

Just wanted to clarify it was djwillis who commited the fix himself ;)

comment:8 Changed 9 years ago by jvprat

Owner: changed from jvprat to djwillis

comment:9 Changed 7 months ago by digitall

Component: Ports
Note: See TracTickets for help on using tickets.