-
Notifications
You must be signed in to change notification settings - Fork 30.3k
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
VS Code is still adding /d /c
to task commands
#169821
Comments
I'm running into this today, version 1.76.2 while trying to execute a shell script:
|
I'm using Makefile Tools with MSYS2 build environment and the same bug occurs: settings.json:
|
So what about now? I have an AutoRun script in cmd to configure |
1.95.3 is still showing this behavior. Using Is there any progress on completing this issue or, or merging Legend-Master's solution? |
Does this issue occur when all extensions are disabled?: Yes
Steps to Reproduce (on Windows):
npm
task that starts a script."terminal.integrated.automationProfile.windows"
as such:pwsh.exe
/["-Command"]
.nu.exe -c /d /c yarn run name
rather thannu.exe -c "yarn run name"
.This was marked as fixed in #93437, but it isn't. A workaround is given, but is IMO unacceptable. If I modify my shell in user settings, I shouldn't have to modify a
tasks.json
that many users will use, and whose preferred shells are different. More importantly, a single shell can be specified, so/bin/sh
won't work for users on Windows, andcmd.exe
/pwsh.exe
/ whatever won't work for users on Linux. This makesterminal.integrated.automationProfile.*
essentially useless.Since
terminal.integrated.automationProfile.*
takes anargs
argument, why add unwanted arguments to it? I understand why defaults may be nice (e.g. setargs
to["-Command"]
whenpath
ispwsh.exe
), but if preferences are explicitly specified, why append new arguments to them if they only work with a single shell?The text was updated successfully, but these errors were encountered: