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

Fix AutogenModulesToggling test #9407

Merged
merged 1 commit into from
Nov 10, 2023

Conversation

mpickering
Copy link
Collaborator

By converting this to a setupTest we use the in-tree Cabal library rather than relying on a proxy of the GHC version to provide the right Cabal library version.

Supersedes #9398

@mpickering
Copy link
Collaborator Author

I tested locally to see that reverting #8690 causes this test to fail.

Copy link
Collaborator

@sergv sergv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! It's much better test now. The previous version was limited by my familiarity with Cabal testing infrastructure.

Copy link
Collaborator

@ulysses4ever ulysses4ever left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense to me, thank you!

@mpickering
Copy link
Collaborator Author

It seems to fail in CI. I will try again locally shortly.

@mpickering
Copy link
Collaborator Author

I think this might be failing on older GHC versions because of a recompilation bug in older GHC versions. I removed the job cancelling logic in CI so I can see the full matrix of GHC versions.

@mpickering mpickering force-pushed the wip/autogen-modules-test branch 3 times, most recently from a1d60e6 to 6449aed Compare November 7, 2023 16:34
@mpickering mpickering added the merge me Tell Mergify Bot to merge label Nov 8, 2023
@mergify mergify bot added the merge delay passed Applied (usually by Mergify) when PR approved and received no updates for 2 days label Nov 10, 2023
By converting this to a setupTest we use the in-tree Cabal library
rather than relying on a proxy of the GHC version to provide the right
Cabal library version.

Supersedes haskell#9398
@Mikolaj Mikolaj force-pushed the wip/autogen-modules-test branch from 6449aed to 9d3a7c7 Compare November 10, 2023 09:49
@mergify mergify bot merged commit 394b241 into haskell:master Nov 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge delay passed Applied (usually by Mergify) when PR approved and received no updates for 2 days merge me Tell Mergify Bot to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants