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
vim_configurable: fix error of gvim when guiSupport = false #91150
Conversation
3f9ce24
to
bacaad0
Compare
shouldn't it be |
Where? |
@@ -154,7 +154,11 @@ in stdenv.mkDerivation rec { | |||
'' + stdenv.lib.optionalString stdenv.isLinux '' | |||
patchelf --set-rpath \ | |||
"$(patchelf --print-rpath $out/bin/vim):${stdenv.lib.makeLibraryPath buildInputs}" \ | |||
"$out"/bin/{vim,gvim} | |||
"$out"/bin/vim | |||
'' + stdenv.lib.optionalString (guiSupport != "false") '' |
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.
If gui support is enabled, the condition will be true, and then we do need to run that patchelf command.
bacaad0
to
0e3b0ef
Compare
Hm, my current use of this command is Alternatively, sidestep the issue: |
0e3b0ef
to
aec4992
Compare
A |
@GrahamcOfBorg eval |
@teto I can confirm once again, that this issue is blocking me from updating to latest unstable. It still perplexes me that I didn't experience this in the past, when I disabled gui support. @arcnmx do you experience this error as well?
|
Yeah, though I (locally) patched nixpkgs last week to work around it so my system wouldn't break. It wasn't until yesterday that I had a chance to check to see if there were any PRs/issues addressing it. I haven't had a chance to test aec4992 yet but looks like it should fix the problem! |
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.
Tested and fixes the issue for me!
if `guiSupport` is either `gtk3` or `gtk2`, then gvim shouldn't be patchelf'ed.
Still getting this error, rebased and pushed on top of the |
Closed in favor https://github.com/NixOS/nixpkgs/pull/92357/files |
Motivation for this change
Otherwise, with the following overlay:
The build fails for me with:
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)