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

[Testing] Unbreak unified build testing #35513

Merged
merged 1 commit into from
Jan 20, 2021

Conversation

davezarzycki
Copy link
Contributor

This test assumes an Apple platform but for reasons unknown, the Linux bots (wrongly?) consider this test to be unsupported.

This test assumes an Apple platform but for reasons unknown, the Linux
bots (wrongly?) consider this test to be unsupported.
@davezarzycki
Copy link
Contributor Author

@swift-ci please smoke test and merge

@davezarzycki
Copy link
Contributor Author

@swift-ci please smoke test macos

@tbkka
Copy link
Contributor

tbkka commented Jan 20, 2021

Hmmmm...... There are some strange things going on here.

  1. The real mystery: Why is this test coming up unsupported on Linux?
  2. This test needs to be fixed before we start running it on Linux. As written, it will not pass on Linux.
  3. This is a "validation" test that isn't run during "smoke test". So you have to use the full "Please test and merge" not the smoke test version.

@tbkka
Copy link
Contributor

tbkka commented Jan 20, 2021

@swift-ci Please test Linux

@@ -8,6 +8,7 @@

// REQUIRES: reflection_test_support
// REQUIRES: executable_test
// REQUIRES: OS=macosx || OS=tvos || OS=watchos || OS=ios
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
// REQUIRES: OS=macosx || OS=tvos || OS=watchos || OS=ios
// REQUIRES: VENDOR=apple

Copy link
Contributor Author

Choose a reason for hiding this comment

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

VENDOR? I'm not at my computer. What does that even mean for Swift's test suite? In Apple's history, they've vended various non-Darwin OSes, so that suggestion feels horribly imprecise

@davezarzycki davezarzycki merged commit 1391a82 into swiftlang:main Jan 20, 2021
@davezarzycki davezarzycki deleted the pr35513 branch January 20, 2021 19:02
@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - ab2f6c7

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.

4 participants