Merge lp:~nha/widelands/release-warnings into lp:widelands
Proposed by
Nicolai Hähnle
Status: | Merged |
---|---|
Merged at revision: | 6546 |
Proposed branch: | lp:~nha/widelands/release-warnings |
Merge into: | lp:widelands |
Diff against target: |
127 lines (+41/-44) 1 file modified
CMakeLists.txt (+41/-44) |
To merge this branch: | bzr merge lp:~nha/widelands/release-warnings |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Jens Beyer | Approve | ||
Widelands Developers | Pending | ||
Review via email: mp+153693@code.launchpad.net |
Description of the change
This should be straightforward: make sure we compile with warnings also in Release mode. Note that the MSVC path is unaffected, since it happens further down in the CMakeLists.txt, so I think/hope that this does not break anything.
Still, some comment from the cmake gurus would be appreciated ;-)
To post a comment you must log in.
Jens, you are most qualified to comment on this.