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

[iOS] Add folly_compiler_flags to Core subspec #23488

Closed
wants to merge 1 commit into from
Closed

[iOS] Add folly_compiler_flags to Core subspec #23488

wants to merge 1 commit into from

Conversation

Salakar
Copy link
Contributor

@Salakar Salakar commented Feb 16, 2019

Summary

This fixes the following iOS build error when using React via pods (using the new Xcode build system);

project/Pods/Folly/folly/portability/Config.h:20:10: 'folly/folly-config.h' file not found

react-native-community/releases#79 (comment)
react-native-community/releases#79 (comment)
react-native-community/releases#79 (comment)

Changelog

[iOS] [Fixed] - Fixed a "'folly/folly-config.h' file not found" build error when using React via CocoaPods

Test Plan

Before:
image

After:
image

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Feb 16, 2019
@Salakar Salakar changed the title [ios] Add folly_compiler_flags to Core subspec [iOS] Add folly_compiler_flags to Core subspec Feb 16, 2019
@Salakar
Copy link
Contributor Author

Salakar commented Feb 16, 2019

(test_ios on master was failing prior to this PR, https://circleci.com/gh/facebook/react-native/73018)

Copy link
Contributor

@cpojer cpojer left a comment

Choose a reason for hiding this comment

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

Yay!

@facebook-github-bot facebook-github-bot added the Import Started This pull request has been imported. This does not imply the PR has been approved. label Feb 17, 2019
Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@cpojer is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@react-native-bot
Copy link
Collaborator

@Salakar merged commit 5560a47 into facebook:master.

@facebook facebook locked as resolved and limited conversation to collaborators Feb 17, 2019
@react-native-bot react-native-bot added the Merged This PR has been merged. label Feb 17, 2019
grabbou pushed a commit that referenced this pull request Feb 18, 2019
Summary:
This fixes the following iOS build error when using React via pods (using the new Xcode build system);

> project/Pods/Folly/folly/portability/Config.h:20:10: 'folly/folly-config.h' file not found

react-native-community/releases#79 (comment)
react-native-community/releases#79 (comment)
react-native-community/releases#79 (comment)

[iOS] [Fixed] - Fixed a "'folly/folly-config.h' file not found" build error when using React via CocoaPods
Pull Request resolved: #23488

Differential Revision: D14121296

Pulled By: cpojer

fbshipit-source-id: 5eb2ba3066aef2bc9cfb95a9172cbef921c47170
ericlewis pushed a commit to ericlewis/react-native that referenced this pull request Feb 20, 2019
Summary:
This fixes the following iOS build error when using React via pods (using the new Xcode build system);

> project/Pods/Folly/folly/portability/Config.h:20:10: 'folly/folly-config.h' file not found

react-native-community/releases#79 (comment)
react-native-community/releases#79 (comment)
react-native-community/releases#79 (comment)

[iOS] [Fixed] - Fixed a "'folly/folly-config.h' file not found" build error when using React via CocoaPods
Pull Request resolved: facebook#23488

Differential Revision: D14121296

Pulled By: cpojer

fbshipit-source-id: 5eb2ba3066aef2bc9cfb95a9172cbef921c47170
@hramos hramos removed Import Started This pull request has been imported. This does not imply the PR has been approved. labels Feb 23, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged This PR has been merged. Platform: iOS iOS applications.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants