Author: brane
Date: Tue Jun 26 08:55:50 2018
New Revision: 1834407

URL: http://svn.apache.org/viewvc?rev=1834407&view=rev
Log:
* CMakeLists.txt: Using PREFIX as a global variable turns out not to be
   a good idea; CMake generates a bunch of warnings because its internals
   use PREFIX for compiler feature checks, etc.

Modified:
    serf/trunk/CMakeLists.txt

Modified: serf/trunk/CMakeLists.txt
URL: 
http://svn.apache.org/viewvc/serf/trunk/CMakeLists.txt?rev=1834407&r1=1834406&r2=1834407&view=diff
==============================================================================
--- serf/trunk/CMakeLists.txt (original)
+++ serf/trunk/CMakeLists.txt Tue Jun 26 08:55:50 2018
@@ -30,7 +30,6 @@ message(WARNING
 
 
 # Build options
-option(PREFIX "Installation root directory" OFF)
 option(LIBDIR "Indstall directory for architecture-dependent libraries" OFF)
 option(APR "Path to APR's install area" OFF)
 option(APU "Path to APR-Util's install area" OFF)
@@ -184,16 +183,8 @@ if(CMAKE_COMPILER_IS_GNUCC OR (CMAKE_C_C
 endif()
 
 # Process other build options
-if(PREFIX)
-  set(CMAKE_INSTALL_PREFIX ${PREFIX})
-else()
-  set(PREFIX ${CMAKE_INSTALL_PREFIX})
-endif()
-
 if(LIBDIR)
   message(WARNING "option LIBDIR is not implemented yet")
-else()
-  set(LIBDIR "${PREFIX}/lib")
 endif()
 
 if(DEBUG)
@@ -245,6 +236,10 @@ endif()
 if(NOT WINDOWS)
   # Use a separate variable scope for the substitutions in serf.pc.in.
   function(make_pkgconfig)
+    set(PREFIX ${CMAKE_INSTALL_PREFIX})
+    if(NOT LIBDIR)
+      set(LIBDIR "${PREIFX}/lib")
+    endif()
     set(VERSION ${SERF_VERSION})
     set(MAJOR ${SERF_MAJOR_VERSION})
     set(INCLUDE_SUBDIR "serf-${SERF_MAJOR_VERSION}")


Reply via email to