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

Solver "rejecting" message is too verbose #4251

Open
ezyang opened this issue Jan 20, 2017 · 2 comments · May be fixed by #10854
Open

Solver "rejecting" message is too verbose #4251

ezyang opened this issue Jan 20, 2017 · 2 comments · May be fixed by #10854

Comments

@ezyang
Copy link
Contributor

ezyang commented Jan 20, 2017

I recently ran into a solver error which was due to my index not being sufficiently up to date (I had updated a Git checkout which specified a bound requiring a version of a package which did not exist at all). Here's the message I got:

cabal: Could not resolve dependencies:
trying: cabal-testsuite-1.25.0.0 (user goal)
next goal: aeson (dependency of cabal-testsuite-1.25.0.0)
rejecting: aeson-1.0.2.1, aeson-1.0.2.0, aeson-1.0.1.0, aeson-1.0.0.0,
aeson-0.11.2.1, aeson-0.11.2.0, aeson-0.11.1.4, aeson-0.11.1.3,
aeson-0.11.1.2, aeson-0.11.1.1, aeson-0.11.1.0, aeson-0.11.0.0, aeson-0.9.0.1,
aeson-0.9.0.0, aeson-0.8.1.1, aeson-0.8.1.0, aeson-0.8.0.2, aeson-0.7.0.6,
aeson-0.7.0.4, aeson-0.6.2.1, aeson-0.6.2.0, aeson-0.6.1.0, aeson-0.6.0.2,
aeson-0.6.0.1, aeson-0.6.0.0, aeson-0.5.0.0, aeson-0.4.0.1, aeson-0.4.0.0,
aeson-0.3.2.14, aeson-0.3.2.13, aeson-0.3.2.12, aeson-0.3.2.11,
aeson-0.3.2.10, aeson-0.3.2.9, aeson-0.3.2.8, aeson-0.3.2.7, aeson-0.3.2.6,
aeson-0.3.2.5, aeson-0.3.2.4, aeson-0.3.2.3, aeson-0.3.2.2, aeson-0.3.2.1,
aeson-0.3.2.0, aeson-0.3.1.1, aeson-0.3.1.0, aeson-0.3.0.0, aeson-0.2.0.0,
aeson-0.1.0.0, aeson-0.10.0.0, aeson-0.8.0.1, aeson-0.8.0.0, aeson-0.7.0.5,
aeson-0.7.0.3, aeson-0.7.0.2, aeson-0.7.0.1, aeson-0.7.0.0 (conflict:
cabal-testsuite => aeson==1.1.*)
After searching the rest of the dependency tree exhaustively, these were the
goals I've had most trouble fulfilling: cabal-testsuite (65), aeson (57)

In terms of "information communicated to me", this is totally backwards. Here are the important pieces of information I want to know:

  • What my constraint is, and where it came from (cabal-testsuite => aeson==1.1.*)
  • That there doesn't exist any version of aeson which fulfills this constraint. I don't need a list of every version in the index
@23Skidoo
Copy link
Member

/cc @grayjay @kosmikus

@grayjay
Copy link
Collaborator

grayjay commented Feb 26, 2017

This issue is related to #4328. I think we should continue printing all rejected versions in the -v3 log but try to summarize the message further in the short log. We could also improve the -v3 log by only printing "aeson" once.

yvan-sraka added a commit to yvan-sraka/cabal that referenced this issue Aug 3, 2023
yvan-sraka added a commit to yvan-sraka/cabal that referenced this issue Aug 29, 2023
This commit makes the following changes:

- Enhancements to the codebase:
  * Adds several TODO/FIXME/HELP comments, laying the groundwork for
    improved solver output as described in RFC 4251 ;
  * Refactors the `showMessages` function to split the logic of
    building the output (now as a `Message'` enumeration) and the
    string representation of it (now formatted by `displayMessage'`).

- Modifications to the solver output:
  * If the `-v3` or `--minimize-conflict-set` flags are not set, it
    now prompts the user to consider using them in cases of the
    "Could not resolve dependencies: ..." error ;
  * The message "(has the same characteristics that caused the
    previous version to fail: ...)" has been rephrased to
    "all other available packages. They are excluded due to the same
    constraint that caused the last version attempted to fail: ..." ;
  * Package lists are now grouped by name. For example, instead of
    displaying `aeson-1.0.2.1, aeson-1.0.2.0, aeson-1.0.1.0, ...`, it
    now shows `aeson: 1.0.2.1, 1.0.2.0, 1.0.1.0, ...`.
yvan-sraka added a commit to yvan-sraka/cabal that referenced this issue Dec 19, 2023
yvan-sraka added a commit to yvan-sraka/cabal that referenced this issue Dec 19, 2023
erikd added a commit to erikd/cabal that referenced this issue Mar 26, 2025
Includes:

* Apply some of @grayjay and @mpickering comments
* Fix haskell#4251

Co-Authored-By: Erik de Castro Lopo <[email protected]>
@erikd erikd linked a pull request Mar 26, 2025 that will close this issue
6 tasks
erikd added a commit to erikd/cabal that referenced this issue Mar 26, 2025
Includes:

* Apply some of @grayjay and @mpickering comments
* Fix haskell#4251

Co-Authored-By: Erik de Castro Lopo <[email protected]>
erikd added a commit to erikd/cabal that referenced this issue Mar 26, 2025
Includes:

* Apply some of @grayjay and @mpickering comments
* Fix haskell#4251

Co-Authored-By: Erik de Castro Lopo <[email protected]>
erikd added a commit to erikd/cabal that referenced this issue Mar 26, 2025
Includes:

* Apply some of @grayjay and @mpickering comments
* Fix haskell#4251

Co-Authored-By: Erik de Castro Lopo <[email protected]>
erikd added a commit to erikd/cabal that referenced this issue Mar 26, 2025
Includes:

* Apply some of @grayjay and @mpickering comments
* Fix haskell#4251

Co-Authored-By: Erik de Castro Lopo <[email protected]>
erikd added a commit to erikd/cabal that referenced this issue Mar 26, 2025
Includes:

* Apply some of @grayjay and @mpickering comments
* Fix haskell#4251

Co-Authored-By: Erik de Castro Lopo <[email protected]>
erikd added a commit to erikd/cabal that referenced this issue Mar 26, 2025
Includes:

* Apply some of @grayjay and @mpickering comments
* Fix haskell#4251

Co-Authored-By: Erik de Castro Lopo <[email protected]>
erikd added a commit to erikd/cabal that referenced this issue Mar 26, 2025
Includes:

* Apply some of @grayjay and @mpickering comments
* Fix haskell#4251

Co-Authored-By: Erik de Castro Lopo <[email protected]>
erikd added a commit to erikd/cabal that referenced this issue Mar 26, 2025
Includes:

* Apply some of @grayjay and @mpickering comments
* Fix haskell#4251

Co-Authored-By: Erik de Castro Lopo <[email protected]>
erikd added a commit to erikd/cabal that referenced this issue Mar 26, 2025
Includes:

* Apply some of @grayjay and @mpickering comments
* Fix haskell#4251

Co-Authored-By: Erik de Castro Lopo <[email protected]>
erikd added a commit to erikd/cabal that referenced this issue Mar 26, 2025
Includes:

* Apply some of @grayjay and @mpickering comments
* Fix haskell#4251

Co-Authored-By: Erik de Castro Lopo <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants