[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-pim
Subject:    Re: [Kde-pim] Boost vs cmake 2.8.8 vs kdepimlibs master
From:       Alexander Neundorf <neundorf () kde ! org>
Date:       2012-12-17 17:37:59
Message-ID: 201212171838.00115.neundorf () kde ! org
[Download RAW message or body]

On Sunday 16 December 2012, Antonis Tsiapaliokas wrote:
> Hello,
> 
> > Attached, can somebody give it a try ?
> > 
> > Alex
> 
> I have test the attached patch with 2.8.8 cmake and it doesn't work.
> With the 2.8.9 cmake, the issues is solved, without the attached patch
> needed.

The attached patch should work. It's not nice, but should work (works at least 
for me).

Alex

["fail_manually_if_boost_is_missing.patch" (text/x-patch)]

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 0aea59c..56f11a0 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -67,7 +67,12 @@ configure_file(${CMAKE_SOURCE_DIR}/CTestCustom.cmake \
${CMAKE_BINARY_DIR}/CTestCu  
 set(Boost_MINIMUM_VERSION "1.34.0")
 find_package(Boost ${Boost_MINIMUM_VERSION} COMPONENTS graph)
-set_package_properties(Boost PROPERTIES DESCRIPTION "Boost C++ Libraries" URL \
"http://www.boost.org" TYPE REQUIRED PURPOSE "Boost must include the boost-graph \
library") +# workaround for a bug in cmake 2.8.8: FindBoost together with \
FeatureSummary fails if Boost +# is required even if boost has been actually found. \
So don't make it required and fail manually. +if(NOT Boost_FOUND)
+  message(FATAL_ERROR "Could not find Boost")
+endif()
+set_package_properties(Boost PROPERTIES DESCRIPTION "Boost C++ Libraries" URL \
"http://www.boost.org" TYPE OPTIONAL PURPOSE "Boost must include the boost-graph \
library")  
 #FindGpgme.cmake already handles the log message but we must ensure it is required.
 find_package(Gpgme REQUIRED)



_______________________________________________
KDE PIM mailing list kde-pim@kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/

[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic