Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 1 Sep 2015 15:17:08 +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: r395765 - in head/graphics/appleseed: . files
Message-ID:  <201509011517.t81FH84e055473@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: danfe
Date: Tue Sep  1 15:17:07 2015
New Revision: 395765
URL: https://svnweb.freebsd.org/changeset/ports/395765

Log:
  Fix mysterious linking problems with bundled LZ4 code: it was due to the
  fact that I had `archivers/liblz4' installed locally, and its header was
  picked up before the bundled one which has "static inline" definition of
  LZ4_compressBound() function vs. normal definition in system-wide header.

Modified:
  head/graphics/appleseed/Makefile
  head/graphics/appleseed/files/patch-CMakeLists.txt

Modified: head/graphics/appleseed/Makefile
==============================================================================
--- head/graphics/appleseed/Makefile	Tue Sep  1 14:42:26 2015	(r395764)
+++ head/graphics/appleseed/Makefile	Tue Sep  1 15:17:07 2015	(r395765)
@@ -40,15 +40,7 @@ PLIST_SUB=	APPHOME=${CMAKE_INSTALL_PREFI
 SUB_FILES=	pkg-message
 SUB_LIST:=	${PLIST_SUB}
 
-.include <bsd.port.pre.mk>
-
 post-patch:
-.if ${OPSYS} == FreeBSD && ${OSVERSION} >= 1100074
-# Fix "undefined reference to `LZ4_compressBound'" upon linking with Clang
-# on -CURRENT (I'm not sure how it can be mapped to specific Clang version)
-	@${REINPLACE_CMD} -e '/LZ4_compressBound/s,static inline ,,' \
-		${WRKSRC}/src/lz4/lz4.h
-.endif
 	@${REINPLACE_CMD} -e 's,Linux,${OPSYS},' \
 		${WRKSRC}/src/appleseed.cli/CMakeLists.txt \
 		${WRKSRC}/src/appleseed.studio/CMakeLists.txt \
@@ -59,4 +51,4 @@ post-patch:
 		${WRKSRC}/src/tools/updateprojectfile/CMakeLists.txt
 	@${RMDIR} ${WRKSRC}/sandbox/docs/api
 
-.include <bsd.port.post.mk>
+.include <bsd.port.mk>

Modified: head/graphics/appleseed/files/patch-CMakeLists.txt
==============================================================================
--- head/graphics/appleseed/files/patch-CMakeLists.txt	Tue Sep  1 14:42:26 2015	(r395764)
+++ head/graphics/appleseed/files/patch-CMakeLists.txt	Tue Sep  1 15:17:07 2015	(r395765)
@@ -19,7 +19,25 @@
      include (src/cmake/config/linux-gcc.txt)
  else ()
      message (FATAL_ERROR "Unsupported platform or compiler (CMAKE_SYSTEM_NAME is \"${CMAKE_SYSTEM_NAME}\"). "
-@@ -546,7 +549,7 @@ endif ()
+@@ -147,6 +150,9 @@ if (WITH_OSL)
+                     ${PROJECT_SOURCE_DIR}/src/appleseed/renderer/kernel/shading/stdosl.h)
+ endif ()
+ 
++# XXX: avoid picking up lz4 headers from /usr/local/include == ${Boost_INCLUDE_DIRS}
++# if `archivers/liblz4' port is installed.
++include_directories (src/lz4)
+ 
+ #--------------------------------------------------------------------------------------------------
+ # Boost libraries.
+@@ -244,7 +250,6 @@ endif ()
+ # Common include paths.
+ include_directories (
+     src/appleseed
+-    src/lz4
+     ${PYTHON_INCLUDE_DIRS}
+ )
+ 
+@@ -546,7 +551,7 @@ endif ()
  
  find_package (Doxygen)
  
@@ -28,7 +46,7 @@
      configure_file (${PROJECT_SOURCE_DIR}/scripts/appleseed.doxyfile.in ${PROJECT_BINARY_DIR}/appleseed.doxyfile @ONLY)
      add_custom_target (
          doc ${DOXYGEN_EXECUTABLE} ${PROJECT_BINARY_DIR}/appleseed.doxyfile
-@@ -596,22 +599,25 @@ install (
+@@ -596,22 +601,25 @@ install (
      DIRECTORY
          sandbox/docs
          sandbox/samples
@@ -57,7 +75,7 @@
  install (
      DIRECTORY src/appleseed/foundation
      DESTINATION include
-@@ -629,3 +635,4 @@ install (
+@@ -629,3 +637,4 @@ install (
      DESTINATION include
      FILES_MATCHING PATTERN "*.h"
  )



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