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
vscode: fix rpath for native modules #52419
Conversation
6565ae9
to
bfdfdf0
Compare
Oh, looks like the Insiders build works now, though the stable build is now broken, Actually, I think I just botched both. |
bfdfdf0
to
80b8dd0
Compare
Seems like the executable works? Though the renderer and extHost don't start up at all, which basically screws everything. |
Alright it seems like it might be due to an RPATH issue since it seems to use the From old to Used |
80b8dd0
to
a03c8ae
Compare
@aszlig might know |
Looks like it's a problem with RPATH since patching in systemd and fontconfig into the RPATH allows it to work. autoPatchelfHook doesn't do it automatically since it doesn't even seem to be required though it does exist when I run ldd. original ldd output: https://pastebin.com/raw/Px8PAbsT after build: https://pastebin.com/raw/ugiykiWU |
There is no direct way to assign priorities, but you can give a dependency precedence over others by using |
a03c8ae
to
c60927f
Compare
c60927f
to
96bf65e
Compare
I removed the hack for libsecret, introduced in[0] and I didn't encounter any problems at runtime. [0]: NixOS#29127
7d0cb57
to
3513063
Compare
Works fine for me, also a crude test of |
Motivation for this change
Affects at least VSCode Insiders 1544768411 and 1545113902.
Current VSCode stable 1.30.1 doesn't need this.
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)