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

replace cmd.exe usage with optional params for windows #128

Merged
merged 1 commit into from
Jun 1, 2019

Conversation

adamralph
Copy link
Owner

cmd.exe is no longer used internally to execute commands on Windows.

There are two new optional parameters, windowsName and windowsArgs. If either the command name or arguments need to be different on Windows, they can be specified as arguments to these parameters. For example:

Run("foo", "bar", windowsName: "cmd.exe", windowsArgs: "/c foo bar");

@adamralph adamralph added enhancement New feature or request breaking This change could break current consumers labels Jun 1, 2019
@adamralph adamralph added this to the 6.0.0 milestone Jun 1, 2019
@adamralph adamralph merged commit e2bc68e into master Jun 1, 2019
@adamralph adamralph deleted the windows-options branch June 1, 2019 08:37
@adamralph adamralph mentioned this pull request Jun 1, 2019
18 tasks
@adamralph
Copy link
Owner Author

Released in alpha 1.

@adamralph
Copy link
Owner Author

@dtitenko @damianh @pawelros @leastprivilege please feel free to kick the tyres on this when you have time. Version is 6.0.0-alpha.1

@leastprivilege
Copy link

I tried it in IdentityModel

https://github.com/IdentityModel/IdentityModel/blob/features/simpleExecTests/build/build.csproj

works fine - both locally and on Azure Pipelines using the VS2017 image.

Once I switch to the VS2019 image - I get

https://dev.azure.com/netidentity/IdentityModel/_build/results?buildId=342

any idea?

@adamralph
Copy link
Owner Author

@leastprivilege no idea, but I got it to work by switching to build.cmd! IdentityModel/IdentityModel#218

I can only guess that something is borked with PowerShell on the new image.

@robertcoltheart
Copy link

robertcoltheart commented Jan 17, 2020

Any chance this can be added to the readme? I spent ages looking for workarounds before I found this PR.

@adamralph
Copy link
Owner Author

@robertcoltheart is this what you had in mind? #185

@robertcoltheart
Copy link

Reviewed, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking This change could break current consumers enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants