Skip to content

test: fix the broken test #77956

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
Dec 5, 2024
Merged

test: fix the broken test #77956

merged 1 commit into from
Dec 5, 2024

Conversation

compnerd
Copy link
Member

@compnerd compnerd commented Dec 4, 2024

swiftlang/swift-driver#1741 fixed a bug in the swift-driver which exposed a bug in this test. The use of %FileCheck is a problem as that sanitises the output substituting BUILD_DIR for the build directory that is now properly computed. Because PATH is being configured to an invalid path the actual driver is not found and the test fails.

swiftlang/swift-driver#1741 fixed a bug in the swift-driver which
exposed a bug in this test. The use of `%FileCheck` is a problem as that
sanitises the output substituting `BUILD_DIR` for the build directory
that is now properly computed. Because `PATH` is being configured to an
invalid path the actual driver is not found and the test fails.
@compnerd
Copy link
Member Author

compnerd commented Dec 4, 2024

@swift-ci please smoke test

@compnerd compnerd enabled auto-merge December 4, 2024 19:24
@compnerd
Copy link
Member Author

compnerd commented Dec 4, 2024

@swift-ci please test Windows platform

@compnerd
Copy link
Member Author

compnerd commented Dec 5, 2024

@swift-ci please smoke test Linux platform

@compnerd
Copy link
Member Author

compnerd commented Dec 5, 2024

@swift-ci please test Windows platform

@compnerd compnerd merged commit 4822920 into swiftlang:main Dec 5, 2024
3 checks passed
@compnerd compnerd deleted the unsanitary branch December 5, 2024 05:55
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.

1 participant