~ubuntu-branches/ubuntu/precise/networkmanagement/precise

From: Mathieu Trudel-Lapierre <mathieu-tl@ubuntu.com>

Subject: Workaround issues in version comparisons with NM's large versions.

Bug-Ubuntu: https://bugs.launchpad.net/bugs/838099

 

The use of rather large version numbers in NM (e.g. 0.8.9997) for pre-releases

of NM 0.9 caused unexpected issues with version comparisons when we actually

made it to 0.9.0: the comparisons made in KDE's KDE_MAKE_VERSION macro and

CMake's macro_ensure_version cause 0.9.0 to be evaluated as "smaller" than

a version number of 0.8.996, for instance.

 

In KDE_MAKE_VERSION, this is because shifting bits around means a maximum third

number of 255 (otherwise it "leaks" forward). In other words, 0.8.255 is the

highest version number to be evaluated as smaller than 0.9.0, 0.8.256 will be

evaluated as equal, and 0.8.257 and on will be larger than 0.9.0.

 

Here we work around these issues by specifying that the minimum version is

to be 0.9.0. Also make sure the solid bits require the same version for NM libs.

 

Index: networkmanagement/CMakeLists.txt

===================================================================

--- networkmanagement.orig/CMakeLists.txt       2011-09-01 10:17:38.862607614 -0400

+++ networkmanagement/CMakeLists.txt    2011-09-01 10:26:02.232608571 -0400

@@ -1,7 +1,7 @@

 project(plasma-networkmanagement)

 

 option(INSTALL_KNM_AUTOSTART "Install the autostart file for knetworkmanager. Warning: This prevents the plasmoid from working properly")

-set (MINIMUM_NM_VERSION_REQUIRED "0.8.9997")

+set (MINIMUM_NM_VERSION_REQUIRED "0.9.0")

 

 find_package(KDE4 REQUIRED)

 include (KDE4Defaults)

Index: networkmanagement/solidcontrolfuture/solid/CMakeLists.txt

===================================================================

--- networkmanagement.orig/solidcontrolfuture/solid/CMakeLists.txt      2011-09-01 10:26:36.002608635 -0400

+++ networkmanagement/solidcontrolfuture/solid/CMakeLists.txt   2011-09-01 10:32:26.022609302 -0400

@@ -3,7 +3,7 @@

     

 set(HAVE_NETWORKMANAGER FALSE)

 if(NETWORKMANAGER_FOUND)

-  macro_ensure_version("0.8.996" ${NETWORKMANAGER_VERSION} NM_0_9)

+  macro_ensure_version(${MINIMUM_NM_VERSION_REQUIRED} ${NETWORKMANAGER_VERSION} NM_0_9)

   if (NM_0_9)

     message(STATUS "Will build networkmanager-0.9")

     set(HAVE_NETWORKMANAGER TRUE)