elmPackages.elm: don't check aeson#378618
elmPackages.elm: don't check aeson#378618mpscholten wants to merge 1 commit intoNixOS:haskell-updatesfrom
Conversation
Checking aeson fails as it depends on nothunks < 0.3, but we provide nothunks >= 0.3. Jailbreaking doesn't work in this context, so I disabled the tests instead.
|
So this seems to be because of this: https://github.com/haskell/aeson/blob/85c2da59d68953e48720e18f632689f385237e58/aeson.cabal#L242-L243 I think what we should do is to add the missing nothunks dependency (in the correct version) to In any case, this same problem appears in WDYT? |
|
Running the test suite would be ideal, but I suspect providing nothunks in the correct versions would mean downgrading it globally since e.g. unordered-containers depends on nothunks which in turn is needed by aeson. I'm not sure what cabal will do actually if we disable the inconsistent dependency check when building a test suite with such a package… |
|
Done in 1ee8e4b. |
Checking aeson fails as it depends on nothunks < 0.3, but we provide nothunks >= 0.3. Jailbreaking doesn't work in this context, so I disabled the tests instead.
I've also created an upstream fix via haskell/aeson#1129
Things done
nix.conf? (See Nix manual)sandbox = relaxedsandbox = truenix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/)Add a 👍 reaction to pull requests you find important.