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

[mojca at macports.org]

Revision: 121112
          https://trac.macports.org/changeset/121112
Author:   mojca at macports.org
Date:     2014-06-17 13:16:44 -0700 (Tue, 17 Jun 2014)
Log Message:
-----------
cmake-1.0.tcl: remove -I${prefix}/include from C(XX)FLAGS (#42872)

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	2014-06-17 19:15:43 UTC (rev 121111)
+++ trunk/dports/_resources/port1.0/group/cmake-1.0.tcl	2014-06-17 20:16:44 UTC (rev 121112)
@@ -81,12 +81,9 @@
     # The environment variable CPPFLAGS is not considered by CMake.
     # (CMake upstream ticket #12928 "CMake silently ignores CPPFLAGS"
     # <http://www.cmake.org/Bug/view.php?id=12928>).
-    # Thus, we have to add them manually to the CFLAGS and CXXFLAGS in the
-    # pre-configure phase.
-    if {${configure.cppflags} ne ""} {
-        configure.cflags-append ${configure.cppflags}
-        configure.cxxflags-append ${configure.cppflags}
-    }
+    #
+    # But adding -I${prefix}/include to CFLAGS/CXXFLAGS is a bad idea.
+    # If any other flags are needed, we need to add them.
 
     # In addition, CMake provides build-type-specific flags for
     # Release (-O3 -DNDEBUG), Debug (-g), MinSizeRel (-Os -DNDEBUG), and
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.macosforge.org/pipermail/macports-changes/attachments/20140617/b28b19ae/attachment.html>