Skip to content

Infer object format when it can't be determined from a triple's successfully parsed environment #201

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 2 commits into from
Aug 11, 2020

Conversation

owenv
Copy link
Contributor

@owenv owenv commented Aug 10, 2020

Builds on #199, only the second commit is new.

This PR infers a triple's object format if the environment is successfully parsed but the format can't be determined from that environment. This can happen with triples like x86_64-unknown-linux-gnu and caused autolink-extract jobs not to be generated sometimes because the object format wasn't correctly inferred as ELF. Unfortunately I had to ifdef out some tests on macOS as a result since swift-autolink-extract isn't available there.

@owenv
Copy link
Contributor Author

owenv commented Aug 10, 2020

@swift-ci please test Linux

@owenv
Copy link
Contributor Author

owenv commented Aug 10, 2020

Well, that broke a lot of tests 😅

@owenv owenv force-pushed the object-format-fix branch from f02904c to 6ce4bf2 Compare August 10, 2020 03:54
@owenv
Copy link
Contributor Author

owenv commented Aug 10, 2020

@swift-ci please test Linux

@owenv owenv force-pushed the object-format-fix branch from 6ce4bf2 to b728c09 Compare August 10, 2020 04:01
@owenv
Copy link
Contributor Author

owenv commented Aug 10, 2020

@swift-ci please test Linux

@owenv owenv force-pushed the object-format-fix branch from b728c09 to 017a1c4 Compare August 10, 2020 04:06
@owenv
Copy link
Contributor Author

owenv commented Aug 10, 2020

@swift-ci please test Linux

1 similar comment
@owenv
Copy link
Contributor Author

owenv commented Aug 10, 2020

@swift-ci please test Linux

@owenv owenv force-pushed the object-format-fix branch from 017a1c4 to f65f94c Compare August 10, 2020 04:14
@owenv
Copy link
Contributor Author

owenv commented Aug 10, 2020

@swift-ci please test Linux

Copy link
Member

@DougGregor DougGregor left a comment

Choose a reason for hiding this comment

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

Nice catch, thank you!

@owenv
Copy link
Contributor Author

owenv commented Aug 11, 2020

@swift-ci test

@owenv
Copy link
Contributor Author

owenv commented Aug 11, 2020

@swift-ci please test macOS

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