Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix iOS mangling #776

Merged
merged 1 commit into from
Jun 23, 2017
Merged

Fix iOS mangling #776

merged 1 commit into from
Jun 23, 2017

Conversation

klefevre
Copy link

Hi,
We had a regression on a project that target iOS while upgrading bindgen from 0.25.3 to 0.25.4.

After a long investigation it appeared that iOS need also a mangling hack. I managed to make a patch and successfully test it.

Thanks a lot for all your work.
Cheers

@emilio
Copy link
Contributor

emilio commented Jun 23, 2017

@bors-servo r+

Thanks a lot, and sorry for the hassle!

@bors-servo
Copy link

📌 Commit 8885365 has been approved by emilio

@bors-servo
Copy link

⌛ Testing commit 8885365 with merge 3106b30...

bors-servo pushed a commit that referenced this pull request Jun 23, 2017
Fix iOS mangling

Hi,
We had a regression on a project that target iOS while upgrading bindgen from `0.25.3` to `0.25.4`.

After a long investigation it appeared that iOS need also a mangling hack. I managed to make a patch and successfully test it.

Thanks a lot for all your work.
Cheers
@bors-servo
Copy link

☀️ Test successful - status-travis
Approved by: emilio
Pushing 3106b30 to master...

@bors-servo bors-servo merged commit 8885365 into rust-lang:master Jun 23, 2017
@klefevre klefevre deleted the fix/ios-mangling branch June 23, 2017 19:32
@klefevre klefevre mentioned this pull request Aug 28, 2017
2 tasks
@upsuper
Copy link
Contributor

upsuper commented Sep 5, 2017

I have a feeling that checking .contains("ios") may not be a good idea. This name is too short, and could potentially conflict in the future.

Probably something like .ends_with("-ios") is more reasonable. Similarly we can convert the macOS check to be .ends_with("-darwin") I guess.

@klefevre
Copy link
Author

klefevre commented Sep 5, 2017

Or we could also check for .contains("apple") ? I agree that a false positive can be triggered with .contains("ios")...

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants