Skip to content

Use Invoke-Program for the Windows build command#3140

Merged
bnbarham merged 1 commit into
swiftlang:mainfrom
bnbarham:invoke-windows
Aug 25, 2025
Merged

Use Invoke-Program for the Windows build command#3140
bnbarham merged 1 commit into
swiftlang:mainfrom
bnbarham:invoke-windows

Conversation

@bnbarham
Copy link
Copy Markdown
Contributor

Windows does not stop scripts when native commands exit with non-zero. Instead, their exit status has to be checked manually. This can be done through an Invoke-Program function that is added to the script running windows_build_command, which also previously prefixed the given command. This is changing in
swiftlang/github-workflows#154 since it doesn't help with multi-line commands - update our modified windows_build_command to use Invoke-Program instead.

Windows does not stop scripts when native commands exit with non-zero.
Instead, their exit status has to be checked manually. This can be done
through an `Invoke-Program` function that is added to the script running
`windows_build_command`, which also previously prefixed the given
command. This is changing in
swiftlang/github-workflows#154 since it doesn't
help with multi-line commands - update our modified
`windows_build_command` to use `Invoke-Program` instead.
@jakepetroules
Copy link
Copy Markdown

@swift-ci test

@bnbarham
Copy link
Copy Markdown
Contributor Author

@swift-ci please test Linux platform

@ahoppen
Copy link
Copy Markdown
Member

ahoppen commented Aug 22, 2025

@swift-ci Please test Windows

3 similar comments
@bnbarham
Copy link
Copy Markdown
Contributor Author

@swift-ci Please test Windows

@ahoppen
Copy link
Copy Markdown
Member

ahoppen commented Aug 23, 2025

@swift-ci Please test Windows

@bnbarham
Copy link
Copy Markdown
Contributor Author

@swift-ci Please test Windows

@bnbarham bnbarham merged commit 544d6fe into swiftlang:main Aug 25, 2025
30 checks passed
@bnbarham bnbarham deleted the invoke-windows branch August 25, 2025 19:07
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