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] some fixes related to nix #3628

Merged
merged 2 commits into from
Oct 5, 2023

Conversation

MangoIV
Copy link
Contributor

@MangoIV MangoIV commented Oct 3, 2023

https://wearezeta.atlassian.net/browse/WPB-4959
https://wearezeta.atlassian.net/browse/WPB-4961

  • repair fixed output derivation cargo packages
  • add benchmark-depends to the devShell
  • make the missing build-tool-depends more reproducible

Currently there are four dependencies (two packages) that cabal is not able to pull from the environment, which are

  • hspec-discover-2.10.10 (lib) (requires build)
  • proto-lens-protoc-0.7.1.1 (lib) (requires build)
  • hspec-discover-2.10.10 (exe:hspec-discover) (requires build)
  • proto-lens-protoc-0.7.1.1 (exe:proto-lens-protoc) (requires build)

I have specified an index-state and a repository for people (like me) who don't have a global repository installed.

This is not too nice, because as you can see, it doesn't pick the version of hspec-discover intended by the pkgs' haskell packages set. I think best would be to pin this somewhere, perhaps in a cabal.project.freeze file (which would need regenerating each time the nixpkgs haskell package set is updated) or directly in the cabal files (which I guess is also hard to maintain).

As far as I can tell this is related to this issue: NixOS/nixpkgs#176887

Fixes #3575

Checklist

  • Add a new entry in an appropriate subdirectory of changelog.d
  • Read and follow the PR guidelines

@CLAassistant
Copy link

CLAassistant commented Oct 3, 2023

CLA assistant check
All committers have signed the CLA.

@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Oct 3, 2023
@MangoIV MangoIV force-pushed the mangoiv/nix/reproducibility-improvements branch 5 times, most recently from 12064bf to 4514741 Compare October 4, 2023 15:20
- repair fixed output derivation cargo packages
- add benchmarkdepends to the devShell
- make the missing build-tool-depends more reproducible
@MangoIV MangoIV force-pushed the mangoiv/nix/reproducibility-improvements branch from 4514741 to 25b1fb9 Compare October 4, 2023 15:31
@MangoIV MangoIV marked this pull request as ready for review October 5, 2023 08:11
@MangoIV MangoIV requested a review from akshaymankar October 5, 2023 08:11
Copy link
Member

@akshaymankar akshaymankar left a comment

Choose a reason for hiding this comment

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

🚀

@MangoIV MangoIV merged commit 568a5bb into develop Oct 5, 2023
@MangoIV MangoIV deleted the mangoiv/nix/reproducibility-improvements branch October 5, 2023 09:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Issue when running "direnv allow" on fresh git clone of repo.
4 participants