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

Allow hashable-1.5 #10178

Closed
wants to merge 1 commit into from
Closed

Allow hashable-1.5 #10178

wants to merge 1 commit into from

Conversation

geekosaur
Copy link
Collaborator

Fixes #10177

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). —N/A

@ulysses4ever
Copy link
Collaborator

CI still picks up 1.4.7:
https://github.com/haskell/cabal/actions/runs/9802518842/job/27067339750?pr=10178#step:13:117

In theory, someone should build under the constraint ==1.5 + allow-newer. Was there a problem with the approach described at the bottom of https://github.com/haskell/cabal/blob/master/CONTRIBUTING.md#hackage-revisions ? Would be good to figure it out...

@geekosaur
Copy link
Collaborator Author

I'm running it locally with overrides before I trigger a manual run. I'm also wondering if the frozen index-state will keep CI from finding it.

@geekosaur
Copy link
Collaborator Author

Well. This is going to be fun: we validate with ghc 9.4.8, but the new hashable requires the base from 9.6.

@ulysses4ever
Copy link
Collaborator

Right, this was the problem last time we tried the manual run. Back then the static build succeeded (it's Independent of others so it may succeed even if others fail) so that was an evidence that this is a viable version bump. But in this case it will also fail because its GHC is older than 9.6 I think?

@geekosaur
Copy link
Collaborator Author

It did fail, yes, and the build log confirmed it was using ghc 9.4.8.

@geekosaur
Copy link
Collaborator Author

Local validate fails:

Integration tests (internal)
  Regression tests
    program options scope all:                 FAIL (1.30s)
      tests/IntegrationTests2.hs:1560:
      q
      expected: Just ["-fno-full-laziness"]
       but got: Just ["-haddock"]
      Use -p '/program options scope all/' to rerun this test only.
    program options scope local:               FAIL (1.25s)
      tests/IntegrationTests2.hs:1585:
      q
      expected: Just ["-fno-full-laziness"]
       but got: Just ["-haddock"]
      Use -p '/program options scope local/' to rerun this test only.
    program options scope specific:            FAIL (1.27s)
      tests/IntegrationTests2.hs:1610:
      q
      expected: Nothing
       but got: Just ["-haddock"]
      Use -p '/program options scope specific/' to rerun this test only.

I don't know how much of this is because I have documentation: True in my ~/.config/cabal/config, but that's never caused problems before. Or if maybe it's related to #10163 somehow, or some other PR affecting haddock.

@ulysses4ever
Copy link
Collaborator

I don't know how much of this is because I have documentation: True in my ~/.config/cabal/config, but that's never caused problems before.

Well, it's cheap to check this guess, isn't it? I'd do that. It's not clear what to do if it's true, but that would fill in one bit of information for the fitire investigation.

@ulysses4ever ulysses4ever added the attention: needs-help Help wanted with this issue/PR label Jul 8, 2024
@geekosaur
Copy link
Collaborator Author

Indeed, disabling documentation: True made the test pass. I presume it needs to specify its own cabal configuration, as several other tests do.

@ulysses4ever ulysses4ever removed the attention: needs-help Help wanted with this issue/PR label Jul 8, 2024
@ulysses4ever
Copy link
Collaborator

Mm, does it only fail with this patch about hashable-1.5? It's pretty confusing...

@geekosaur
Copy link
Collaborator Author

I doubt it, I just don't run local validates that often and last time I did I'd disabled documentation because it was crashing cabal-install HEAD with an internal assert.

@ulysses4ever
Copy link
Collaborator

ah, good

@geekosaur
Copy link
Collaborator Author

And confirmed, it also fails on master with documentation: True. I'll look into making IntegrationTests2 use its own cabal config so it's not affected by the local environment.

@geekosaur
Copy link
Collaborator Author

Mrf. It already plays games with the cabal config.

@geekosaur geekosaur closed this Jul 18, 2024
@geekosaur geekosaur deleted the hashable-1.5 branch July 18, 2024 03:38
@ulysses4ever
Copy link
Collaborator

@geekosaur any particular reason this one was closed?

@ulysses4ever
Copy link
Collaborator

The battle goes on in #10244

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow hashable-1.5
2 participants