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

Enable new stdlib build in macOS smoke tests #78477

Merged

Conversation

etcwilde
Copy link
Contributor

@etcwilde etcwilde commented Jan 7, 2025

Enabling running the new standard library build in the macOS smoke tests to ensure that the new build doesn't break.

@etcwilde
Copy link
Contributor Author

etcwilde commented Jan 7, 2025

@swift-ci please smoke test macOS Platform

1 similar comment
@etcwilde
Copy link
Contributor Author

etcwilde commented Jan 8, 2025

@swift-ci please smoke test macOS Platform

@etcwilde etcwilde force-pushed the ewilde/macos-smoketest-enable-new-build branch from 378fca0 to b053867 Compare January 8, 2025 18:51
@etcwilde
Copy link
Contributor Author

etcwilde commented Jan 8, 2025

@swift-ci please smoke test macOS

@etcwilde etcwilde force-pushed the ewilde/macos-smoketest-enable-new-build branch from b053867 to f4c594b Compare January 8, 2025 20:51
@etcwilde
Copy link
Contributor Author

etcwilde commented Jan 8, 2025

@swift-ci please smoke test macOS

Fixed an issue in the convenience wrapper that resulted in building all
copies of the standard library for the default compiler architecture
instead of the desired architecture. To fix this, we pull the desired
target triple and deployment target from the existing variables and pass
that into the CMake invocation.
@etcwilde etcwilde force-pushed the ewilde/macos-smoketest-enable-new-build branch from f4c594b to e0e418f Compare January 9, 2025 00:23
@etcwilde
Copy link
Contributor Author

etcwilde commented Jan 9, 2025

@swift-ci please smoke test

@etcwilde
Copy link
Contributor Author

etcwilde commented Jan 9, 2025

@swift-ci please smoke test macOS

@etcwilde
Copy link
Contributor Author

@swift-ci please smoke test

@etcwilde
Copy link
Contributor Author

@swift-ci please smoke test macOS

@etcwilde etcwilde merged commit 74d4917 into swiftlang:main Jan 23, 2025
3 checks passed
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