-
Notifications
You must be signed in to change notification settings - Fork 37
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
Relax lens lower bound #23
Comments
Thanks for reporting this issue! Sure thing, I'll make a revision on hackage and reflect it in the repo later this morning (got to run right now). |
I wonder how nixpkgs is broken. Also note that latest |
@endgame Hmm I just checked and the latest lens release is 4.17, which is included in both the previous version range and the one for servant-multipart-0.11.3, so I'm not sure exactly what the new bounds should be, and what the problem is. Could you show us the error message? |
The version of lens currently in nixpkgs on my GNU/Linux and macOS/darwin machines both have lens-4.16.1. Also, is servant-multipart using 0.15 specific features from {servant, servant-foreign, servant-server}? Those are also not currently in nixpkgs. nixpkgs master has lens-4.17, so this is not especially pressing, but it would be nice to have since it's not using recent lens features. |
Aaaaah, so we're requiring too recent a lens version... |
Yes. From memory you're using |
Note to self: I'm really confused. Why you can upgrade to |
At the time I reported this, nixpkgs on darwin had lens-4.16 and servant-multipart-0.11.3, and I was chasing build failures. nixpkgs on my GNU/Linux machine is still giving me lens-4.16 but servant-multipart-0.11.2, so it builds. |
servant-multipart-0.11.3 depends on
lens >= 4.17 && < 4.18
, while the previous version depended onlens >= 4.0 && < 4.18
.servant-multipart uses four functions from
Control.Lens
, and looking at the lens changelog none of them seem to have recent changes to behaviour. Can the lower bound please be relaxed, and a metadata revision pushed to hackage?(Among other things, this is currently breaking installing servant-multipart via nixpkgs on darwin.)
The text was updated successfully, but these errors were encountered: