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
[20.09] nomad: 0.11.4 -> 0.11.8, 0.12.2 -> 0.12.9 #106416
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, this is me just being lazy |
Would probably be worth it to include bumps from #106827 too. |
6a2668c
to
149c29e
Compare
Result of 2 packages built:
|
LGTM, but the last two commits made me realise that Nomad 0.11/0.12 was never pinned to go 1.14 in 20.09, see #96414. Not sure what's the feeling on backporting that change too. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Still LGTM.
Result of nixpkgs-review pr 106416
run on x86_64-linux 1
2 packages built:
- nomad
- nomad_0_12
(cherry picked from commit 168a85b)
This commit pins go versions for nomad 0.11 and 0.12. Future versions of Nomad should have their versions pinned from the beginning, even if they support the latest-at-the-time version of Go to prevent accidental version bumps on unsupported go versions. See NixOS#96414 for further discussion around this change. (cherry picked from commit 329a922)
(cherry picked from commit d63af62)
(cherry picked from commit dff0cad)
(cherry picked from commit ed6f710)
(cherry picked from commit e8469ca)
(cherry picked from commit 582abe1)
(cherry picked from commit decc5fb)
(cherry picked from commit 2ff401d)
(cherry picked from commit 2f87869)
(cherry picked from commit de94bb4)
8dba9c4
to
16edbb1
Compare
backported |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Still LGTM 😅
thanks @maxeaubrey |
Motivation for this change
closes: #106213
closes: #106215
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)