From owner-freebsd-ports-bugs@FreeBSD.ORG Tue May 1 14:20:13 2012 Return-Path: Delivered-To: freebsd-ports-bugs@hub.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 65BC11065676 for ; Tue, 1 May 2012 14:20:13 +0000 (UTC) (envelope-from gnats@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2001:4f8:fff6::28]) by mx1.freebsd.org (Postfix) with ESMTP id 3A5E98FC08 for ; Tue, 1 May 2012 14:20:13 +0000 (UTC) Received: from freefall.freebsd.org (localhost [127.0.0.1]) by freefall.freebsd.org (8.14.5/8.14.5) with ESMTP id q41EKDLe020468 for ; Tue, 1 May 2012 14:20:13 GMT (envelope-from gnats@freefall.freebsd.org) Received: (from gnats@localhost) by freefall.freebsd.org (8.14.5/8.14.5/Submit) id q41EKDm8020467; Tue, 1 May 2012 14:20:13 GMT (envelope-from gnats) Resent-Date: Tue, 1 May 2012 14:20:13 GMT Resent-Message-Id: <201205011420.q41EKDm8020467@freefall.freebsd.org> Resent-From: FreeBSD-gnats-submit@FreeBSD.org (GNATS Filer) Resent-To: freebsd-ports-bugs@FreeBSD.org Resent-Reply-To: FreeBSD-gnats-submit@FreeBSD.org, Raphael Kubo da Costa Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id AEBAE106566B; Tue, 1 May 2012 14:18:40 +0000 (UTC) (envelope-from kubito@gmail.com) Received: from mail-yx0-f182.google.com (mail-yx0-f182.google.com [209.85.213.182]) by mx1.freebsd.org (Postfix) with ESMTP id 3CBD08FC0A; Tue, 1 May 2012 14:18:40 +0000 (UTC) Received: by yenl9 with SMTP id l9so2256568yen.13 for ; Tue, 01 May 2012 07:18:33 -0700 (PDT) Received: by 10.236.191.233 with SMTP id g69mr4201291yhn.20.1335881913868; Tue, 01 May 2012 07:18:33 -0700 (PDT) Received: from localhost ([201.82.202.123]) by mx.google.com with ESMTPS id d63sm90771816yhh.17.2012.05.01.07.18.30 (version=TLSv1/SSLv3 cipher=OTHER); Tue, 01 May 2012 07:18:32 -0700 (PDT) Message-Id: <4f9ff0b8.6373ec0a.5b1a.ffff96d3@mx.google.com> Date: Tue, 01 May 2012 07:18:32 -0700 (PDT) From: Raphael Kubo da Costa Sender: Raphael Kubo da Costa To: FreeBSD-gnats-submit@FreeBSD.org X-Send-Pr-Version: 3.113 Cc: pav@FreeBSD.org, amdmi3@FreeBSD.org Subject: ports/167488: [PATCH] graphics/lprof-devel: Make the port build with CMake 2.8.8 X-BeenThere: freebsd-ports-bugs@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Ports bug reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 01 May 2012 14:20:13 -0000 >Number: 167488 >Category: ports >Synopsis: [PATCH] graphics/lprof-devel: Make the port build with CMake 2.8.8 >Confidential: no >Severity: non-critical >Priority: low >Responsible: freebsd-ports-bugs >State: open >Quarter: >Keywords: >Date-Required: >Class: change-request >Submitter-Id: current-users >Arrival-Date: Tue May 01 14:20:12 UTC 2012 >Closed-Date: >Last-Modified: >Originator: Raphael Kubo da Costa >Release: FreeBSD 9.0-STABLE amd64 >Organization: FreeBSD Project >Environment: System: FreeBSD gibbon 9.0-STABLE FreeBSD 9.0-STABLE #41: Fri Apr 27 10:40:12 BRT 2012 >Description: Note: this PR currently blocks ports/167140, which updates CMake to 2.8.8. lprof-devel used SET_DIRECTORY_PROPERTIES(PROPERTIES INCLUDE_DIRECTORIES) instead of the more standard INCLUDE_DIRECTORIES() call to add directories to a target's include path. This only worked by chance, and a change in CMake 2.8.8 breaks this altogether: The attached patch replaces the current calls in a way that hopefully achieves the same results. Another possibility is to just mark the port as BROKEN: looking at the build system code, there are many other ugly things it does, plus upstream seems to have been dead for at least 4 years and no port depends on it. Added file(s): - files/patch-src-3rdparty-vigra-CMakeLists.txt - files/patch-src-ICCtoIT8-CMakeLists.txt - files/patch-src-lcms-CMakeLists.txt - files/patch-src-liblprof-CMakeLists.txt Port maintainer (amdmi3@FreeBSD.org) is cc'd. Generated with FreeBSD Port Tools 0.99_6 (mode: change, diff: CVS) >How-To-Repeat: >Fix: --- lprof-devel-20080514_8.patch begins here --- Index: files/patch-src-3rdparty-vigra-CMakeLists.txt =================================================================== RCS file: files/patch-src-3rdparty-vigra-CMakeLists.txt diff -N files/patch-src-3rdparty-vigra-CMakeLists.txt --- /dev/null 1 Jan 1970 00:00:00 -0000 +++ files/patch-src-3rdparty-vigra-CMakeLists.txt 1 May 2012 14:12:49 -0000 @@ -0,0 +1,15 @@ +--- src/3rdparty/vigra/CMakeLists.txt 2008-02-19 18:01:35.000000000 -0300 ++++ src/3rdparty/vigra/CMakeLists.txt 2012-05-01 10:38:31.000000000 -0300 +@@ -1,6 +1,9 @@ +-SET_DIRECTORY_PROPERTIES( +- PROPERTIEs +- INCLUDE_DIRECTORIES "${CMAKE_CURRENT_SOURCE_DIR}/include;${PNG_INCLUDE_DIR};${JPEG_INCLUDE_DIR};${TIFF_INCLUDE_DIR}" ++INCLUDE_DIRECTORIES( ++ BEFORE ++ "${CMAKE_CURRENT_SOURCE_DIR}/include" ++ "${PNG_INCLUDE_DIR}" ++ "${JPEG_INCLUDE_DIR}" ++ "${TIFF_INCLUDE_DIR}" + ) + + SET(IMPEX_SOURCES Index: files/patch-src-CMakeLists.txt =================================================================== RCS file: /home/pcvs/ports/graphics/lprof-devel/files/patch-src-CMakeLists.txt,v retrieving revision 1.1 diff -u -u -r1.1 patch-src-CMakeLists.txt --- files/patch-src-CMakeLists.txt 5 Jun 2008 12:39:13 -0000 1.1 +++ files/patch-src-CMakeLists.txt 1 May 2012 14:12:49 -0000 @@ -1,11 +1,32 @@ ---- src/CMakeLists.txt.orig 2008-02-20 00:01:32.000000000 +0300 -+++ src/CMakeLists.txt 2008-05-24 05:13:22.000000000 +0400 -@@ -129,7 +129,7 @@ +--- src/CMakeLists.txt 2008-02-19 18:01:32.000000000 -0300 ++++ src/CMakeLists.txt 2012-05-01 11:00:11.000000000 -0300 +@@ -129,7 +129,7 @@ ENDFOREACH(lprof_inc_dir) # Normally these are set for us. Since we're overriding the normal include path we've got to add them ourselves, but it's # probably a good idea to be explicit about what we use anyway. -SET(lprof_qt_incs "${QT_INCLUDE_DIR};${QT_QT_INCLUDE_DIR};${QT_QTCORE_INCLUDE_DIR};${QT_QTGUI_INCLUDE_DIR};${QT_QTSVG_INCLUDE_DIR};${QT_QT3SUPPORT_INCLUDE_DIR};${QT_QTASSISTANT_INCLUDE_DIR}") -+SET(lprof_qt_incs "${QT_QT_INCLUDE_DIR};${QT_QTCORE_INCLUDE_DIR};${QT_QTGUI_INCLUDE_DIR};${QT_QT3SUPPORT_INCLUDE_DIR};${QT_QTASSISTANT_INCLUDE_DIR};${QT_INCLUDE_DIR}") ++SET(lprof_qt_incs "${QT_QT_INCLUDE_DIR};${QT_QTCORE_INCLUDE_DIR};${QT_QTGUI_INCLUDE_DIR};${QT_QTSVG_INCLUDE_DIR};${QT_QT3SUPPORT_INCLUDE_DIR};${QT_QTASSISTANT_INCLUDE_DIR}") IF(NOT LIBVIGRAIMPEX_FOUND) +@@ -143,9 +143,18 @@ ELSE(NOT LIBVIGRAIMPEX_FOUND) + # Static/nonstatic flag should've been added by find script + ENDIF(NOT LIBVIGRAIMPEX_FOUND) + +-SET_DIRECTORY_PROPERTIES( +- PROPERTIES +- INCLUDE_DIRECTORIES "${lprof_inc_dirs};${lprof_qt_incs};${ARGYLL_INCLUDE_DIRS};${impex_include};${LIBUSB_INCLUDE_DIR};${X11_INCLUDE_DIR};${ZLIB_INCLUDE_DIR};${JPEG_INCLUDE_DIR};${TIFF_INCLUDE_DIR};${CMAKE_CURRENT_BINARY_DIR};${WINDDK_INCLUDE_DIR}" ++INCLUDE_DIRECTORIES( ++ BEFORE ++ "${lprof_inc_dirs}" ++ "${lprof_qt_incs}" ++ "${ARGYLL_INCLUDE_DIRS}" ++ "${impex_include}" ++ "${LIBUSB_INCLUDE_DIR}" ++ "${X11_INCLUDE_DIR}" ++ "${ZLIB_INCLUDE_DIR}" ++ "${JPEG_INCLUDE_DIR}" ++ "${TIFF_INCLUDE_DIR}" ++ "${CMAKE_CURRENT_BINARY_DIR}" + ) + Index: files/patch-src-ICCtoIT8-CMakeLists.txt =================================================================== RCS file: files/patch-src-ICCtoIT8-CMakeLists.txt diff -N files/patch-src-ICCtoIT8-CMakeLists.txt --- /dev/null 1 Jan 1970 00:00:00 -0000 +++ files/patch-src-ICCtoIT8-CMakeLists.txt 1 May 2012 14:12:49 -0000 @@ -0,0 +1,17 @@ +--- src/ICCtoIT8/CMakeLists.txt 2008-02-19 18:02:10.000000000 -0300 ++++ src/ICCtoIT8/CMakeLists.txt 2012-05-01 10:37:22.000000000 -0300 +@@ -7,9 +7,11 @@ ADD_EXECUTABLE(icc2it8 + ${ICC2IT8_SOURCES} + ) + +-SET_DIRECTORY_PROPERTIES( +- PROPERTIES +- INCLUDE_DIRECTORIES "${CMAKE_SOURCE_DIR}/src/lcms/include;${CMAKE_SOURCE_DIR}/src/liblprof;${ARGYLL_INCLUDE_DIRS}" ++INCLUDE_DIRECTORIES( ++ BEFORE ++ "${CMAKE_SOURCE_DIR}/src/lcms/include" ++ "${CMAKE_SOURCE_DIR}/src/liblprof" ++ "${ARGYLL_INCLUDE_DIRS}" + ) + + TARGET_LINK_LIBRARIES(icc2it8 liblprof lcmslocal ${COMMON_PLATFORM_LIBS}) Index: files/patch-src-argyll-CMakeLists.txt =================================================================== RCS file: /home/pcvs/ports/graphics/lprof-devel/files/patch-src-argyll-CMakeLists.txt,v retrieving revision 1.1 diff -u -u -r1.1 patch-src-argyll-CMakeLists.txt --- files/patch-src-argyll-CMakeLists.txt 8 Oct 2008 15:00:04 -0000 1.1 +++ files/patch-src-argyll-CMakeLists.txt 1 May 2012 14:12:49 -0000 @@ -1,5 +1,20 @@ ---- src/argyll/CMakeLists.txt.orig 2008-09-08 00:27:28.000000000 +0400 -+++ src/argyll/CMakeLists.txt 2008-09-08 00:27:28.000000000 +0400 +--- src/argyll/CMakeLists.txt 2008-02-19 18:02:10.000000000 -0300 ++++ src/argyll/CMakeLists.txt 2012-05-01 11:02:19.000000000 -0300 +@@ -5,9 +5,11 @@ ENDFOREACH(argyll_inc_dir) + + SET(ARGYLL_INCLUDE_DIRS "${ARGYLL_INCLUDE_DIRS}" CACHE PATH "Argyll cms include directories" FORCE) + +-SET_DIRECTORY_PROPERTIES( +- PROPERTIES +- INCLUDE_DIRECTORIES "${ARGYLL_INCLUDE_DIRS};${LIBUSB_INCLUDE_DIR};${CMAKE_SOURCE_DIR}/src/moncalqt;${WINDDK_INCLUDE_DIR}" ++INCLUDE_DIRECTORIES( ++ BEFORE ++ "${ARGYLL_INCLUDE_DIRS}" ++ "${LIBUSB_INCLUDE_DIR}" ++ "${CMAKE_SOURCE_DIR}/src/moncalqt" + ) + + SET(ARGYLL_SOURCES @@ -83,3 +83,21 @@ PROPERTIES COMPILE_FLAGS "${argyll_flags}" Index: files/patch-src-lcms-CMakeLists.txt =================================================================== RCS file: files/patch-src-lcms-CMakeLists.txt diff -N files/patch-src-lcms-CMakeLists.txt --- /dev/null 1 Jan 1970 00:00:00 -0000 +++ files/patch-src-lcms-CMakeLists.txt 1 May 2012 14:12:49 -0000 @@ -0,0 +1,14 @@ +--- src/lcms/CMakeLists.txt 2008-02-19 18:02:38.000000000 -0300 ++++ src/lcms/CMakeLists.txt 2012-05-01 10:36:07.000000000 -0300 +@@ -1,8 +1,6 @@ +-# Note that this will override the basic include path, which we want since there might be a stock +-# lcms.h in the OS. +-SET_DIRECTORY_PROPERTIES( +- PROPERTIES +- INCLUDE_DIRECTORIES "${CMAKE_CURRENT_SOURCE_DIR}/include" ++INCLUDE_DIRECTORIES( ++ BEFORE ++ "${CMAKE_CURRENT_SOURCE_DIR}/include" + ) + + SET(LCMSLOCAL_SOURCES Index: files/patch-src-liblprof-CMakeLists.txt =================================================================== RCS file: files/patch-src-liblprof-CMakeLists.txt diff -N files/patch-src-liblprof-CMakeLists.txt --- /dev/null 1 Jan 1970 00:00:00 -0000 +++ files/patch-src-liblprof-CMakeLists.txt 1 May 2012 14:12:49 -0000 @@ -0,0 +1,15 @@ +--- src/liblprof/CMakeLists.txt 2008-02-19 18:02:38.000000000 -0300 ++++ src/liblprof/CMakeLists.txt 2012-05-01 10:35:39.000000000 -0300 +@@ -18,7 +18,9 @@ ADD_LIBRARY(liblprof + ${LIBLPROF_SOURCES} + ) + +-SET_DIRECTORY_PROPERTIES( +- PROPERTIES +- INCLUDE_DIRECTORIES "${CMAKE_SOURCE_DIR}/src/lcms/include;${ARGYLL_INCLUDE_DIRS}" ++INCLUDE_DIRECTORIES( ++ BEFORE ++ "${CMAKE_SOURCE_DIR}/src/lcms/include" ++ "${ARGYLL_INCLUDE_DIRS}" + ) ++ --- lprof-devel-20080514_8.patch ends here --- >Release-Note: >Audit-Trail: >Unformatted: