Skip to content

treewide: Raise bounds on optparse-applicative#2543

Merged
mergify[bot] merged 1 commit intodhall-lang:mainfrom
JackKelly-Bellroy:optparse-applicative-0.18
Oct 20, 2023
Merged

treewide: Raise bounds on optparse-applicative#2543
mergify[bot] merged 1 commit intodhall-lang:mainfrom
JackKelly-Bellroy:optparse-applicative-0.18

Conversation

@JackKelly-Bellroy
Copy link
Contributor

optparse-applicative-0.18.0.0 is a deprecated release which used layoutSmart from package prettyprinter, which causes the runtime to blow up in certain cases.

See: pcapriotti/optparse-applicative#475 (comment)

Tested with GHC 8.10.7 using cabal test all --constraint 'optparse-applicative == 0.18.1.0' --allow-newer=hnix:optparse-applicative --allow-newer=dhall-nixpkgs:turtle --keep-going. Doctests are failing, I'm not sure why. They also fail for me on main.

@Gabriella439
Copy link
Collaborator

Gabriella439 commented Oct 20, 2023

Usually I don't explicitly blacklist deprecated releases via bounds and instead rely on the fact that Cabal's solver will avoid deprecated releases

@JackKelly-Bellroy
Copy link
Contributor Author

Cleaned up the bounds and force-pushed.

@Gabriella439 Gabriella439 enabled auto-merge (squash) October 20, 2023 00:52
@Gabriella439
Copy link
Collaborator

This looks great! Thank you 😊

@Gabriella439 Gabriella439 disabled auto-merge October 20, 2023 01:07
@mergify mergify bot merged commit c566f30 into dhall-lang:main Oct 20, 2023
@lrworth
Copy link
Contributor

lrworth commented Dec 14, 2023

I'd appreciate an analogous hackage revision for dhall-json.

@Gabriella439
Copy link
Collaborator

I published matching revisions for all the remaining packages (including dhall-json)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants