-
-
Notifications
You must be signed in to change notification settings - Fork 14.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
haskellPackages.cabal-plan-bounds: Unbreak, add myself as maintainer #242609
Conversation
#240387 updates the default GHC to 9.4 which uses 3.8 by default. Can you rebase your PR on |
Indeed, not with cabal-plan-bounds, but its dependency I tried to see if jailbreaking helps, but oddly, this doesn’t work:
I still get
I wonder what I am doing wrong. |
jailbreak-cabal intentionally leaves bounds inside conditionals alone in order not to mess with Cabal's flag solver. I guess base-compat is in a conditonal? In that case you need to patch the cabal file (for now at least, until haskell/cabal#9016 is done). |
Yes, that’s it. Annoying. |
Looks like there is a new revision of |
You can pick new revision early like this or, of course, wait for the next hackage bump. overrideCabal (drv: {
revision = assert drv.revision == "1"; "2";
editedCabalFile = "sha256-…";
}) |
That was my plan :-D |
Just had a look, the latest revision is in, but
Not sure what to do about it. |
I just noticed The build failure linked in the comment (https://hydra.nixos.org/build/230967016) is not what's currently failing it, but the wrong base-compat version:
Is there any plan on how to get this fixed? I assume |
That is not very surprising since the package or its codependencies currently have no maintainer.
Yes, we will probably stick with 9.4 until at least 23.11. Since cabal-plan-bounds is a leave package I think an overrideScope might work. |
In that specific instance, it's |
I think it is probably easiest and most worthwhile making cabal-plan compatible with Stackage LTS 21 upstream which would help everyone out. |
Using cabal-plan 0.7.2.3 instead of 0.7.3.0 might be lts-compatible |
New attempt at #261261 |
Description of changes
The package could easily be unbroken by passing a new version of Cabal-syntax.
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)