commit 0050049e450ee3a815710c84d724b1575e496800
author David Wagner <david.wagner@intel.com> Tue Feb 18 10:21:01 2014 +0100
committer David Wagner <david.wagner@intel.com> Tue Feb 18 10:21:01 2014 +0100
tree a38328b496ae68e4a5a732694fff10a8194cbf0b
parent f50298799d1d73c0c40940ca5e8cb71a6b28f798

CMake: beautification, fix some comments

Signed-off-by: David Wagner <david.wagner@intel.com>

CMakeLists.txt

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 283f452..b12c5f6 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -8,7 +8,7 @@
 
 project(parameter-framework)
 
-set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Werror -Wall -Wextra") # -Wno-unused-parameter
+set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Werror -Wall -Wextra")
 
 add_subdirectory(xmlserializer)
 add_subdirectory(parameter)

parameter/CMakeLists.txt

diff --git a/parameter/CMakeLists.txt b/parameter/CMakeLists.txt
index 0d962ee..1bbebfb 100644
--- a/parameter/CMakeLists.txt
+++ b/parameter/CMakeLists.txt
@@ -168,10 +168,11 @@
     RuleParser.cpp
     EnumValuePair.cpp)
 
-include_directories("include")
-include_directories("${PROJECT_SOURCE_DIR}/xmlserializer")
-include_directories("${PROJECT_SOURCE_DIR}/utility")
-include_directories("${PROJECT_SOURCE_DIR}/remote-processor")
+include_directories(
+    include
+    "${PROJECT_SOURCE_DIR}/xmlserializer"
+    "${PROJECT_SOURCE_DIR}/utility"
+    "${PROJECT_SOURCE_DIR}/remote-processor")
 
 # No need to link with libremote-processor: it is accessed via dlopen()
 target_link_libraries(parameter xmlserializer pfw_utility)

remote-process/CMakeLists.txt

diff --git a/remote-process/CMakeLists.txt b/remote-process/CMakeLists.txt
index 80fd177..0cf477a 100644
--- a/remote-process/CMakeLists.txt
+++ b/remote-process/CMakeLists.txt
@@ -2,8 +2,9 @@
 
 # TODO: separate remote-processor's includes in half (public/private)
 # And use only public headers here
-include_directories("${PROJECT_SOURCE_DIR}/remote-processor")
-include_directories("${PROJECT_SOURCE_DIR}/utility")
+include_directories(
+    "${PROJECT_SOURCE_DIR}/remote-processor"
+    "${PROJECT_SOURCE_DIR}/utility")
 
 target_link_libraries(remote-process remote-processor pfw_utility)
 

test/test-platform/CMakeLists.txt

diff --git a/test/test-platform/CMakeLists.txt b/test/test-platform/CMakeLists.txt
index afc7dc5..c379171 100644
--- a/test/test-platform/CMakeLists.txt
+++ b/test/test-platform/CMakeLists.txt
@@ -2,12 +2,13 @@
     main.cpp
     TestPlatform.cpp)
 
-# FIXME
+# FIXME: Supress the need for the -Wno-unused-parameter
 set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-unused-parameter")
 
-include_directories("${PROJECT_SOURCE_DIR}/parameter/include")
-include_directories("${PROJECT_SOURCE_DIR}/remote-processor")
-include_directories("${PROJECT_SOURCE_DIR}/utility")
+include_directories(
+    "${PROJECT_SOURCE_DIR}/parameter/include"
+    "${PROJECT_SOURCE_DIR}/remote-processor"
+    "${PROJECT_SOURCE_DIR}/utility")
 
 target_link_libraries(test-platform parameter remote-processor)
 

utility/CMakeLists.txt

diff --git a/utility/CMakeLists.txt b/utility/CMakeLists.txt
index ac4d5b7..698c781 100644
--- a/utility/CMakeLists.txt
+++ b/utility/CMakeLists.txt
@@ -3,5 +3,5 @@
     Utility.cpp
     NaiveTokenizer.cpp)
 
-# Needed for linking libpfw_utility against shared libraries e.g. libparameter)
+# '-fPIC' needed for linking against shared libraries (e.g. libparameter)
 set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -fPIC")

xmlserializer/CMakeLists.txt

diff --git a/xmlserializer/CMakeLists.txt b/xmlserializer/CMakeLists.txt
index c4358b1..77b4fb3 100644
--- a/xmlserializer/CMakeLists.txt
+++ b/xmlserializer/CMakeLists.txt
@@ -11,7 +11,7 @@
     XmlStringDocSource.cpp)
 
 include(FindLibXml2)
-# TODO: check for the "XInclude" feature in libxml2
+# TODO: check for the "XInclude" feature in libxml2 (how to ?)
 include_directories(${LIBXML2_INCLUDE_DIR})
 target_link_libraries(xmlserializer ${LIBXML2_LIBRARIES})