[112838] trunk/dports/_resources/port1.0/group/cmake-1.0.tcl

jeremyhu at macports.org jeremyhu at macports.org
Sat Nov 2 10:15:57 PDT 2013


Revision: 112838
          https://trac.macports.org/changeset/112838
Author:   jeremyhu at macports.org
Date:     2013-11-02 10:15:57 -0700 (Sat, 02 Nov 2013)
Log Message:
-----------
Consider the configure.cflags and configure.cxxflags for +debug variant of the CMake PortsGroup (#40648).

The CMAKE_BUILD_TYPE "Debug" uses CMAKE_C_FLAGS_DEBUG and CMAKE_CXX_FLAGS_DEBUG to specify the 
compiler flags. These variables are set to "-g" plus the respective configure.cflags and 
configure.cxxflags. Be aware that configure.cflags and configure.cxxflags contain 
configure.optflags by default, which are not deleted for the Debug build. The port author has to 
set configure.optflags to "-O0" to turn off optimization.

Modified Paths:
--------------
    trunk/dports/_resources/port1.0/group/cmake-1.0.tcl

Modified: trunk/dports/_resources/port1.0/group/cmake-1.0.tcl
===================================================================
--- trunk/dports/_resources/port1.0/group/cmake-1.0.tcl	2013-11-02 17:14:29 UTC (rev 112837)
+++ trunk/dports/_resources/port1.0/group/cmake-1.0.tcl	2013-11-02 17:15:57 UTC (rev 112838)
@@ -138,6 +138,14 @@
 variant debug description "Enable debug binaries" {
     configure.args-delete   -DCMAKE_BUILD_TYPE=Release
     configure.args-append   -DCMAKE_BUILD_TYPE=Debug
+    # Consider the configure.cflags and configure.cxxflags for Debug builds.
+    # Attention, they contain configure.optflags by default!
+    if {${configure.cflags} != ""} {
+        configure.args-append -DCMAKE_C_FLAGS_DEBUG="-g ${configure.cflags}"
+    }
+    if {${configure.cxxflags} != ""} {
+        configure.args-append -DCMAKE_CXX_FLAGS_DEBUG="-g ${configure.cxxflags}"
+    }
 }
 
 # cmake doesn't like --enable-debug, so in case a portfile sets
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.macosforge.org/pipermail/macports-changes/attachments/20131102/d7cce8e5/attachment-0001.html>


More information about the macports-changes mailing list