Remove "-any" from AnyVersion pretty-print output.

[m-renaud]

Behaviour Change

Old behaviour:

λ> pretty AnyVersion
-any

New behaviour:

λ> pretty AnyVersion

Testing

cabal run hackage-tests roundtrip

There is one failure in the Workflow package version 0.7.0.0 which uses the -any syntax (see Workflow.cabal) in addition to a version bound check: TCache -any && <1.0. Is this even valid? It would be equivalent to TCache <1.0.

TCache -any && <1.0

TCache && <1.0
       ^ Parse error

How would we go about fixing this @phadej ?

Also, let me know if there's other places you'd like to see updated, I just updates the pretty function as suggested in the other PR.

TODO

• Fix lib-tests
• Update changelog
• Update docs

---

Please include the following checklist in your PR:

• Patches conform to the coding conventions.
• Any changes that could be relevant to users have been recorded in the changelog (add file to changelog.d directory).
• The documentation has been updated, if necessary.

Please also shortly describe how you tested your change. Bonus points for added tests!

[m-renaud]

Alright, @phadej as expected the round-trip property tests fail since print . parse "-any" does not equal "-any". Looking at the parsing code its not clear exactly how a lack of version range gets turned into the AnyVersion constructor, do you know how that happens? Is there a different parser for the version range other than what is in Cabal/Distribution/Types/VersionRange/Internal.hsCabal/Distribution/Types/VersionRange/Internal.hs ? The only place AnyVersion is constructed in the parser is here 😕

[phadej]

See #6742

[phadej]

This is wrong. The Dependency output have to be modified.

See what's done in #6742. That PR is not complete, I run out of time on Friday, so didn't put it up. Sorry for that. I changed the Dependency output as I was updating parser-tests outputs anyway.