On Tue, Jun 06, 2017 at 10:34:11PM +0200, John Paul Adrian Glaubitz wrote:
> Updated patch attached.

Ok, the patch is fine now as-is.

Please note, Thunderbird will currently still FTBFS on sh4 with:

LDDIR>>/obj-thunderbird/dist/include/nspr 
-I/<<PKGBUILDDIR>>/obj-thunderbird/dist/include/nspr 
-I/<<PKGBUILDDIR>>/obj-thunderbird/dist/include/nss 
-I/<<PKGBUILDDIR>>/obj-thunderbird/dist/private/nss 
-I/<<PKGBUILDDIR>>/obj-thunderbird/dist/include/nspr 
-DNSS_ENABLE_TLS13_SHORT_HEADERS object.c
In file included from 
/<<PKGBUILDDIR>>/obj-thunderbird/dist/system_wrappers/sys/ucontext.h:3:0,
                 from /usr/include/signal.h:326,
                 from 
/<<PKGBUILDDIR>>/obj-thunderbird/dist/system_wrappers/signal.h:3,
                 from 
/<<PKGBUILDDIR>>/obj-thunderbird/dist/include/mozilla/Assertions.h:67,
                 from 
/<<PKGBUILDDIR>>/obj-thunderbird/dist/include/mozilla/Array.h:12,
                 from 
/<<PKGBUILDDIR>>/mozilla/js/src/jit/BacktrackingAllocator.h:10,
                 from 
/<<PKGBUILDDIR>>/mozilla/js/src/jit/BacktrackingAllocator.cpp:7,
                 from 
/<<PKGBUILDDIR>>/obj-thunderbird/js/src/Unified_cpp_js_src8.cpp:2:
/<<PKGBUILDDIR>>/mozilla/js/src/jit/BaselineCacheIR.cpp: In member function 
'bool BaselineCacheIRCompiler::emitLoadFixedSlotResult()':
/<<PKGBUILDDIR>>/mozilla/js/src/jit/BaselineCacheIR.cpp:857:55: error: 
reference to 'R0' is ambiguous
     masm.loadValue(BaseIndex(obj, scratch, TimesOne), R0);
                                                       ^
/usr/include/sh4-linux-gnu/sys/ucontext.h:43:3: note: candidates are: 
<anonymous enum> R0
   R0 = 0,
   ^~
/<<PKGBUILDDIR>>/mozilla/js/src/jit/none/SharedICRegisters-none.h:18:31: note:  
               constexpr const js::jit::ValueOperand js::jit::R0
 static constexpr ValueOperand R0 = JSReturnOperand;
                               ^
/<<PKGBUILDDIR>>/mozilla/js/src/jit/BaselineCacheIR.cpp: In member function 
'bool BaselineCacheIRCompiler::emitLoadDynamicSlotResult()':
/<<PKGBUILDDIR>>/mozilla/js/src/jit/BaselineCacheIR.cpp:871:55: error: 
reference to 'R0' is ambiguous
     masm.loadValue(BaseIndex(obj, scratch, TimesOne), R0);
                                                       ^
/usr/include/sh4-linux-gnu/sys/ucontext.h:43:3: note: candidates are: 
<anonymous enum> R0
   R0 = 0,
   ^~
/<<PKGBUILDDIR>>/mozilla/js/src/jit/none/SharedICRegisters-none.h:18:31: note:  
               constexpr const js::jit::ValueOperand js::jit::R0
 static constexpr ValueOperand R0 = JSReturnOperand;
                               ^
/<<PKGBUILDDIR>>/mozilla/js/src/jit/BaselineCacheIR.cpp: In member function 
'bool BaselineCacheIRCompiler::emitLoadUnboxedPropertyResult()':
/<<PKGBUILDDIR>>/mozilla/js/src/jit/BaselineCacheIR.cpp:886:76: error: 
reference to 'R0' is ambiguous
     masm.loadUnboxedProperty(BaseIndex(obj, scratch, TimesOne), fieldType, R0);
                                                                            ^
(...)

However, this is a known bug in glibc which has been fixed in glibc_2.25 which
won't unfortunately find its way into Debian before Stretch is released.

Adrian

-- 
 .''`.  John Paul Adrian Glaubitz
: :' :  Debian Developer - [email protected]
`. `'   Freie Universitaet Berlin - [email protected]
  `-    GPG: 62FF 8A75 84E0 2956 9546  0006 7426 3B37 F5B5 F913

Reply via email to