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

rules_haskell_tests: Fix ghcide stack pointing to wrong snapshot file #2183

Merged
merged 2 commits into from
May 6, 2024

Conversation

avdv
Copy link
Member

@avdv avdv commented May 6, 2024

The stack snapshot for GHC 9.2.8 was a symlink to the default snapshot file. So
when no GHC_VERSION was set in the environment, it meant the default snapshot
file was used, but the default version is GHC 9.4.6 now.

This led to an error on buildkite as base in GHC 9.4 was bumped to 4.17:

Error: Setup.hs: Encountered missing or private dependencies:
base >=4.8 && <4.17

Symlink the default stack snapshot files to the specific version (not the other way around).

Fixes #2181

@avdv avdv requested a review from ylecornec May 6, 2024 09:32
The stack snapshot for GHC 9.2.8 was a symlink to the default snapshot file. So
when no `GHC_VERSION` was set in the environment, it meant the default snapshot
file was used, which is the one for GHC 9.2 but the default GHC version is 9.4.6.

This led to an error on buildkite as `base` in GHC 9.4 was bumped to 4.17:
```
Error: Setup.hs: Encountered missing or private dependencies:
base >=4.8 && <4.17
```

Symlink the default stack snapshot files to the specific version (not the other
way around).
@avdv avdv force-pushed the cb/fix-default-ghc-snapshot branch from 01fcee7 to 94b7222 Compare May 6, 2024 09:43
@avdv avdv added the merge-queue merge on green CI label May 6, 2024
@mergify mergify bot merged commit cbf5726 into master May 6, 2024
49 checks passed
@mergify mergify bot deleted the cb/fix-default-ghc-snapshot branch May 6, 2024 10:58
@mergify mergify bot removed the merge-queue merge on green CI label May 6, 2024
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.

build on Bazel CI broken
2 participants