-
Notifications
You must be signed in to change notification settings - Fork 481
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
Eliminate compiler warnings and turn on -Werr #776
Comments
I saw these too, and this is pretty unfortunate as these are recent Bazsi On Tue, Nov 10, 2015 at 2:05 PM, Tibor Benke notifications@github.com
|
ehh... I think we should do something with these warnings :-( |
I think the OSE tree is not that far, we only have a handful of warnings,
Bazsi On Tue, Nov 10, 2015 at 4:28 PM, Budai Laszlo notifications@github.com
|
Hmm... Then -Werr is a real option. |
+1 for the -Werr On Tue, Nov 10, 2015 at 5:08 PM, Budai Laszlo notifications@github.com
|
With the last two branches, syslog-ng should compile warning free with this configuration: I am enabling redis/riemann/java to see if they are warning free as well. |
I wanted to turn this on, but I see some warnings in internal ivykis:
|
I've checked the build log on my own computer, and
Could you elaborate why On Wed, Dec 16, 2015 at 2:10 PM, Tibor Benke notifications@github.com
|
I can see -pthread on that line.
|
@bazsi: Just to elaborate, do you mean on your own computer too? Do you get On Wed, Dec 16, 2015 at 9:42 PM, Balazs Scheidler notifications@github.com
|
I missed the build output as it was hidden while reading your email on my On the other hand I do know that ivykis used to use weak symbols on
|
I'll send a PR and we will see what happens on Travis. |
It fails: #843 |
I used the master branch.
To do:
The text was updated successfully, but these errors were encountered: