-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Switch to ARC as default for target configuration #2262
Comments
Closing as a duplicate of #267. |
This is a different issue. The current behaviour was done to support old deployment targets ('4.0') if I recall correctly. @alloy can we switch this behaviour? |
^ this change would be super useful, as it's the main reason for the warning above this title: http://guides.cocoapods.org/making/using-pod-lib-create#adding-travis-ci |
@irrationalfab I don’t understand, I think you’re confused with treating dispatch types as objects. Afaik this is indeed a duplicate of #267. |
@alloy that was about only changing the default value of the |
@fabiopelosin I'm not understanding your argument that this isn't a duplicate. |
@segiddins One thing is to change the default value if the build setting is not specified in the Podspec ( The other is to change how the target is configured: either ARC enabled and some files are excluded or ARC disabled and some files are included. Currently the latter approach is in use. I think that it was picked because there was an issue with Xcode 4... so we should attempt to change it. |
👍 |
Ah yes, agreed. |
At present each Cocoapods targert is set to
CLANG_ENABLE_OBJC_ARC=NO
and every ARC file is flagged with-fobjc-arc
.It would make more sense to set
CLANG_ENABLE_OBJC_ARC=YES
and flag the non-ARC files with-fno-objc-arc
The reason for this is when I edit a storyboard in a pod, Interface Builder generates non-ARC code for me.
The text was updated successfully, but these errors were encountered: