Bug#864974: thunderbird: Missing AtomicOperations for multiple architectures cause FTBFS

2017-06-21 Thread 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

Bug#864974: thunderbird: Missing AtomicOperations for multiple architectures cause FTBFS

2017-06-21 Thread Carsten Schoenert
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

Bug#864974: thunderbird: Missing AtomicOperations for multiple architectures cause FTBFS

2017-06-18 Thread John Paul Adrian Glaubitz
On 06/18/2017 10:48 AM, John Paul Adrian Glaubitz wrote: > On 06/18/2017 10:40 AM, John Paul Adrian Glaubitz wrote: >> powerpc and powerpcspe: __ppc__ > > Correction: powerpc and powerpcspe should actually be covered: Hmm, looks like I was mislead by the gitweb view. Looking at the source,

Bug#864974: thunderbird: Missing AtomicOperations for multiple architectures cause FTBFS

2017-06-18 Thread John Paul Adrian Glaubitz
On 06/18/2017 10:40 AM, John Paul Adrian Glaubitz wrote: > powerpc and powerpcspe: __ppc__ Correction: powerpc and powerpcspe should actually be covered: #elif defined(__ppc__) || defined(__PPC__) # include "jit/none/AtomicOperations-ppc.h" Not sure why the JavaScript engine crashes here

Bug#864974: thunderbird: Missing AtomicOperations for multiple architectures cause FTBFS

2017-06-18 Thread John Paul Adrian Glaubitz
Source: icedove Version: 1:52.2.0-1 Severity: serious Justification: fails to build from source Hi! thunderbird fails to build from source on multiple architectures, including s390x, because the proper AtomicOperations header is not included for the affected architectures. Looking at [1], all