Skip to content
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

Improve error message for unsatisfiable package constraints (issue #2643) #2727

Merged
merged 2 commits into from Jul 31, 2015

Conversation

grayjay
Copy link
Collaborator

@grayjay grayjay commented Jul 18, 2015

This commit adds the sources of constraints to debugging and error messages,
e.g., "main config file" or "command line flag".

@kosmikus
Copy link
Contributor

I'm happy with the changes. Unless anyone objects, I'll merge this soon. Thanks again.

@grayjay
Copy link
Collaborator Author

grayjay commented Jul 24, 2015

@kosmikus Thanks for the help!

@dcoutts
Copy link
Contributor

dcoutts commented Jul 25, 2015

@kosmikus if you're happy with it then please go ahead.

@grayjay thanks for all the hard work!

@23Skidoo
Copy link
Member

@grayjay Can you please rebase your branch against master?

…skell#2643)

This commit adds the sources of constraints to debugging and error messages,
e.g., "main config file" or "command line flag".
@grayjay
Copy link
Collaborator Author

grayjay commented Jul 31, 2015

I rebased.

@23Skidoo
Copy link
Member

@kosmikus seems to be happy with this, so merging.

@grayraw Thanks for your contribution!

23Skidoo added a commit that referenced this pull request Jul 31, 2015
Improve error message for unsatisfiable package constraints (issue #2643)
@23Skidoo 23Skidoo merged commit fe71660 into haskell:master Jul 31, 2015
@kosmikus
Copy link
Contributor

@23Skidoo Thanks a lot for doing the merge.

@grayjay
Copy link
Collaborator Author

grayjay commented Jul 31, 2015

Thanks for merging!

@grayjay grayjay deleted the track-package-constraints branch July 31, 2015 06:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants