-
-
Notifications
You must be signed in to change notification settings - Fork 14.1k
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
tmux: fix tmux sha256 hash #49327
tmux: fix tmux sha256 hash #49327
Conversation
@GrahamcOfBorg build tmux |
Success on x86_64-linux (full log) Attempted: tmux Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: tmux Partial log (click to expand)
|
@ookhoi can you adjust the commit message to fit the format described in CONTRIBUTING.MD ? I am a bit inclined to say we should switch to the got revision instead of tags.. They seem to not honor the concept of release tags.. Or github changed something there... |
Uh, did they move tags again? a6dc11e That seems a bad practice. For this expression we could use hashes instead of tag names to specify the revision to download, but that would be relatively ugly. |
Success on aarch64-linux (full log) Attempted: tmux Partial log (click to expand)
|
I don't get any hash mismatches, both on linux and darwin.
|
Of course, is it correct now?
I'm affraid I do not understand but I suspect it will make the tmux fetch more robust. What should I change for that? |
I'm on https://nixos.org/channels/nixos-unstable-small (not sure if that matters) |
There was a change 11d ago because configure.ac was still mentioning "2.8-rc" |
|
Here, logs of checking the same fetching derivation, different results within a couple minutes.
vs.
|
Strange that we seem to get different results. On my local linux machine the new hash looks correct:
|
@ookhoi since upstream seems to have moved the tag, let's pin this to the corresponding upstream git commit instead of the version tag to ensure it's stable:
|
Sorry for the delay @xeji |
I'm running |
@GrahamcOfBorg build tmux |
Success on x86_64-darwin (full log) Attempted: tmux Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: tmux Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: tmux Partial log (click to expand)
|
Motivation for this change
fix:
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)