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
gitstatus: fix zsh integration #96136
Conversation
cc @mmlb too—I can’t seem to add you to the reviewers list for whatever reason. |
@GrahamcOfBorg build gitAndTools.gitstatus |
At some point the upstream changed and gitstatus.prompt.zsh stopped working. The issue was that this file expects to be able to run the "install" script in the same directory. - Install the "install" script and "build.info" file that gitstatus.prompt.zsh expects to be able to find when it runs. - Patch the install script so that it defaults to not automatically installing gitstatus. This can still be overridden by setting GITSTATUS_AUTO_INSTALL=1, although I'm not sure why anyone would do this if they've already installed gitstatus with Nix. - Add an install check phase that tests the zsh integration to prevent this derivation from breaking in the same way in the future. (This also ends up testing the binary itself.) Fixes NixOS#96135.
94394ee
to
c278c6f
Compare
@GrahamcOfBorg build gitAndTools.gitstatus |
Tested on NixOS and this indeed fixes the zsh integration. |
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.
Sorry, I don't feel well equipped to review this change. It does not seem to cause a regression for me though.
Can only add org members afair. |
Yep, I'm not part of the org (not sure how to get into it actually...). @bdesham I don't use the zsh integration so am not really set up to test this. The diff looks good to me, the test makes sense too. That along with @ryneeverett's input I'd give this a . @mweinelt can you hit the merge button? |
Motivation for this change
At some point the upstream changed and gitstatus.prompt.zsh stopped working. The issue was that this file expects to be able to run the "install" script in the same directory.
Fixes #96135.
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)