Skip to content

Conversation

@develop7
Copy link
Collaborator

@develop7 develop7 commented Sep 7, 2025

Repro:

  1. Install GCC 15 (openSUSE Tumbleweed has it for a while now)
  2. run stack build

Expected: compilation succeds

Instead: build fails building hashtables-1.3.1, reproducing gregorycollins/hashtables#98

This PR makes sure hashtables' version is >= 1.4.1, one with the issue fixed. Stack.yaml specifies 1.4.2 because it's the next version after in the stackage snapshots: https://github.com/commercialhaskell/stackage-snapshots/blob/master/lts/24/0.yaml#L5300

cabal.project Outdated

-- avoid hashtables pre-1.4.1 that fail to build with GCC 15; see https://github.com/gregorycollins/hashtables/issues/98
constraints:
hashtables >= 1.4.1
Copy link
Member

Choose a reason for hiding this comment

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

This line causes rebuilds for Nix, too - which we don't need, because we're not running GCC 15 there, yet.

You can see this in the failing CI jobs:

hashtables-1.4.2 (lib) (requires download & build)

Maybe we should just do something like allow-newer here for it or so. The stack deps are pinned anyway, and Nix could happily use the older version.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

or maybe just remove it altogether from cabal.project? next nixpkgs upgrade will either have 1.4.2 since it's next version in stackage or won't build in case of GCC upgrade only and will be fixed subsequently

Copy link
Member

Choose a reason for hiding this comment

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

if removal works, then yes.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

done, please review again.

@develop7 develop7 force-pushed the fix-hashtables_gcc_15 branch 2 times, most recently from 351762d to cf5e7e2 Compare November 18, 2025 12:36
Introduces hashtables >= 1.4.1 requirement
@develop7
Copy link
Collaborator Author

rebased the PR against latest main, nix build errors seem to be caused by recent cloudflare incident

@wolfgangwalther wolfgangwalther merged commit ae2d650 into PostgREST:main Nov 18, 2025
18 of 22 checks passed
@develop7 develop7 deleted the fix-hashtables_gcc_15 branch November 20, 2025 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants