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

Improve reproducibility of the nix workflows #328

Merged
merged 2 commits into from
Mar 7, 2024
Merged

Conversation

musjj
Copy link
Contributor

@musjj musjj commented Feb 29, 2024

The nix-installer-action from DeterminateSystems was causing weird reproducibility issues, where the derivation hash produced in the CI differs from the one produced by my local environment.

I'm not sure why it's happening, but simply switching to the install-nix-action provided by Cachix fixes this issue. You can test this by building locally, then comparing the produced nix store path with the one in the CI logs.

Also added a lib.cleanSource pass to the source tree, to further improve reproducibility.

musjj added 2 commits March 1, 2024 02:22
The nix-installer-action from DeterminateSystems was causing weird
reproducibility issues, where the derivation hash produced in the CI
differs from the one produced by my local environment.

I'm not sure why it's happening, but simply switching to the
install-nix-action provided by Cachix fixes this issue.
@Aylur
Copy link
Owner

Aylur commented Mar 7, 2024

I'll be honest, I still have no idea how this workflow works
@NotAShelf

@NotAShelf
Copy link
Contributor

In general, I find cachix's much more unreliable - had many occurrences of random fetching and permission issues in the CI while using it. However, if it's good enough for Hyprland's nix CI then it's good enough for me.

LGTM, this can be merged @Aylur.

@Aylur Aylur merged commit 5ada482 into Aylur:main Mar 7, 2024
2 checks passed
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