Merge lp:~aacid/unity-api/cmake_warning-- into lp:unity-api

Proposed by Albert Astals Cid
Status: Merged
Approved by: MichaƂ Sawicz
Approved revision: 111
Merged at revision: 111
Proposed branch: lp:~aacid/unity-api/cmake_warning--
Merge into: lp:unity-api
Diff against target: 17 lines (+2/-1)
1 file modified
CMakeLists.txt (+2/-1)
To merge this branch: bzr merge lp:~aacid/unity-api/cmake_warning--
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Michi Henning Pending
Review via email: mp+194148@code.launchpad.net

Commit message

Avoid cmake warnings

    CMake Warning (dev) in CMakeLists.txt:
      Syntax Warning in cmake code at

        /home/tsdgeos_work/phablet/unity-api/unity-api/CMakeLists.txt:53:88

      Argument not separated from preceding token by whitespace.
    This warning is for project developers. Use -Wno-dev to suppress it.

    CMake Warning (dev) in CMakeLists.txt:
      Syntax Warning in cmake code at

        /home/tsdgeos_work/phablet/unity-api/unity-api/CMakeLists.txt:53:107

      Argument not separated from preceding token by whitespace.
    This warning is for project developers. Use -Wno-dev to suppress it.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Michi Henning (michihenning) wrote :

I haven't seen this before, at least not with cmake 2.8.11.2.

What version of cmake is this happening with?

Revision history for this message
Albert Astals Cid (aacid) wrote :

cmake 2.8.12

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) :
review: Approve (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Autolanding.
Approved revid is not set in launchpad. This is most likely a launchpad issue and re-approve should fix it. There is also a chance (although a very small one) this is a permission problem of the ps-jenkins bot.
http://jenkins.qa.ubuntu.com/job/unity-api-autolanding/64/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-api-trusty-amd64-autolanding/3
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-api-trusty-armhf-autolanding/3
        deb: http://jenkins.qa.ubuntu.com/job/unity-api-trusty-armhf-autolanding/3/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-api-trusty-i386-autolanding/3

review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'CMakeLists.txt'
2--- CMakeLists.txt 2013-10-18 02:33:12 +0000
3+++ CMakeLists.txt 2013-11-06 14:26:29 +0000
4@@ -50,11 +50,12 @@
5 find_program(CPPCHECK_COMMAND NAMES cppcheck)
6 if (CPPCHECK_COMMAND)
7 set(CPPCHECK_COMMAND_OPTIONS --check-config --inline-suppr --enable=all -q --error-exitcode=2)
8- set(CPPCHECK_COMMAND_OPTIONS ${CPPCHECK_COMMAND_OPTIONS} --template '{file}({line}): {severity} ({id}): {message}')
9+ set(CPPCHECK_COMMAND_OPTIONS ${CPPCHECK_COMMAND_OPTIONS} --template "{file}({line}): {severity} ({id}): {message}")
10 add_custom_target(cppcheck COMMAND ${CPPCHECK_COMMAND} ${CPPCHECK_COMMAND_OPTIONS}
11 ${CMAKE_SOURCE_DIR}/src
12 ${CMAKE_SOURCE_DIR}/test
13 ${CMAKE_BINARY_DIR}/test
14+ VERBATIM
15 )
16 else()
17 message(WARNING "Cannot find cppcheck: cppcheck target will not be available")

Subscribers

People subscribed via source and target branches

to all changes: