NixOS / nixpkgs

Nix Packages collection & NixOS
MIT License
18.35k stars 14.31k forks source link

github-desktop: cannot find vscode installation #334253

Open UlyssesZh opened 3 months ago

UlyssesZh commented 3 months ago

Describe the bug

I installed github-desktop from 24.05 branch and vscode-with-extensions from master branch. GitHub Desktop cannot find my VSCode installation in the integration settings.

Steps To Reproduce

Steps to reproduce the behavior:

  1. Install github-desktop and vscode-with-extensions.
  2. Launch GitHub Desktop and open integration settings.
  3. See that there are not editors found.

Expected behavior

Having integration between GitHub Desktop and VSCode.

Screenshots

image

Additional context

Notify maintainers

@dan4ik605743

Metadata


Add a :+1: reaction to issues you find important.

s0me1newithhand7s commented 3 months ago

can you try nix-shell -p vscode github-desktop? (if gui works under nix-shell :D)

UlyssesZh commented 3 months ago

Doesn't seem to work.

s0me1newithhand7s commented 3 months ago

Doesn't seem to work.

'k, i'll try it myself.. asap.

s0me1newithhand7s commented 3 months ago

i cannot reproduce it, strange..

UlyssesZh commented 3 months ago

https://github.com/shiftkey/desktop/blob/3d20d0dc269186e067f1846cb3df1eed314f66d4/app/src/lib/editors/linux.ts#L41-L48

It seems to find editors in hard-coded paths? This should not find installations in /nix/store then.

s0me1newithhand7s commented 3 months ago

https://github.com/shiftkey/desktop/blob/3d20d0dc269186e067f1846cb3df1eed314f66d4/app/src/lib/editors/linux.ts#L41-L48

It seems to find editors in hard-coded paths? This should not find installations in /nix/store then.

yep, maybe a PR with Nixified path will help?

s0me1newithhand7s commented 3 months ago

https://github.com/shiftkey/desktop/pull/1115

'k