Skip to content
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

Create /etc/shells if it is missing #309

Merged
merged 1 commit into from
May 6, 2022
Merged

Create /etc/shells if it is missing #309

merged 1 commit into from
May 6, 2022

Conversation

jcdickinson
Copy link
Contributor

Resolves: #288

The shell discovery process in vscode crashes if /etc/shells does not exist, resulting in no shells being available (including any custom profiles). This will link /var/run/host/etc/shells to /etc/shells if the latter is missing.

Resolves: #288

The shell discovery process in vscode crashes if `/etc/shells` does not exist, resulting in no shells being available (including any custom profiles). This will link `/var/run/host/etc/shells` to `/etc/shells` if the latter is missing.
@flathubbot
Copy link
Contributor

Started test build 89432

@jcdickinson
Copy link
Contributor Author

Untested, will test this in a few hours.

@flathubbot
Copy link
Contributor

Build 89432 successful
To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/87240/com.visualstudio.code.flatpakref

@jcdickinson jcdickinson marked this pull request as ready for review May 2, 2022 21:31
@jcdickinson
Copy link
Contributor Author

Tested on Silverblue 36.

@nikodunk
Copy link
Contributor

nikodunk commented May 3, 2022

Works well for me on Fedora 36 Workstation, as long as I have my settings.json set up as directed at first startup.

Thank you for the fix! This enables me to stop using the deprecated workaround for settings.json listed in the bug #288 thread.

@nikodunk
Copy link
Contributor

nikodunk commented May 3, 2022

@nedrichards does this make sense to merge? Thank you!

@sangsatori
Copy link

So far works well for me (Fedora 35 Silverblue). 👍

@nikodunk
Copy link
Contributor

nikodunk commented May 5, 2022

Yep - working flawlessly. Been working with it full-time for the past 3 days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[1.64] Terminal profiles stopped working
5 participants