-
-
Notifications
You must be signed in to change notification settings - Fork 11
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
Fails to release bounds on conditional build-depends #15
Comments
It seems that this was previously discussed in #7. |
Yes, this is a known limitation. |
peti
pushed a commit
to NixOS/nixpkgs
that referenced
this issue
Feb 28, 2020
If you actually look at the changelog for 1.4.2.1, you'll see that it mentions that it was only intended for ghc-7.0.4, which is why it has a dependency on an inconveniently old version of `unix`, which is *also* hidden behind a flag, so that we can't jailbreak it (per NixOS/jailbreak-cabal#15). So the right thing to do would appear to me to be to override our default version to 1.4.2.0, until such time as the maintainer makes a newer release. `ConfigFile` was collateral damage, so unmark it broken, as well as `MissingH` itself.
peti
pushed a commit
to NixOS/nixpkgs
that referenced
this issue
Feb 28, 2020
If you actually look at the changelog for 1.4.2.1, you'll see that it mentions that it was only intended for ghc-7.0.4, which is why it has a dependency on an inconveniently old version of `unix`, which is *also* hidden behind a flag, so that we can't jailbreak it (per NixOS/jailbreak-cabal#15). So the right thing to do would appear to me to be to override our default version to 1.4.2.0, until such time as the maintainer makes a newer release. `ConfigFile` was collateral damage, so unmark it broken, as well as `MissingH` itself.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Consider a case like
comonad
's cabal file:When
jailbreak-cabal
is run on this I see the following result:Here we see that
jailbreak-cabal
did not modify the conditionalbuild-depends
.The text was updated successfully, but these errors were encountered: