Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 3 Apr 2020 08:16:30 +0000 (UTC)
From:      Alexey Dokuchaev <danfe@FreeBSD.org>
To:        ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org
Subject:   svn commit: r530474 - in head/graphics/alembic: . files
Message-ID:  <202004030816.0338GUuT092039@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: danfe
Date: Fri Apr  3 08:16:29 2020
New Revision: 530474
URL: https://svnweb.freebsd.org/changeset/ports/530474

Log:
  Unbreak the build when previous version of the port is installed by
  preventing system-wide include directories being injected into the
  global CMake state and thus taking precedence over target's include
  directories.  Hook provided test suite to our framework while here.
  
  PR:	245298

Added:
  head/graphics/alembic/files/patch-lib_Alembic_AbcCoreHDF5_Tests_CMakeLists.txt   (contents, props changed)
Modified:
  head/graphics/alembic/Makefile
  head/graphics/alembic/files/patch-CMakeLists.txt

Modified: head/graphics/alembic/Makefile
==============================================================================
--- head/graphics/alembic/Makefile	Fri Apr  3 08:16:00 2020	(r530473)
+++ head/graphics/alembic/Makefile	Fri Apr  3 08:16:29 2020	(r530474)
@@ -27,4 +27,7 @@ HDF5_CMAKE_ON=	-DUSE_HDF5:BOOL=ON
 PLIST_FILES=	lib/libAlembic.so.${PORTVERSION} \
 		lib/libAlembic.so.${PORTVERSION:R}
 
+do-test:
+	@cd ${TEST_WRKSRC} && ctest -C ${CMAKE_BUILD_TYPE} ${_MAKE_JOBS}
+
 .include <bsd.port.mk>

Modified: head/graphics/alembic/files/patch-CMakeLists.txt
==============================================================================
--- head/graphics/alembic/files/patch-CMakeLists.txt	Fri Apr  3 08:16:00 2020	(r530473)
+++ head/graphics/alembic/files/patch-CMakeLists.txt	Fri Apr  3 08:16:29 2020	(r530474)
@@ -1,6 +1,6 @@
---- CMakeLists.txt.orig	2017-03-24 00:18:21 UTC
+--- CMakeLists.txt.orig	2019-10-26 04:47:57 UTC
 +++ CMakeLists.txt
-@@ -111,7 +111,7 @@ IF ("${CMAKE_SYSTEM_NAME}" MATCHES "Darw
+@@ -112,7 +112,7 @@ IF ("${CMAKE_SYSTEM_NAME}" MATCHES "Darwin")
  ENDIF()
  
  SET(LINUX FALSE)
@@ -9,7 +9,7 @@
      SET(LINUX TRUE)
  ENDIF()
  
-@@ -140,9 +140,6 @@ IF ("${CMAKE_BUILD_TYPE}" MATCHES "Debug
+@@ -141,9 +141,6 @@ IF ("${CMAKE_BUILD_TYPE}" MATCHES "Debug")
      ENDIF()
  ELSEIF ("${CMAKE_BUILD_TYPE}" MATCHES "Release")
      ADD_DEFINITIONS(-DNDEBUG=1 -UDEBUG)
@@ -19,3 +19,24 @@
  ENDIF()
  
  IF (NOT ${WINDOWS})
+@@ -228,14 +225,18 @@ ENDIF()
+ 
+ # IlmBase
+ INCLUDE("./cmake/AlembicIlmBase.cmake")
+-INCLUDE_DIRECTORIES(${ALEMBIC_ILMBASE_INCLUDE_DIRECTORY})
++# XXX: do not pollute global state: include directories correctly added
++# via TARGET_INCLUDE_DIRECTORIES in the `lib/Alembic/CMakeLists.txt'
++#INCLUDE_DIRECTORIES(${ALEMBIC_ILMBASE_INCLUDE_DIRECTORY})
+ 
+ # HDF5
+ IF (USE_HDF5)
+     FIND_PACKAGE(ZLIB REQUIRED)
+     SET(ALEMBIC_WITH_HDF5 "1")
+     INCLUDE("./cmake/AlembicHDF5.cmake")
+-    INCLUDE_DIRECTORIES(${HDF5_INCLUDE_DIRS})
++    # XXX: do not pollute global state: include directories correctly added
++    # via TARGET_INCLUDE_DIRECTORIES in the `lib/Alembic/CMakeLists.txt'
++    #INCLUDE_DIRECTORIES(${HDF5_INCLUDE_DIRS})
+     SET(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -DH5_USE_18_API")
+ ENDIF()
+ 

Added: head/graphics/alembic/files/patch-lib_Alembic_AbcCoreHDF5_Tests_CMakeLists.txt
==============================================================================
--- /dev/null	00:00:00 1970	(empty, because file is newly added)
+++ head/graphics/alembic/files/patch-lib_Alembic_AbcCoreHDF5_Tests_CMakeLists.txt	Fri Apr  3 08:16:29 2020	(r530474)
@@ -0,0 +1,12 @@
+--- lib/Alembic/AbcCoreHDF5/Tests/CMakeLists.txt.orig	2019-10-26 04:47:57 UTC
++++ lib/Alembic/AbcCoreHDF5/Tests/CMakeLists.txt
+@@ -33,7 +33,8 @@
+ ##
+ ##-*****************************************************************************
+ 
+-INCLUDE_DIRECTORIES(${CMAKE_SOURCE_DIR}/lib ${PROJECT_BINARY_DIR}/lib)
++INCLUDE_DIRECTORIES(${CMAKE_SOURCE_DIR}/lib ${PROJECT_BINARY_DIR}/lib
++    ${HDF5_INCLUDE_DIRS})
+ 
+ SET(CXX_FILES
+     ArchiveTests.cpp



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202004030816.0338GUuT092039>