[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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

From: Mark H Weaver
Subject: Re: 01/03: llvm: Build with RTTI by default.
Date: Sun, 18 Nov 2018 06:06:42 -0500
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux)

Hi Eric,

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:

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'.

IceCat should not be left broken longer than necessary.


reply via email to

[Prev in Thread] Current Thread [Next in Thread]