Skip to content

Call python3 to execute XCTest's build_script.py. #65958

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

Merged
merged 1 commit into from
May 17, 2023
Merged

Call python3 to execute XCTest's build_script.py. #65958

merged 1 commit into from
May 17, 2023

Conversation

fischman-bcny
Copy link
Contributor

A workaround for the fact that fixing this in XCTest's repository (swiftlang/swift-corelibs-xctest#441) is blocked on XCTest's CI being broken for unrelated reasons (and ones which I cannot reproduce locally).

A workaround for the fact that fixing this in XCTest's repository
(swiftlang/swift-corelibs-xctest#441) is blocked on XCTest's
CI being broken for unrelated reasons (and ones which I cannot reproduce).
@compnerd
Copy link
Member

@swift-ci please test

Copy link
Member

@compnerd compnerd left a comment

Choose a reason for hiding this comment

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

This seems like a reasonable thing to do. The project does use python3. Windows in general also does the same thing of specifying the interpreter. We can do future enhancements to allow the user to control the interpreter outside of PATH.

@compnerd compnerd merged commit fcdca08 into swiftlang:main May 17, 2023
MaxDesiatov added a commit that referenced this pull request Jun 15, 2023
[5.8] Call python3 to execute XCTest's build_script.py

A workaround for the fact that fixing this in XCTest's repository (swiftlang/swift-corelibs-xctest#441) is blocked on XCTest's CI being broken for unrelated reasons (and ones which I cannot reproduce).

Risk: low, all other Python scripts in the `swift` and `swift-package-manager` repositories already include this change and use `python3` in both `main` and `release/5.9` branches. This specific change has been on the `main` and `release/5.9` branches for a while now.
Original PRs: #65958 for `main` and #66415 for `release/5.9`
Reviewed by: @compnerd, @al45tair, and @shahmishal 
Resolves: unblocks PRs on the `swift-corelibs-foundation` repository that build XCTest as a part of their CI jobs.
Tests: N/A
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.

2 participants