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

clang: error: argument unused during compilation: '-I core' #41

Closed
ryandesign opened this issue Sep 30, 2013 · 4 comments
Closed

clang: error: argument unused during compilation: '-I core' #41

ryandesign opened this issue Sep 30, 2013 · 4 comments
Assignees
Milestone

Comments

@ryandesign
Copy link

Trying to compile latest potion with clang and ccache produces this message:

CC core/potion.opic
clang: error: argument unused during compilation: '-I core'

It's because ccache is not yet fully clang-compatible, and it produces this warning, and you've requested by specifying -Werror in config.mak that all warnings be turned into errors.

In MacPorts we run into this kind of problem a lot, and recommend that you don't ship build systems that use -Werror, because you never know what kinds of new warnings will appear in compilers in the future.

@rurban
Copy link
Member

rurban commented Sep 30, 2013

If ccache doesn't accept -I core does it compile at all?
It does but throws a lot of new warnings: -Warray-bounds -Wtautological-compare -Wparentheses-equality besides the 2 unused arguments ccache warnings. I'll have to check why.

I'll remove -Werror for the upcoming 0.1 release version but for the development cycle I'm fine with it.

@ghost ghost assigned rurban Sep 30, 2013
@ryandesign
Copy link
Author

Yes, it compiles; ccache just throws an incorrect warning. It will be fixed in ccache 3.2 which is not out yet. https://bugzilla.samba.org/show_bug.cgi?id=8118

@rurban
Copy link
Member

rurban commented Sep 30, 2013

Fixes added with
commit fe8e5b5
Author: Reini Urban rurban@cpanel.net
Date: Mon Sep 30 14:09:50 2013 -0500

ccache clang: add -Qunused-arguments
According to https://bugzilla.samba.org/show_bug.cgi?id=8118

commit 5cb9251
Author: Reini Urban rurban@cpanel.net
Date: Mon Sep 30 12:05:13 2013 -0500

[potion #41] fix for ccache clang
ccache clang is not -Werror safe

to p2, will be backported to potion soon.

@rurban
Copy link
Member

rurban commented Oct 4, 2013

Fixed in potion/master with commit 032ea07
Will be in the upcoming 0.1 release

@rurban rurban closed this as completed Oct 4, 2013
rurban pushed a commit that referenced this issue Jan 14, 2014
ccache clang is not -Werror safe
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

No branches or pull requests

2 participants