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

Always use shell when we exec julia on Unix #3032

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

davidanthoff
Copy link
Member

I think whenever we start a new Julia process, we actually should use a shell on Linux/Mac because there might be some startup script that actually modifies the PATH that is needed.

My understanding is that createTerminal does this automatically, so in a way we are just mimic that behavior here. It might be that createTerminal also uses a shell on Windows, but not sure. But in any case, it seems to me that on Windows we generally don't want to add that extra overhead.

@pfitzseb
Copy link
Member

Mh, not particularly a fan of this because people should just properly set up their system. Since that might be a bit unrealistic, we should presumably just go with this...

createTerminal definitely does not go through a shell as it's currently used, so we'll need to adjust that as well (which may very well turn out to be a bit of a nightmare, because we need to handle escaping for ~arbitrary shells properly...).

@davidanthoff
Copy link
Member Author

Ah, you are right, I had misread the docs for createTerminal. Yeah, so let's not merge this here then.

The thing that is really confusing to me is that it seems we have systems where the LS and the REPL start (both of course launched not by us directly but rather by some VS Code internal code) and at the same time when we try to launch Julia for say the test process it seems to fail. That is how I am interpreting julia-vscode/TestItemRunner.jl#19... If that is so, then there seems to be some difference in how we launch things relative to how say createTerminal launches things...

@davidanthoff davidanthoff marked this pull request as draft September 9, 2022 00:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants