-
Notifications
You must be signed in to change notification settings - Fork 52
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
Unable to build with Giphy on Xcode 12 #96
Comments
hey @drewolbrich going to download the GM. in the meantime, can you share your Podfile? |
Hi @cgmaier thank you for responding. I had the same issue with the previous version of Xcode 12 beta, but I did not have time to investigate it then. The issue is not unique to the GM. Our Podfile is here: http://traipse.com/giphy/Podfile.txt The Giphy reference is down near the bottom. If I remove the Let me know if there is anything else you need or would like me to try. |
This happens if the deployment target is >= 13.4 |
Here's minimal sample project reproducing the issue. |
Thank you. |
fwiw my project's deployment target is 12.1. |
It seems that this doesn't happen from source, but if one of the dependencies is delivered as a binary. E.g. https://s3.amazonaws.com/sdk.mobile.giphy.com/SDK/1.3.0/GiphySDK.zip |
can you guys try this? compiled with Xcode 12 compiler:
|
@cgmaier I am in the middle of juggling something else, maybe for 1/2 hour, but I will try that immediately after I'm done. |
@cgmaier This didn't work for us. We also had to run it through a modified local podspec since PINCache 3.1.0 doesn't exist on trunk yet. |
We also tried manually integrating the previous version & using carthage. Both seems to work. This might be a Cocoapods issue. |
ok gotcha. not sure what happened with PINCache |
thanks guys - I just updated the podspec here:
be sure to run this first!
|
@cgmaier I tried adding Giphy via What version of Xcode are you using to build GiphyUISDK? |
@drewolbrich I don't think it's picking up the correct binary. Are you sure you're not including, say 1.3.0 ? |
it works with 13.3 target deployment info - but when I bump up to 13.4 I get compile error, though different than those noted above.
seems similar to this issue, if not the same one: |
hey all - have an update! believe some aspect of these issues has to do with the extra architectures included in build process to support simulators for apple silicon Macs. some discussion here: this works for me now for deployment targets below & above 13.4
|
@cgmaier The instructions above did fix the issue here, amazing, and thanks for the quick investigation ⭐ 🥂 Can we expect you to make this into an official Podspec soon? |
very much appreciate the speedy confirm. yes I will push an official release shortly |
@cgmaier can you elaborate on how the issue is connected to the extra simulator arch for Apple Silicon Macs? |
@cgmaier This is working for me now using the podspec on AWS. I had to take Xcode outside and have a stern conversation with it, but ultimately it came around. |
@cgmaier Also, thank you !!! 😄 |
hey @tmspzz I pasted the wrong link following my high level understanding of the issue is that Xcode 12s command line build process produces duplicate architectures in order to support simulators on the new Apple Silicon macs (arm) which create conflicts. I believe the solution provided in the above thread is to exclude those Apple Silicon simulator architectures. this whole ordeal reaffirms the value of SPM / .xcframework support #49 |
@cgmaier If you don't mind, please post here when the next official release is available. |
We have a bunch of other Pods in our Podfile and when after running the recommended commands above, we end up with a bunch of new errors ("duplicate architecture symbols", among others). If we uninstall all Pods and then reinstall all Pods, it removes all of our new errors, but the original error from this chain still persists. Please let us know when the official updated release is available for the Giphy pod! In the meantime, we were able to temporarily get around the issue with the following: |
@cgmaier I have been using However, I just discovered that in the simulator, it only appears to work for debug builds. Non-debug builds fail with this error: I see this error when building for the simulator for iOS 12, 13, and 14. I tried deleting DerivedData and deleting the Cocopods cache and Pods directories and running Non-debug builds do work fine on the device, so that is a workaround for the short term. |
hey all, I've finally pushed an official release here https://github.com/Giphy/giphy-ios-sdk-ui-example/releases/tag/v2.0.3 |
@drewolbrich I am seeing another workaround here: Carthage/Carthage#3019 (comment) |
@cgmaier when using the official v2.0.3 sdk my app crashes now after presenting the GiphyViewController. It looks like the crash is happening somewhere in PINCache. When using the podspec linked in previous comments the crash does not happen. It looks like that pod spec is installing PINCache 2.3 where the official one installs v2.0. |
Thank you, this worked beautifully. |
hey @KrisConrad my apologies for the late reply. this is now fixed in 2.0.4. |
Awesome, thanks! |
Hello cgmaier this command is not working in my mac |
its giving the issue like this |
update docs with new core sdk download link
any update |
Using Xcode 12 GM, I'm unable to build against GiphyUISDK. I get the following errors:
I'm confused about the "Compiling for iOS 10" part, because everywhere in my project I'm explicitly specifying 12.1 as the deployment target.
I tried updating my Podfile post_install handler to explicitly set
IPHONEOS_DEPLOYMENT_TARGET
, but no dice.Is there any obvious solution to this I'm overlooking?
The text was updated successfully, but these errors were encountered: