NixOS / nixpkgs

Nix Packages collection & NixOS
MIT License
17.99k stars 14.01k forks source link

jl: build failure due to ghc 9 #168256

Closed schuelermine closed 2 years ago

schuelermine commented 2 years ago

Describe the bug

builder for '/nix/store/rhspfhlw8414pfy394l77v9mp7fyqahv-jl-0.0.5.drv' failed with exit code 1;
last 10 log lines:
>    | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
> [7 of 9] Compiling JL.Inferer       ( src/JL/Inferer.hs, dist/build/JL/Inferer.o, dist/build/JL/Inferer.dyn_o )
>
> src/JL/Inferer.hs:12:1: warning: [-Wunused-imports]
>     The import of ‘Control.Monad.Writer’ is redundant
>       except perhaps to import instances from ‘Control.Monad.Writer’
>     To import instances alone, use: import Control.Monad.Writer()
>    |
> 12 | import           Control.Monad.Writer
>    | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
For full logs, run 'nix log /nix/store/rhspfhlw8414pfy394l77v9mp7fyqahv-jl-0.0.5.drv'.

Steps To Reproduce

Steps to reproduce the behavior:

  1. Build pkgs.jl

Notify maintainers

@fgaz

Metadata

Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.

[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.17.1, NixOS, 22.05 (Quokka), 22.05.20220403.bc4b9ee`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.8.0pre20220322_d5d4d98`
 - channels(root): `"nixos, nixpkgs"`
 - channels(anselmschueler): `""`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`
iblech commented 2 years ago

Thank you for reporting this issue; I can reproduce it. The problem is not in the warning displayed in the last 10 log lines, but in an earlier error:

[4 of 9] Compiling JL.Serializer    ( src/JL/Serializer.hs, dist/build/JL/Serializer.o, dist/build/JL/Se
rializer.dyn_o )

src/JL/Serializer.hs:39:30: error:
    • Couldn't match type ‘Data.HashMap.Internal.HashMap Text’
                     with ‘Data.Aeson.KeyMap.KeyMap’
      Expected: Object
        Actual: Data.HashMap.Internal.HashMap Text Value
    • In the first argument of ‘Object’, namely ‘(fmap coreToValue hm)’
      In the expression: Object (fmap coreToValue hm)
      In a case alternative:
          RecordCore hm -> Object (fmap coreToValue hm)
   |     
39 |     RecordCore hm -> Object (fmap coreToValue hm)
   |                              ^^^^^^^^^^^^^^^^^^^

Unfortunately I don't think that I will be able to contribute further to this issue.

fgaz commented 2 years ago

I'm aware of this failure, but fixing it will require a bit of patching. Upstream didn't update the package to ghc 9 yet. Right now I don't have much time to do that, so in the meantime you could install it from stable, where it does build.

Please change the title to something like "jl: build failure due to ghc 9"

cdepillabout commented 2 years ago

This has been correctly marked broken in 3e6d4c0.

See also https://github.com/NixOS/nixpkgs/pull/168049 where there was a small amount of discussion about marking this broken.

neilmayhew commented 2 years ago

I've created an upstream PR.

neilmayhew commented 2 years ago

The PR is now merged, and the package is now on Hackage for the first time. However, since it's not been added to Stackage Nightly yet, it won't make its way into hackagePackages for quite some time. I'll update the derivation to use the new version directly from Hackage.