Bug#650151: FTBFS on Alpha: Some tweaks needed

[Michael Cree]

On 27/11/11 22:20, Mike Hommey wrote:
> On Sun, Nov 27, 2011 at 01:52:09PM +1300, Michael Cree wrote:
>> Source: iceweasel
>> Version: 8.0-3
>> 
>> The following tweaks are all that are needed to build a working iceweasel.
>>
>> Please add the following to the patch
>> "Add-mips-hppa-ia64-s390-and-sparc-defines-in-ipc-chr.patch":
>>
>>  #elif defined(__s390__)
>>  #define ARCH_CPU_S390 1
>>  #define ARCH_CPU_32_BITS 1
>> +#elif defined(__alpha__)
>> +#define ARCH_CPU_ALPHA 1
>> +#define ARCH_CPU_64_BITS 1
>>  #else
>>  #error Please add support for your architecture in build/build_config.h
>>  #endif
>>
>> There remains a bug in the linker (binutils) that is causing problems to
>> large C++ projects and to get a working iceweasel full optimisation
>> needs to be disabled.  Please add into debian/rules the following:
>>
>> ifneq (,$(filter alpha,$(DEB_BUILD_ARCH)))
>>         CONFIGURE_FLAGS += --enable-optimize=-O1
>> endif
> 
> Seriously, I doubt this is all that is needed. I would be surprised if
> the resulting binary actually works and passes the test suites.

Actually things have just got better since we are now defaulting to
gcc-4.6 on Alpha.  The crash of iceweasel at invocation, and also
crashes in icedove, disappear when they are built with gcc-4.6, so we no
longer need the use of -O1 optimisation to get a working iceweasel.
Without -O1, though, we get exposed to the binutils bug that results in
GPREL16 relocation errors during linking in very large C++ projects so
we need to link with -Wl,--no-relax.

In summary, then, please add the IPC defines quoted above, and
-Wl,--no-relax to MAIN_LDFLAGS when building on Alpha.

Thanks,
Michael.