-
Notifications
You must be signed in to change notification settings - Fork 385
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
Add support for VSCode based IDE inside Flatpak #1577
base: main
Are you sure you want to change the base?
Conversation
a442ad7
to
59cb805
Compare
59cb805
to
cd4654f
Compare
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.
LGTM overall, couple of small comments. Thanks for the contribution again!
Just as a reminder, we'll need to create a follow up PR to display VSCodium Insiders in DevPod Desktop.
Edit: linting failed
Name: config.IDECodiumInsiders, | ||
DisplayName: "Codium Insiders", | ||
Options: vscode.Options, | ||
Icon: "https://devpod.sh/assets/codium_insiders.svg", // TODO to be uploaded |
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.
This is available now, you can remove the TODO
:)
@@ -24,5 +24,6 @@ const ( | |||
IDEPositron IDE = "positron" | |||
IDEMarimo IDE = "marimo" | |||
IDECodium IDE = "codium" | |||
IDECodiumInsiders IDE = "codium-insiders" |
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.
We'll have to expose this in DevPod Desktop as well, will create a follow up PR
pkg/ide/vscode/open.go
Outdated
return fmt.Errorf("couldn't find the %s binary", flavor) | ||
} | ||
|
||
if codePath[0] == "flatpak" { | ||
log.Debugf("Running with Flatpak suing the package %s.", codePath[2]) |
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.
nitpick:
log.Debugf("Running with Flatpak suing the package %s.", codePath[2]) | |
log.Debugf("Running with Flatpak using the package %s.", codePath[2]) |
log.Warnf("The IDE is already running via Flatpak. If you are encountering SSH connectivity issues, make sure to give read access to your SSH config file (e.g flatpak override %s --filesystem=%s) and restart your IDE.", codePath[2], sshConfigPath) | ||
} | ||
|
||
codePath = slices.Insert(codePath, 2, fmt.Sprintf("--filesystem=%s:ro", sshConfigPath)) |
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.
Ah, I see. Nice
This adds support for VSCode based IDE installed via Flatpak on Linux. Note that this depends on a limitation in
open-remote-ssh
Note: this is also adding support for Codium Insiders. Let me know if you want me to split the commit or PR