alecs/plasma-framework
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Only include QCA includes where needed

Browse Source 
There's no need to make all the framework look for the QCA includes while
they're only being used by the remote part. It could possibly be made more
specific, but I don't think those are yet being used anyway.
This commit is contained in:
Aleix Pol 2013-11-11 18:29:00 +01:00
parent 1d3da96d88
commit 392fa639b6
2 changed files with 5 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
CMakeLists.txt
View File

@ -126,12 +126,6 @@ add_definitions(-DQT_NO_URL_CAST_FROM_STRING)
remove_definitions(-DQT_NO_CAST_FROM_ASCII -DQT_STRICT_ITERATORS -DQT_NO_CAST_FROM_BYTEARRAY -DQT_NO_KEYWORDS) remove_definitions(-DQT_NO_CAST_FROM_ASCII -DQT_STRICT_ITERATORS -DQT_NO_CAST_FROM_BYTEARRAY -DQT_NO_KEYWORDS)
if(QCA2_FOUND)
# include_directories(
# ${QCA2_INCLUDE_DIR}
# )
endif()
include(PlasmaMacros.cmake) include(PlasmaMacros.cmake)
################# configure checks and create the configured files ################# ################# configure checks and create the configured files #################

5
src/plasma/CMakeLists.txt
View File

@ -1,3 +1,8 @@
if(QCA2_FOUND)
include_directories(
${QCA2_INCLUDE_DIR}
)
endif()
# This option should be removed, or moved down as far as possible. # This option should be removed, or moved down as far as possible.
# That means porting the existing frameworks to the CMake automoc # That means porting the existing frameworks to the CMake automoc