[CDRIVER-669] Turn compiler warnings into errs on Evergreen Created: 18/May/15  Updated: 27/May/16  Resolved: 09/Dec/15

Status: Closed
Project: C Driver
Component/s: None
Affects Version/s: None
Fix Version/s: 1.3.1

Type: New Feature Priority: Major - P3
Reporter: A. Jesse Jiryu Davis Assignee: Hannes Magnusson
Resolution: Done Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Related
related to CDRIVER-958 Improve configure flags on evergreen Closed
related to CDRIVER-702 Warnings due to GCC extensions (-wped... Closed
is related to CDRIVER-667 "make dist" test in Evergreen Closed
is related to CDRIVER-1044 Enable -Werror for clang on Evergreen Closed
Epic Link: C Driver in Evergreen

 Description   

The goal is to compile without warnings everywhere; not, mainly, because every warning is cause for alarm, but because compiler warnings make attractive targets for low-value bug reports and pull requests that drown out more important project activity.



 Comments   
Comment by Hannes Magnusson [ 09/Dec/15 ]

-Werror is now enabled for GCC on Solaris (32 & 64bit), and Ubuntu.

Created another ticket to track this for clang as that'll require fixing painful align warnings...

Comment by Githook User [ 09/Dec/15 ]

Author:

{u'username': u'bjori', u'name': u'Hannes Magnusson', u'email': u'bjori@php.net'}

Message: CDRIVER-669: Turn compiler warnings into errors for GCC on Evergreen
Branch: master
https://github.com/mongodb/mongo-c-driver/commit/da87fdcdaaa8a06b07133362ff5cdb046645c7c5

Comment by Githook User [ 02/Nov/15 ]

Author:

{u'username': u'bjori', u'name': u'Hannes Magnusson', u'email': u'bjori@php.net'}

Message: CDRIVER-669: That really didn't go as planned – revert for a while
Branch: master
https://github.com/mongodb/mongo-c-driver/commit/0e6a904c25a69c3e10a66e67693a1e6d9e14499c

Comment by Githook User [ 02/Nov/15 ]

Author:

{u'username': u'bjori', u'name': u'Hannes Magnusson', u'email': u'bjori@php.net'}

Message: CDRIVER-669: Disable certain warnings on certain platforms
Branch: master
https://github.com/mongodb/mongo-c-driver/commit/9bf35f456fa5ec9efd877b7d34646289c732ef3c

Comment by Githook User [ 02/Nov/15 ]

Author:

{u'username': u'bjori', u'name': u'Hannes Magnusson', u'email': u'bjori@php.net'}

Message: CDRIVER-669: redefinition of typedef 'mongoc_topology_t' is a C11 feature [-Wtypedef-redefinition]
Branch: master
https://github.com/mongodb/mongo-c-driver/commit/a657bd29d5d567583c39c1633f370a9288d83e11

Comment by Githook User [ 02/Nov/15 ]

Author:

{u'username': u'bjori', u'name': u'Hannes Magnusson', u'email': u'bjori@php.net'}

Message: CDRIVER-669: Fix -wshadow warning
Branch: master
https://github.com/mongodb/mongo-c-driver/commit/f265d1dd5c7b9da140534a60313526934e1f99ee

Comment by Hannes Magnusson [ 02/Nov/15 ]

That really didn't work out as I had hoped on OSX and Solaris

Comment by Githook User [ 28/Oct/15 ]

Author:

{u'username': u'bjori', u'name': u'Hannes Magnusson', u'email': u'bjori@php.net'}

Message: CDRIVER-669 & CDRIVER-958: Add -Werror into CFLAGS & improve configure flags

This way we mark builds as failed on new warnings rather then let them
pile up.
Branch: master
https://github.com/mongodb/mongo-c-driver/commit/de54db303649ade993240c4b7b3b0feaa750882f

Generated at Wed Feb 07 21:10:14 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.