Skip to content

Fix GHC version bound in CPP #57

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

Merged
merged 1 commit into from
Aug 17, 2020

Conversation

sjakobi
Copy link
Member

@sjakobi sjakobi commented Jun 19, 2020

@chessai
Copy link
Member

chessai commented Jun 19, 2020

We shouldn't have CPP referring to unstable branches of GHC in the source. So you should do the thing Ben mentioned in the other PR.

@sjakobi
Copy link
Member Author

sjakobi commented Jun 19, 2020

Well, since the current HEAD version is still 8.11.…, my GHC MR can't validate without this patch. I have indeed pushed it to the GitLab mirror too, so maybe it's not necessary to merge it to master.

@chessai
Copy link
Member

chessai commented Jun 19, 2020

If it's in the GitLab mirror then your MR can validate, right?

@sjakobi
Copy link
Member Author

sjakobi commented Jun 19, 2020

If it's in the GitLab mirror then your MR can validate, right?

Yep, CI is green there. If you've got a minute, could you possibly review it too? It's a base change, so core CLC territory. :)

@sjakobi
Copy link
Member Author

sjakobi commented Jun 19, 2020

FWIW, I wouldn't mind if this PR is closed. I'm just somewhat confused about the workflow of cross-submodule changes.

@chessai
Copy link
Member

chessai commented Jun 19, 2020

@sjakobi, OK, closing (which I planned to do anyway). Sorry, I know GHC's submodule usage is a bit weird.

@chessai chessai closed this Jun 19, 2020
ghc-mirror-bot pushed a commit to ghc/ghc that referenced this pull request Jun 24, 2020
@sjakobi
Copy link
Member Author

sjakobi commented Aug 3, 2020

Re-opening, based on the discussion in https://gitlab.haskell.org/ghc/ghc/-/merge_requests/3388#note_291686.

@sjakobi sjakobi reopened this Aug 3, 2020
@sjakobi sjakobi force-pushed the wip/sjakobi/ghc-mr-3388 branch from 0ade68f to a169feb Compare August 3, 2020 12:22
@sjakobi
Copy link
Member Author

sjakobi commented Aug 3, 2020

Re-pushed since there were some CI failures.

@sjakobi
Copy link
Member Author

sjakobi commented Aug 17, 2020

Ping @chessai.

@chessai chessai merged commit b8c4fb4 into haskell:master Aug 17, 2020
@sjakobi sjakobi deleted the wip/sjakobi/ghc-mr-3388 branch August 17, 2020 15:10
@sjakobi
Copy link
Member Author

sjakobi commented Aug 17, 2020

Cheers! :)

ghc-mirror-bot pushed a commit to ghc/ghc that referenced this pull request Aug 17, 2020
ghc-mirror-bot pushed a commit to ghc/ghc that referenced this pull request Aug 17, 2020
ghc-mirror-bot pushed a commit to ghc/ghc that referenced this pull request Sep 18, 2020
ghc-mirror-bot pushed a commit to ghc/ghc that referenced this pull request Sep 19, 2020
ghc-mirror-bot pushed a commit to ghc/ghc that referenced this pull request Sep 24, 2020
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.

2 participants