-
Notifications
You must be signed in to change notification settings - Fork 712
Rebase fixLinkDeps onto solver branch #3360
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
Conversation
@grayjay I'm somewhat confused. What is the status on #3327 now? Do we have a solution? Does the |
Yes, I think the answer to that isn't clear yet, but it would be good if we have have the test cases you have for #3327 available on the |
I cherry-picked that test case, but I think it is currently passing because of a different fix. |
Aha, I see. That explains my confusion :) That's a little unfortunate. |
Yeah, there is a lot of overlap between conflict sets. I'll try to come up with a more specific test. |
I added a test in #3363. |
All but the last two commits are from #3359.
See #3357. I rebased https://github.com/edsko/cabal/tree/pr/FixLinkDeps, except for the last commit, which handles a different issue. I also removed my commits, because they weren't needed.
/cc @edsko