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

Windows Terminal as external terminal does not launch in the workspace directory #90734

Closed
tangmi opened this issue Feb 15, 2020 · 4 comments · Fixed by #90773
Closed

Windows Terminal as external terminal does not launch in the workspace directory #90734

tangmi opened this issue Feb 15, 2020 · 4 comments · Fixed by #90773
Assignees
Labels
feature-request Request for new features or functionality good first issue Issues identified as good for first-time contributors help wanted Issues identified as good community contribution opportunities verification-needed Verification of issue is requested verified Verification succeeded windows VS Code on Windows issues
Milestone

Comments

@tangmi
Copy link

tangmi commented Feb 15, 2020

  • VSCode Version: 1.42.1 (user setup)
  • OS Version: Windows_NT x64 10.0.18363

Steps to Reproduce:

  1. Set terminal.external.windowsExec to wt
  2. Run "Open New External Terminal" from the command palette

A new instance of Windows Terminal opens, but with the default directory, instead of ${workspaceFolder}.

Windows Terminal Preview 0.9 recently released, allowing a starting directory to be specified with -d <starting dir>.

Possible solutions?:

Does this issue occur when all extensions are disabled?: Yes

@Tyriar Tyriar added feature-request Request for new features or functionality windows VS Code on Windows issues help wanted Issues identified as good community contribution opportunities labels Feb 15, 2020
@Tyriar Tyriar added this to the Backlog milestone Feb 15, 2020
@Tyriar Tyriar added the good first issue Issues identified as good for first-time contributors label Feb 15, 2020
MonoLizard added a commit to MonoLizard/vscode that referenced this issue Feb 16, 2020
@MonoLizard
Copy link
Contributor

Will be fixed by #90773, my first pr for VsCode cheers :)

@chriscollinsboxuk
Copy link

chriscollinsboxuk commented Mar 20, 2020

Does anyone know if the change in #90773 is going to be merged? I'd love to see this issue resolved, as it's the only thing that's preventing me from fully switching over from ConEmu to Windows Terminal.

@Tyriar Tyriar modified the milestones: Backlog, April 2020 Apr 5, 2020
Tyriar added a commit that referenced this issue Apr 5, 2020
…support

Fix #90734 Add starting directory parameter for the new Windows Terminal
@Sleepful
Copy link

Sleepful commented Apr 16, 2020

What about linux? Mine doesn't open in the opened project directory either, and I can't pass arguments to the "terminal.external.linuxExec" setting option. For example, with Konsole I would only need to add --workdir ${workspaceFolder} as an argument.

@Tyriar
Copy link
Member

Tyriar commented Apr 16, 2020

@Sleepful this issue's about Windows Terminal, if there is a popular terminal (like Konsole) that doesn't work we can open a new issue and open it to PRs. Here's the relevant code:

execPromise.then(exec => {
const env = cwd ? { cwd } : undefined;
const child = spawner.spawn(exec, [], env);

@Tyriar Tyriar added the verification-needed Verification of issue is requested label Apr 27, 2020
@roblourens roblourens added the verified Verification succeeded label Apr 29, 2020
@github-actions github-actions bot locked and limited conversation to collaborators May 21, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature-request Request for new features or functionality good first issue Issues identified as good for first-time contributors help wanted Issues identified as good community contribution opportunities verification-needed Verification of issue is requested verified Verification succeeded windows VS Code on Windows issues
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants
@roblourens @chriscollinsboxuk @Tyriar @tangmi @Sleepful @MonoLizard and others