Re: gnu: ghc-8: Update to 8.0.2

[Federico Beffa]

On Sat, Jan 28, 2017 at 12:35 AM, Ludovic Courtès <address@hidden> wrote:
> Hi Federico,
>
> Federico Beffa <address@hidden> skribis:
>
>> I've updated our ghc-8 Haskell compiler to the latest version (8.0.2)
>> and in doing so I've investigated why both 8.0.1 and 8.0.2 do not work
>> without setting LD_LIBRARY_PATH to the required system libraries.
>> I've found that NIX had the same problem.  The root cause for them was
>> that their gcc-wrapper was not able to properly handle arguments
>> passed through response files.  I suspect that we have the same
>> problem.
>
> We do.  (This and more was discussed at
> <https://gcc.gnu.org/ml/gcc/2016-10/threads.html#00122>.)
>
> But really, “response files” are relics.  OTOH, it may not be hard to
> change ‘ld-wrapper’ to read them (and there’s a good opportunity now
> that we’re working on ‘core-updates’, if you want to give it a try.
> :-)).

Thanks for the pointer.  At the moment I do not have time to work on it.

>
>> For the moment I've borrowed a patch for GHC that they did use before
>> fixing the gcc-wrapper. Going forward it would probably be wise to fix
>> our gcc/ld wrapper as well.
>
> Sounds good.
>
>> From 35c4fa12cf2eb7316583b41c0c4e8b60f7a59bdc Mon Sep 17 00:00:00 2001
>> From: Federico Beffa <address@hidden>
>> Date: Wed, 25 Jan 2017 18:21:43 +0100
>> Subject: [PATCH] gnu: ghc-8: Update to 8.0.2.
>>
>> * gnu/packages/haskell.scm (ghc-8): Update to 8.0.2.
>
> Please mention the changes to ‘arguments’ (deleted phase, etc.)
>
>> * gnu/packages/patches/ghc-dont-pass-linker-flags-via-response-files.patch:
>>   New file.
>> * gnu/local.mk (dist_patch_DATA): Add it.
>
>
>> +++ 
>> b/gnu/packages/patches/ghc-dont-pass-linker-flags-via-response-files.patch
>> @@ -0,0 +1,24 @@
>> +See 
>> https://github.com/NixOS/nixpkgs/commit/a421e7bd4a28c69bded8b17888325e31554f61a1
>
> Maybe add “Don’t add linker flags via ‘response files’ since ld-wrapper
> doesn’t handle them.”
>
> Also: https://gcc.gnu.org/ml/gcc/2016-10/msg00151.html
>
> OK with these changes.

Done and pushed as commit d8e85b20325073d90cfaf3060889d59d91362deb.

Regards,
Fede