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

[Fixed JENKINS-48803] Updating PowerShell executable name on linux #55

Closed
wants to merge 2 commits into from

Conversation

LiamLeane
Copy link

The reasoning is really stupid and I certainly hope they setup an alias in a future release but powershell is now pwsh, this is in the latest RC and is part of the GA release for powershell next week. Given all previous releases of Powershell for Linux were pre-release I propose that detecting which executable name to use should not be required; if the durable task plugin is updated then so too should powershell (alternatively they can manually alias the command until a Powershell update is applied).

jglick
jglick previously approved these changes Jan 24, 2018
Copy link
Member

@jtnord jtnord left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can not assume that this is the case.

You have JENKINS-47510 for starters which means this will always be broken on windows agents launched via SSH from a Linux master.

And even then you have the joy of what and where launcher.isUnix() may be set (because it is used before the channel is available - and then using cygwin or other SSHd servers on Windows that use / mimic / as a path would probably return true for Linux even if the referenced JIRA was fixed.

@gabloe
Copy link
Contributor

gabloe commented Feb 7, 2018

This change needs to come in with some additional design. Ideally, I would like to see it as an option in Workflow Durable Task Step and then exposed in durable task so that the user can select which edition of PowerShell they want to target (Windows PowerShell, or PowerShell Core). Otherwise, there are too many different configurations to handle that go beyond checking for Windows or *nix, such as the scenario @jtnord pointed out.

@jetersen
Copy link
Member

jetersen commented Oct 6, 2019

This does not consider Windows that may have pwsh installed

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 this pull request may close these issues.

5 participants