Re: 01/03: llvm: Build with RTTI by default.

[Ludovic Courtès]

Hello,

Mark H Weaver <address@hidden> skribis:

> address@hidden (Eric Bavier) writes:
>
>> bavier pushed a commit to branch master
>> in repository guix.
>>
>> commit fc9dbf41311d99d0fd8befc789ea7c0e35911890
>> Author: Eric Bavier <address@hidden>
>> Date:   Fri Oct 5 22:58:43 2018 -0500
>>
>>     llvm: Build with RTTI by default.
>>     
>>     Increases size of address@hidden by 2.5MiB (2.5%), but saves building 
>> specialized
>>     llvm's with rtti enabled for packages that require RTTI when linking 
>> with the
>>     llvm libraries.
>>     
>>     * gnu/packages/llvm.scm (llvm-3.8-with-rtti): Remove variable.
>>     (llvm)[arguments]: Add '-DLLVM_REQUIRES_RTTI=1' to #:configure-flags.
>>     * gnu/packages/audio.scm (faust-2)[native-inputs]:
>>     'llvm-3.8-with-rtti' -> 'llvm-3.8'.
>
> Unfortunately, this seems to have broken Rust, and thus IceCat.  See:
>
>   https://hydra.gnu.org/build/3179552
>
> I don't have time to look at this right now, but please also keep in
> mind the following commit:
>
>   commit f8cba3ff4fc6aed0c539700aa47b6e8f4c25c34d
>   Author: Marius Bakke <address@hidden>
>   Date:   Sat Nov 17 22:54:03 2018 +0100
>   
>       gnu: mesa: Stay on RTTI-less LLVM.
>       
>       * gnu/packages/llvm.scm (llvm-without-rtti): New public variable.
>       * gnu/packages/gl.scm (mesa)[inputs]: Use that instead of LLVM.
>
> I would suggest simply reverting both of these commits for now, and
> building this change out on another branch, possibly 'core-updates'.

This commit also triggered more than 1K rebuilds per architecture, so
Marius “semi-reverted” it in f8cba3ff4fc6aed0c539700aa47b6e8f4c25c34d
after we discussed it on IRC.

Ludo’.