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

Cabal release for GHC 8.4 #5075

Closed
bgamari opened this issue Jan 29, 2018 · 24 comments
Closed

Cabal release for GHC 8.4 #5075

bgamari opened this issue Jan 29, 2018 · 24 comments
Assignees
Labels
Milestone

Comments

@bgamari
Copy link
Contributor

bgamari commented Jan 29, 2018

GHC 8.4 is quickly approaching. What is the status here?

@phadej
Copy link
Collaborator

phadej commented Jan 29, 2018

I need to fix a bug in #5055, will do that tomorrow.

@23Skidoo
Copy link
Member

I'll create a branch once that is merged.

@angerman
Copy link
Collaborator

We need #4874 for hadrian (reloc).

@angerman
Copy link
Collaborator

There should be no release before #5053 is merged.

@phadej
Copy link
Collaborator

phadej commented Jan 30, 2018

@phadej
Copy link
Collaborator

phadej commented Jan 30, 2018

@bgamari is https://ghc.haskell.org/trac/ghc/wiki/Commentary/Libraries/VersionHistory correct, time will stay at 1.8.0.2. @hvr when parsec-3.1.12 will be released? (Do we want to have >=3.1.12 lower bound?)

@Ericson2314
Copy link
Collaborator

Hoping to land #4383, so we will only have 2.0 allowing ambiguous libraries

@phadej
Copy link
Collaborator

phadej commented Feb 5, 2018

AFAICS everything is in, can we cut the 2.2 branch?

@angerman
Copy link
Collaborator

angerman commented Feb 5, 2018

#4874

@phadej
Copy link
Collaborator

phadej commented Feb 5, 2018

@angerman hopefully it will be merged by the time @23Skidoo notices my message! :)

@angerman
Copy link
Collaborator

angerman commented Feb 5, 2018

Yea, I'm running some final hadrian tests, and will fix #5111 for inclusion (which unbreaks #4892, which broken when trying to get the flags to hide... for some reason my cross validation ran against some stale code :( -- it's now only blocked by the use of LambdaCase, I believe; will fix.)

@Ericson2314
Copy link
Collaborator

Ooo I hope I make it. #5100 also should be finished as we've got spurious warnings right now without it.

@phadej
Copy link
Collaborator

phadej commented Feb 5, 2018

No please, no #5100. Or any other non-trivial patches.

  • Fixing bugs 👍
  • Writing docs 👍
  • Improving error messages: 👍
  • Breaking API 👎

The branch had to be cut for stabilization already 20 days ago (Jan 15th). The feature creep was enormous, let's not make it any worse.

ping @23Skidoo

@23Skidoo
Copy link
Member

23Skidoo commented Feb 5, 2018

+1 on not including #5100 at this stage. I'll cut the branch later today or tomorrow, the remaining fixes will be cherry-picked.

@Ericson2314
Copy link
Collaborator

The branch had to be cut for stabilization already 20 days ago

A cabal branch or GHC branch?

@23Skidoo
Copy link
Member

23Skidoo commented Feb 5, 2018

Cabal branch.

@Ericson2314
Copy link
Collaborator

Understood now, talking to @phadej on IRC. The plan was for Jan 15, but it has already slipped. Never mind me then.

@23Skidoo
Copy link
Member

23Skidoo commented Feb 5, 2018

Yep, there was an announcement on cabal-devel: https://mail.haskell.org/pipermail/cabal-devel/2017-December/010410.html.

@angerman
Copy link
Collaborator

#4874 is now green as well (after we gave travis a chance to turn green again ;-))

@23Skidoo
Copy link
Member

23Skidoo commented Feb 14, 2018

Created the 2.2 branch.

@bgamari
Copy link
Contributor Author

bgamari commented Mar 2, 2018

As documented in #14862, the HEAD of Cabal 2.2 breaks the ghc-8.4. @angerman has put quite a bit of time into trying to identify the culprit but there is currently no solution. As the release is supposed to go out tomorrow, I think we need to start looking for alternatives.

The last known working commit is d2cf3f1. I recommend that this commit be designated 2.2.0.0 so we can release GHC in the same state that the release candidate was cut in. It's hard to tell precisely what fixes we will miss out on by doing this, but a quick glance suggests that it's nothing particularly awful.

@23Skidoo
Copy link
Member

23Skidoo commented Mar 2, 2018

If we're sure that the basedir patch is the source of the problems, I can just revert it.

@hvr
Copy link
Member

hvr commented Mar 2, 2018

I'd like to look into the windows issue; this must be something simple... it's really just about a backslash getting eaten somewhere (the trouble was afaik that @angerman had no windows build env to try/debug it interactively)

@quasicomputational
Copy link
Contributor

Cabal 2.2 has been released, the linked bugs have been resolved, and there aren't howls of pain about brokenness, so I guess this issue's served its purpose.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

7 participants