Skip to content

test: plumb --triple to the resilience test helper #25209

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

Conversation

compnerd
Copy link
Member

@compnerd compnerd commented Jun 2, 2019

The resilience test helper builds up invocations of the tooling. In
order to do this, we need to know what host we are building for. Plumb
the value for -triple from the test harness into the utility. This
will be used subsequently to enable additional testing for Windows.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@compnerd
Copy link
Member Author

compnerd commented Jun 2, 2019

CC: @slavapestov @mikeash

@compnerd compnerd closed this Jun 2, 2019
@compnerd compnerd reopened this Jun 2, 2019
@compnerd
Copy link
Member Author

compnerd commented Jun 2, 2019

@swift-ci please smoke test

The resilience test helper builds up invocations of the tooling.  In
order to do this, we need to know what host we are building for.  Plumb
the value for `-triple` from the test harness into the utility.  This
will be used subsequently to enable additional testing for Windows.
@compnerd compnerd force-pushed the there-is-a-crack-in-the-plumbing branch from fd946df to 1e630a5 Compare June 3, 2019 00:55
@compnerd
Copy link
Member Author

compnerd commented Jun 3, 2019

@swift-ci please smoke test

@swift-ci
Copy link
Contributor

swift-ci commented Jun 3, 2019

Build failed
Swift Test Linux Platform
Git Sha - fd946df5fecd47eb6fd0ace6a735e5124d9402bc

@swift-ci
Copy link
Contributor

swift-ci commented Jun 3, 2019

Build failed
Swift Test OS X Platform
Git Sha - fd946df5fecd47eb6fd0ace6a735e5124d9402bc

@mikeash
Copy link
Contributor

mikeash commented Jun 3, 2019

LGTM. @slavapestov is more familiar with this than me, though, so he should weigh in too.

@compnerd
Copy link
Member Author

compnerd commented Jun 3, 2019

This is currently not doing any changes, so Im going to merge this for the next set of changes. I can address any additional comments from @slavapestov post-commit if there are any.

@compnerd compnerd merged commit 47c8642 into swiftlang:master Jun 3, 2019
@compnerd compnerd deleted the there-is-a-crack-in-the-plumbing branch June 3, 2019 17:38
@mikeash
Copy link
Contributor

mikeash commented Jun 3, 2019

Righto!

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