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

tests: Make structured hash tests invariant to GHC version #10270

Merged
merged 1 commit into from
Aug 25, 2024

Conversation

mpickering
Copy link
Collaborator

In 9.8 the Generic instance for tuples changed (see https://gitlab.haskell.org/ghc/ghc/-/issues/24291) for more details.

Therefore we remove the dependency on the Generic instance and the hashes will be invariant across GHC versions (for now).

Fixes #10269

Please read Github PR Conventions and then fill in one of these two templates.


Template B: This PR does not modify behaviour or interface

E.g. the PR only touches documentation or tests, does refactorings, etc.

Include the following checklist in your PR:

  • Patches conform to the coding conventions.
  • Is this a PR that fixes CI? If so, it will need to be backported to older cabal release branches (ask maintainers for directions).

@mpickering mpickering force-pushed the wip/structured-hash-fix branch from 63895af to 3fb0a3d Compare August 21, 2024 12:00
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.

Not a fan of the boilerplate but devX is more important.

@mpickering mpickering force-pushed the wip/structured-hash-fix branch from 3fb0a3d to 7a69615 Compare August 21, 2024 13:39
Copy link
Collaborator

@geekosaur geekosaur left a comment

Choose a reason for hiding this comment

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

I had seen another package using that tuple definition and had wondered why; I think you just answered it.

@mpickering mpickering added the merge me Tell Mergify Bot to merge label Aug 23, 2024
@mergify mergify bot added the merge delay passed Applied (usually by Mergify) when PR approved and received no updates for 2 days label Aug 25, 2024
In 9.8 the Generic instance for tuples changed (see
https://gitlab.haskell.org/ghc/ghc/-/issues/24291) for more details.

Therefore we remove the dependency on the `Generic` instance and the
hashes will be invariant across GHC versions (for now).

Fixes #10269
@Mikolaj Mikolaj force-pushed the wip/structured-hash-fix branch from 7a69615 to b90ccb3 Compare August 25, 2024 10:32
@mergify mergify bot merged commit 4f50de3 into master Aug 25, 2024
51 checks passed
@mergify mergify bot deleted the wip/structured-hash-fix branch August 25, 2024 12:45
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.

Structured hash tests are not invariant on GHC version
3 participants