-
Notifications
You must be signed in to change notification settings - Fork 530
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
Fix "Argument list too long: recursive header expansion failed" issue #275
Conversation
This is nice PR |
Thanks! Sorry for the late reply. Do you have any idea why this solves the issue? |
@koenpunt Can't say for sure, but this is what I think: I suspect that |
Sounds acceptable. I'll see if I can make the search paths even more specific, and if that doesn't work I'll merge as-is. Thanks! 🚀 |
Please merge. I had to make these changes in my project |
This worked at one point, now getting this
|
@evanjmg I had a lot of issues with my project as well, and the solution in my case was to stop including |
This change prevents the project from working properly with manual linking (no CocoaPods). It can't find the GoogleMobileAds framework. |
@rawrmaan you have to add GoogleMobileAds to your project yourself. |
@koenpunt Yeah, I know. For some reason, this change prevents Xcode from finding the framework added to my project. I have my own branch where I reverted the change and it works perfectly. |
Fixes #209.
See this comment, and this comment.