Skip to content

Conversation

@hamishmack
Copy link
Contributor

An alternative would have been to set the version only in the .pc file based on Cargo.toml. However, the derivation’s .version must be accurate so that haskell.nix planning can detect it without building.

Therefore, we hardcode .version in the derivation, and add a check against Cargo.toml to ensure it stays in sync.

An alternative would have been to set the version only in the `.pc` file
based on `Cargo.toml`. However, the derivation’s `.version` must be
accurate so that haskell.nix planning can detect it without building.

Therefore, we hardcode `.version` in the derivation, and add a check
against `Cargo.toml` to ensure it stays in sync.
@hamishmack hamishmack requested a review from angerman September 8, 2025 01:26
Copy link
Contributor

@angerman angerman left a comment

Choose a reason for hiding this comment

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

Thank you!

@angerman angerman merged commit bfdd1c3 into master Sep 8, 2025
3 checks passed
@angerman angerman deleted the hkm/fix-blst-version branch September 8, 2025 02:21
hamishmack added a commit that referenced this pull request Sep 9, 2025
Haskell.nix expects the derivations `.version` and the version in the `.pc` file to match.

This is a follow up to #603
hamishmack added a commit that referenced this pull request Sep 9, 2025
Haskell.nix expects the derivations `.version` and the version in the
`.pc` file to match.

This is a follow up to #603
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.

3 participants