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

Xcode12 compatibility: Remove VALID_ARCHS from User-Defined settings #1053

Merged
merged 1 commit into from
Oct 27, 2020
Merged

Xcode12 compatibility: Remove VALID_ARCHS from User-Defined settings #1053

merged 1 commit into from
Oct 27, 2020

Conversation

carloscuesta
Copy link
Contributor

@carloscuesta carloscuesta commented Oct 7, 2020

Closes #1036

Description

This fixes the following issue: #1036

The RCTOneSignal target has a User-Defined setting called VALID_ARCHS. This is not compatible anymore with Xcode12. So we should remove it.

@carloscuesta
Copy link
Contributor Author

Hey @rgomezp can you take a look at this?

@rgomezp rgomezp requested a review from emawby October 14, 2020 20:59
Copy link
Contributor

@emawby emawby left a comment

Choose a reason for hiding this comment

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

We need the project to still work in at least Xcode 11. Does this change affect the Xcode 11 setting?

@carloscuesta
Copy link
Contributor Author

carloscuesta commented Oct 17, 2020

It works too on Xcode11 @emawby

Screenshot 2020-10-18 at 12 54 41

@carloscuesta
Copy link
Contributor Author

@emawby Did I need to make anything else to get this merged?

Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants