[Debian-ports-devel] Bug#864974: thunderbird: Missing AtomicOperations for multiple architectures cause FTBFS

[John Paul Adrian Glaubitz]

Hi Carsten!

The architectures in question don’t use the JIT, that’s why they can all use a common header for the atomics code.

Regarding the non-JIT architectures, there was only one change and that was the renaming of the generic headers from -sparc.h and -ppc.h to -feeling-lucky.h.

All the non-JIT archs can use the -feeling-lucky.h header now which is identical to -sparc.h and -ppc.h.

Adrian

> On Jun 21, 2017, at 7:30 PM, Carsten Schoenert <c.schoenert at t-online.de> wrote:
> 
> Hello Adrian,
> 
>> On Mon, Jun 19, 2017 at 12:44:49AM +0200, John Paul Adrian Glaubitz wrote:
>> ... 
>> Hmm, looks like I was mislead by the gitweb view. Looking at the
>> source, AtomicOperations-ppc.h is included for everything but
>> s390x.
>> 
>> So, it's safe to assume it FTBFS on s390x because of the missing
>> atomic operations. No idea about the other architectures though,
>> it should definitely work.
> 
> correct, I've no idea why all these non-rc architectures currently
> failing too.
> 
> You pointed to a AtomicOperations.h that is based on Mozilla 55 with
> some dramatically changes within the JIT stuff against 52.2.0 so I was
> conservative and only picked a modification from Mike [1] for s390x that
> should work for now.
> 
> For more like pulling those changes to the JIT compiler the needed
> adjustments to the patch queue afterwards we haven't enough resources
> now.
> 
> The Thunderbird guys preparing a small fixup release 52.2.1 an we will
> probably delay a new upload until they release this version.
> 
> [1] https://anonscm.debian.org/cgit/pkg-mozilla/iceweasel.git/commit/?h=esr52/master&id=9d8d85289512755140a8b0d69467d58ab8b677fe
> 
> Regards
> Carsten