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

Some issues running code-server within code-server #4519

Closed
code-asher opened this issue Nov 16, 2021 · 4 comments · Fixed by coder/vscode#22
Closed

Some issues running code-server within code-server #4519

code-asher opened this issue Nov 16, 2021 · 4 comments · Fixed by coder/vscode#22
Assignees

Comments

@code-asher
Copy link
Member

It should spin up a new code-server as long as something in the format code-server [file] is not passed (in that case it opens in the existing instance) but it seems to spawn some other CLI instead.

Installing extensions also seems to hang at the end.

@code-asher code-asher added this to the 4.0.0 milestone Nov 16, 2021
@jsjoeio
Copy link
Contributor

jsjoeio commented Nov 22, 2021

I think this is an edge case that can be moved to 4.0.1. @code-asher let me know if you disagree.

@jsjoeio jsjoeio modified the milestones: 4.0.0, 4.0.1 Nov 22, 2021
@code-asher
Copy link
Member Author

code-asher commented Nov 24, 2021 via email

@jsjoeio jsjoeio modified the milestones: 4.0.1, 4.0.0 Nov 29, 2021
@jsjoeio
Copy link
Contributor

jsjoeio commented Nov 29, 2021

(for example from the product)

Didn't consider that scenario. Fair point! Moved back to 4.0.0

@jsjoeio
Copy link
Contributor

jsjoeio commented Dec 7, 2021

Heads-up: we've started working on this.

code-asher added a commit to coder/vscode that referenced this issue Dec 7, 2021
This partially fixes coder/code-server#4519.

We use an environment variable to distinguish the parent process from
the child so without sanitizing when you spawn code-server from VS Code
it always looks like we are in the parent and the process will fail
since it is not actually the child.
code-asher added a commit to code-asher/code-server that referenced this issue Dec 7, 2021
This sanitizes our environment variables so code-server does not always
think it is a child spawn.

Fixes coder#4519.
code-asher added a commit that referenced this issue Dec 8, 2021
* Remove extra VS Code CLI spawn

We already spawn VS Code's CLI when necessary in the lines below.
Having the CLI spawn unconditionally when in a VS Code environment makes
it impossible to run code-server within code-server (for example to
develop it).

* Update VS Code

This sanitizes our environment variables so code-server does not always
think it is a child spawn.

Fixes #4519.
ZauberNerd pushed a commit to ZauberNerd/vscode that referenced this issue Dec 23, 2021
This partially fixes coder/code-server#4519.

We use an environment variable to distinguish the parent process from
the child so without sanitizing when you spawn code-server from VS Code
it always looks like we are in the parent and the process will fail
since it is not actually the child.
ZauberNerd pushed a commit to ZauberNerd/vscode that referenced this issue Dec 23, 2021
This partially fixes coder/code-server#4519.

We use an environment variable to distinguish the parent process from
the child so without sanitizing when you spawn code-server from VS Code
it always looks like we are in the parent and the process will fail
since it is not actually the child.
ZauberNerd pushed a commit to ZauberNerd/vscode that referenced this issue Dec 23, 2021
This partially fixes coder/code-server#4519.

We use an environment variable to distinguish the parent process from
the child so without sanitizing when you spawn code-server from VS Code
it always looks like we are in the parent and the process will fail
since it is not actually the child.
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 a pull request may close this issue.

2 participants