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
git: 2.17.1 -> 2.18.0, round 2 #42575
Conversation
@layus is the hero behind this! See: #42376 (comment) Anyway I haven't tested this with the breakage it caused on various PR's, will do so shortly. |
LGTM -- tried |
LGTM but I'd prefer someone else to merge / confirm since I/we got it "wrong" the first time around :). |
I tested it too and it seems to work ✋ |
lgtm |
the chicken says: staging, maybe? |
Sounds like a plan, no rush here AFAIK anyway. |
…t-2.18.0"" This reverts commit 6d1955e.
6f75ae6
to
bb00076
Compare
@GrahamcOfBorg build git gitFull |
What was causing the failures with the last major bump causing it to be reverted 3 times? Is it a reasonable risk here as well? Is it perhaps an idea to test it by building a smaller job? |
Good question--after that silliness we (well, @layus mostly, IIRC) fixed and started always running git's (installCheck) tests to ensure things at least mostly work as expected. These tests would have caught the problem we encountered last time, re:#38387. Things were made worse by updates being merged separately (perhaps separate PR's? I don't recall) onto both master and staging, and we kept reverting as the two were merged into each other :(. The update to 2.18.0 was reverted once already, but not due to general usage failure in the package itself but because our fetchgit apparently relied on behavior upstream changed-- but so IMO this is different since git was working "as intended". Since breakage resulting from git working as expected seems unlikely, I'd say staging is appropriate but if folks disagree I certainly won't stand in the way of performing additional testing first. |
Failure on x86_64-darwin (full log) Attempted: git, gitFull Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: git, gitFull Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: git, gitFull Partial log (click to expand)
|
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)