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

Stops the Window installer from hanging #52365

Merged
merged 5 commits into from
Apr 9, 2019

Conversation

cmcmarrow
Copy link
Contributor

*Removed “-NoNewWindow” which was the cause of the hang when pip ran
*Cleaned up start-process-and-test-exitcode.psm1
*Switched the order of Removed Temporary and script complete in “build_env_3.ps1”
*www.github.com/saltstack/salt/issues/52265

What does this PR do?

*Removed “-NoNewWindow” which was the cause of the hang when pip ran
*Cleaned up start-process-and-test-exitcode.psm1
*Switched the order of Removed Temporary and script complete in “build_env_3.ps1”

What issues does this PR fix or reference?

www.github.com/saltstack/salt/issues/52265

Tests written?

No

Commits signed with GPG?

Yes

Please review Salt's Contributing Guide for best practices.

See GitHub's page on GPG signing for more information about signing commits with GPG.

*Removed “-NoNewWindow” which was the cause of the hang when pip ran
*Cleaned up start-process-and-test-exitcode.psm1
*Switched the order of Removed Temporary and script complete in “build_env_3.ps1”
*www.github.com/saltstack/salt/issues/52265
pkg/windows/build_env_3.ps1 Outdated Show resolved Hide resolved
pkg/windows/build_env_2.ps1 Outdated Show resolved Hide resolved
@cmcmarrow cmcmarrow requested a review from twangboy April 9, 2019 18:29
@twangboy twangboy merged commit 6b84ab2 into saltstack:develop Apr 9, 2019
garethgreenaway added a commit to garethgreenaway/salt that referenced this pull request Sep 18, 2019
dwoz added a commit that referenced this pull request Dec 29, 2019
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.

3 participants