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

-Werror incorrectly specified in podspec #84

Closed
cfcommando opened this issue Apr 8, 2015 · 5 comments
Closed

-Werror incorrectly specified in podspec #84

cfcommando opened this issue Apr 8, 2015 · 5 comments
Labels

Comments

@cfcommando
Copy link

Including GCC_TREAT_WARNINGS_AS_ERRORS in the podspec's xcconfig parameters doesn't mean that ComponentKit alone will be compiled with -Werror. Instead a line will be added to Pods/Target\ Support\ Files/Pods/Pods.*.xcconfig, against which project and pods code will be compiled.

While treating warnings as errors is a good policy, it's (unfortunately) not a reasonable constraint to impose on other projects. The flag should be removed. Fortunately, this should be equivalent and applicable more narrowly to ComponentKit specifically:

+  s.compiler_flags = '-Werror -Wall'
   s.xcconfig = {
     'CLANG_CXX_LANGUAGE_STANDARD' => 'c++11',
     'CLANG_CXX_LIBRARY' => 'libc++',
-    'GCC_TREAT_WARNINGS_AS_ERRORS' => 'YES'
@dstnbrkr
Copy link
Contributor

dstnbrkr commented Apr 8, 2015

Thanks. It's unexpected that this flag leaks out to all pods.

@ocrickard ocrickard added the bug label Apr 8, 2015
@cfcommando
Copy link
Author

This is basically a build-break. @ocrickard do you think you could get it in before tagging the next release, please?

@ocrickard
Copy link
Contributor

@dstnbrkr is the one who's the tagging master.

@ocrickard
Copy link
Contributor

#91

@dstnbrkr
Copy link
Contributor

dstnbrkr commented Apr 9, 2015

Fixed in master, will be included in next pod release

@dstnbrkr dstnbrkr closed this as completed Apr 9, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants