From a97c23d173b7492f9b6125341a413456d171c53b Mon Sep 17 00:00:00 2001 From: David Sansome Date: Sun, 19 Dec 2010 15:10:26 +0000 Subject: [PATCH] Fix liblastfm detection. Protip: variables can't be used before they're set. --- CMakeLists.txt | 11 ++++------- src/CMakeLists.txt | 4 ++++ 2 files changed, 8 insertions(+), 7 deletions(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index e82821f16..ca7bf58fa 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -56,10 +56,6 @@ endif (WIN32) find_library(LASTFM_LIBRARIES lastfm) find_path(LASTFM_INCLUDE_DIRS lastfm/ws.h) -if (ENABLE_LIBLASTFM AND LASTFM_LIBRARIES AND LASTFM_INCLUDE_DIRS) - set(HAVE_LIBLASTFM ON) -endif (ENABLE_LIBLASTFM AND LASTFM_LIBRARIES AND LASTFM_INCLUDE_DIRS) - if (APPLE) find_library(GROWL Growl) option(ENABLE_SPARKLE "Sparkle updating" ON) @@ -106,9 +102,6 @@ include_directories(${GSTREAMER_INCLUDE_DIRS}) include_directories(${GLIB_INCLUDE_DIRS}) include_directories(${GLIBCONFIG_INCLUDE_DIRS}) include_directories(${LIBXML_INCLUDE_DIRS}) -if (HAVE_LIBLASTFM) - include_directories(${LASTFM_INCLUDE_DIRS}) -endif (HAVE_LIBLASTFM) include_directories("3rdparty/universalchardet") # Remove GLU and GL from the link line - they're not really required @@ -164,6 +157,10 @@ if(ENABLE_SOUNDMENU AND INDICATEQT_FOUND) set(HAVE_LIBINDICATE ON) endif(ENABLE_SOUNDMENU AND INDICATEQT_FOUND) +if(ENABLE_LIBLASTFM AND LASTFM_LIBRARIES AND LASTFM_INCLUDE_DIRS) + set(HAVE_LIBLASTFM ON) +endif(ENABLE_LIBLASTFM AND LASTFM_LIBRARIES AND LASTFM_INCLUDE_DIRS) + if(ENABLE_VISUALISATIONS) # When/if upstream accepts our patches then these options can be used to link # to system installed projectM instead. diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt index 3f592bd83..cab18065e 100644 --- a/src/CMakeLists.txt +++ b/src/CMakeLists.txt @@ -23,6 +23,10 @@ if(HAVE_LIBINDICATE) include_directories(${INDICATEQT_INCLUDE_DIRS}) endif(HAVE_LIBINDICATE) +if(HAVE_LIBLASTFM) + include_directories(${LASTFM_INCLUDE_DIRS}) +endif(HAVE_LIBLASTFM) + cmake_policy(SET CMP0011 NEW) include(../cmake/AddEngine.cmake) include(../cmake/ParseArguments.cmake)