[Nix-dev] Re: [Nix-commits] SVN commit: nix - 19547 - eelco - nixpkgs/branches/stdenv-updates/pkgs/top-level

[Ludovic Courtès]

Hello,

Eelco Dolstra <e.dolstra at tudelft.nl>
writes:

> Lluís Batlle wrote:
>
>> 2010/1/19 Eelco Dolstra <e.dolstra at tudelft.nl>:
>>>   gcc44_real = lowPrio (wrapGCC (makeOverridable (import ../development/compilers/gcc-4.4) {
>>> -    inherit fetchurl stdenv texinfo gmp mpfr ppl cloogppl
>>> +    inherit fetchurl stdenv texinfo gmp mpfr /* ppl cloogppl */
>>>       gettext which noSysDirs;
>>>     profiledCompiler = true;
>>>   }));
>>> @@ -3704,10 +3704,9 @@
>>>
>>>   gmp = import ../development/libraries/gmp {
>>>     inherit stdenv fetchurl m4;
>>> +    cxx = false;
>>>   };
>>>
>>> -  # `gmpxx' used to mean "GMP with C++ bindings".  Now `gmp' has C++ bindings
>>> -  # by default, so that distinction is obsolete.
>>>   gmpxx = gmp;
>> 
>> If the gmp used in the gcc expression includes c++ bindings (so, a
>> reference to the other gcc libstdc++), we are not solving anything
>> disabling ppl or cloogppl. 
>
> Maybe you missed the "cxx = false" line :-)  I've verified that the
> bootstrap-tools no longer appear in the closure of stdenv-linux.
>
> But I do see now that the "gmpxx = gmp" line is no longer appropriate, will fix.

Can we have C++ bindings built in the general case, i.e., when outside
the bootstrap stdenv?  See [0] for a discussion.

Thanks,
Ludo’.

[0] http://thread.gmane.org/gmane.linux.distributions.nixos/2378