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
vscodium: Symlink backwards-compabible vscodium
executable
#63522
Conversation
Can we make it so if either |
4ee7fa3
to
dda2f50
Compare
I'm not sure I understand what you mean. |
dda2f50
to
693480a
Compare
@@ -33,6 +33,11 @@ in | |||
|
|||
sourceRoot = "."; | |||
|
|||
postInstall = '' | |||
# Install legacy support: | |||
ln -s $out/bin/${executableName} $out/bin/vscodium |
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.
Could you explain why we need a backwards-compatible symlink? Do upstream or other linux distributions provide such a symlink?
@@ -69,6 +70,8 @@ in | |||
|
|||
installPhase = | |||
if system == "x86_64-darwin" then '' | |||
runHook preInstall | |||
|
|||
mkdir -p $out/lib/vscode $out/bin | |||
cp -r ./* $out/lib/vscode | |||
ln -s $out/lib/vscode/Contents/Resources/app/bin/${executableName} $out/bin |
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.
ln -s $out/lib/vscode/Contents/Resources/app/bin/${executableName} $out/bin | |
ln -s $out/lib/vscode/Contents/Resources/app/bin/${executableName} $out/bin | |
runHook postInstall |
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.
You'll need preInstall
after the else
too.
I'm not sure this would be relevant to users anymore. |
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)