Merge lp:~hjd/widelands/scan-build-cmake into lp:widelands
Proposed by
Hans Joachim Desserud
Status: | Merged |
---|---|
Merged at revision: | 6631 |
Proposed branch: | lp:~hjd/widelands/scan-build-cmake |
Merge into: | lp:widelands |
Diff against target: |
32 lines (+1/-15) 1 file modified
utils/scan-build.sh (+1/-15) |
To merge this branch: | bzr merge lp:~hjd/widelands/scan-build-cmake |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Widelands Developers | Pending | ||
Review via email: mp+175269@code.launchpad.net |
Description of the change
Yesterday I learnt that it is sufficient to run `scan-build cmake` to have it replace the compiler in the necessary places. Thus there is no need to find and add the script manually. This should also expand cross-platform support quite a bit.
PS.
SirVer: I think this might work on OS X too now, if you want to test it. :)
To post a comment you must log in.
Thanks! Merged.
I didn't try running scan-build. There are currently so much stuff going on with Widelands that I am not generating extra work for myself if I can avoid it :).