[Bug 503185] Re: fails to build on armel/lucid

[Dave Martin]

alan woodland wrote (19 January 2010 09:55):

> I think there might be a problem in that patch still...
> 
> On the Debian machine it built on last night __ARM_ARCH_4__
> was not defined, __ARM_ARCH_4T__ was defined, and __thumb__
> wasn't defined.
> This meant that it ended up using the ARMv5 of the #else,
> which used the blx instruction and failed to build because of
> it.
> 
> I think changing it to read:
> #if defined(__ARM_ARCH_4__) || defined(__ARM_ARCH_4T__)
> should fix things. Does that seem sane to you?
> 
> Thanks,
> Alan

Yes, that makes sense.

A modified, slightly tidied-up patch which I prepared yesterday is
attached here.  For some reason, I thought I'd already uploaded it for
some reason, but it looks I didn't...

** Attachment added: "Updated patch which copes with -march=armv4t"
   http://launchpadlibrarian.net/38079473/cr_atomic_arm.diff

-- 
fails to build on armel/lucid
https://bugs.launchpad.net/bugs/503185
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.

-- 
ubuntu-bugs mailing list
ubuntu-bugs at lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs