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: Prevent ghc-internal from being reinstalled (see #10087) #10093

Merged
merged 2 commits into from
Jun 13, 2024

Conversation

adamgundry
Copy link
Member

GHC 9.10 ships with a new wired-in package, ghc-internal, which cannot be reinstalled. This commit prevents cabal-install from attempting it. See #10087.

It would be good to include this alongside GHC 9.10 support, if possible.


  • Patches conform to the coding conventions.
  • Any changes that could be relevant to users have been recorded in the changelog.
  • The documentation has been updated, if necessary.
  • Manual QA notes have been included.
  • Tests have been added. (Ask for help if you don’t know how to write them! Ask for an exemption if tests are too complex for too little coverage!)

QA Notes

Attempting to build a package called ghc-internal that has no dependencies should result in a dependency solving error "constraint from non-reinstallable package requires installed instance".

@ulysses4ever ulysses4ever added squash+merge me Tell Mergify Bot to squash-merge and removed attention: needs-review labels Jun 9, 2024
@Mikolaj
Copy link
Member

Mikolaj commented Jun 10, 2024

CI has been broken and is now fixed, so let me rebase to restart it with the fixes included.

@Mikolaj
Copy link
Member

Mikolaj commented Jun 10, 2024

@mergify rebase

GHC 9.10 ships with a new wired-in package, ghc-internal, which cannot
be reinstalled. This commit prevents cabal-install from attempting it.
Copy link
Contributor

mergify bot commented Jun 10, 2024

rebase

✅ Branch has been successfully rebased

@Mikolaj Mikolaj force-pushed the wip/non-reinstallable-ghc-internal branch from be2e1c7 to 20d3a52 Compare June 10, 2024 23:45
@mergify mergify bot added the merge delay passed Applied (usually by Mergify) when PR approved and received no updates for 2 days label Jun 12, 2024
@mergify mergify bot merged commit b1c8ee6 into master Jun 13, 2024
53 checks passed
@mergify mergify bot deleted the wip/non-reinstallable-ghc-internal branch June 13, 2024 01:52
@geekosaur
Copy link
Collaborator

@mergify backport 3.12

Copy link
Contributor

mergify bot commented Jun 13, 2024

backport 3.12

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request Jun 13, 2024
)

GHC 9.10 ships with a new wired-in package, ghc-internal, which cannot
be reinstalled. This commit prevents cabal-install from attempting it.

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
(cherry picked from commit b1c8ee6)
mergify bot added a commit that referenced this pull request Jun 13, 2024
) (#10108)

GHC 9.10 ships with a new wired-in package, ghc-internal, which cannot
be reinstalled. This commit prevents cabal-install from attempting it.

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
(cherry picked from commit b1c8ee6)

Co-authored-by: Adam Gundry <[email protected]>
erikd pushed a commit to erikd/cabal that referenced this pull request Jun 17, 2024
… (haskell#10093)

GHC 9.10 ships with a new wired-in package, ghc-internal, which cannot
be reinstalled. This commit prevents cabal-install from attempting it.

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
@ulysses4ever
Copy link
Collaborator

We missed a changelog entry on this PR, and this is a user-visible change. We should strive to do better next time.

mmhat pushed a commit to mmhat/cabal that referenced this pull request Jun 24, 2024
… (haskell#10093)

GHC 9.10 ships with a new wired-in package, ghc-internal, which cannot
be reinstalled. This commit prevents cabal-install from attempting it.

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
tbidne pushed a commit to tbidne/cabal that referenced this pull request Jul 3, 2024
… (haskell#10093)

GHC 9.10 ships with a new wired-in package, ghc-internal, which cannot
be reinstalled. This commit prevents cabal-install from attempting it.

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cabal-install: solver merge delay passed Applied (usually by Mergify) when PR approved and received no updates for 2 days squash+merge me Tell Mergify Bot to squash-merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants