Skip to content

fix FRAMEWORK_SEARCH_PATHS when using carthage - #203

Merged
browniefed merged 1 commit into
tinycreative:masterfrom
michaelknoch:patch-1
Jul 31, 2018
Merged

fix FRAMEWORK_SEARCH_PATHS when using carthage#203
browniefed merged 1 commit into
tinycreative:masterfrom
michaelknoch:patch-1

Conversation

@michaelknoch

Copy link
Copy Markdown
Contributor

Due to this commit 8a77f78 the package is currently broken for users who are using carthage instead of cocoapods

@bensie

bensie commented May 27, 2018

Copy link
Copy Markdown
Contributor

Also if you're installing the Intercom framework manually, then the original implementation was correct:

"$(SRCROOT)/../../../ios/**"

@peterp

peterp commented Jun 5, 2018

Copy link
Copy Markdown

🥇 Thanks for this, whilst we're waiting for this to be merged, you can add use this in your package.json instead:

"react-native-intercom": "chatterbugapp/react-native-intercom#fix-header-search-paths-for-carthage",

@michaelknoch

Copy link
Copy Markdown
Contributor Author

hello? @browniefed ? any feedback? can we merge this?

@browniefed
browniefed merged commit b65ae6f into tinycreative:master Jul 31, 2018
@browniefed

Copy link
Copy Markdown
Contributor

v11.2 is out

@bensie

bensie commented Aug 1, 2018

Copy link
Copy Markdown
Contributor

@browniefed is there a reason that the original "$(SRCROOT)/../../../ios/**" can't also be included? 8a77f78 removed it, but could it keep and just add the other ones for pods and carthage?

@browniefed

Copy link
Copy Markdown
Contributor

Uhm no I don't believe so, throw a PR up and add it.

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.

4 participants