On 12/5/2020 3:45 AM, Dimitri Karamazov wrote:
Update opencolorio to 1.1.1

Required by graphics/krita and graphics/blender
both Build,run tested on amd64.

The python module is not used by any of the above
programs so I removed it. Or should I let it be as
is?

I'll be keeping this updated for blender.


Part of the patch does not apply as is.

The text leading up to this was:
--------------------------
|Index: patches/patch-export_OpenColorIO_OpenColorABI_h_in
|===================================================================
|RCS file: patches/patch-export_OpenColorIO_OpenColorABI_h_in
|diff -N patches/patch-export_OpenColorIO_OpenColorABI_h_in
|--- patches/patch-export_OpenColorIO_OpenColorABI_h_in 21 Nov 2014 17:25:49 -0000      1.2
|+++ /dev/null  1 Jan 1970 00:00:00 -0000
--------------------------
Patching file patches/patch-export_OpenColorIO_OpenColorABI_h_in using Plan A...
Empty context always matches.
Hunk #1 failed at 0.
1 out of 1 hunks failed--saving rejects to patches/patch-export_OpenColorIO_OpenColorABI_h_in.rej


Any comments/OK's?

Index: Makefile
===================================================================
RCS file: /cvs/ports/graphics/opencolorio/Makefile,v
retrieving revision 1.10
diff -u -p -r1.10 Makefile
--- Makefile    12 Jul 2019 20:47:04 -0000      1.10
+++ Makefile    5 Dec 2020 08:32:04 -0000
@@ -2,16 +2,17 @@

  COMMENT =             color management library

-V =                    20140911
-DISTNAME =             OpenColorIO-$V
-PKGNAME =              opencolorio-$V
-REVISION =             2
+V =                    1.1.1
+GH_ACCOUNT =           AcademySoftwareFoundation
+GH_PROJECT =           OpenColorIO
+GH_TAGNAME =           v${V}
+DISTNAME =             ${GH_PROJECT:L}-${V}

-SHARED_LIBS +=  OpenColorIO               1.0 # 1.0
+SHARED_LIBS +=         OpenColorIO               1.1 # 1.1.1

  CATEGORIES =          graphics

-HOMEPAGE =             http://opencolorio.org/index.html
+HOMEPAGE =             https://opencolorio.org/

  MAINTAINER =          Pascal Stumpf <pas...@stumpf.co>

@@ -22,27 +23,33 @@ WANTLIB += m pthread ${COMPILER_LIBCXX}

  COMPILER =            base-clang ports-gcc base-gcc

-MASTER_SITES =         http://deftly.net/
-
-MODULES =              devel/cmake \
-                       lang/python
+MODULES =              devel/cmake
  LIB_DEPENDS =         textproc/tinyxml \
                        devel/yaml-cpp

-CONFIGURE_ARGS +=      -DPYTHON="${MODPY_BIN}" \
-                       -DUSE_EXTERNAL_TINYXML=ON \
+CONFIGURE_ARGS +=      -DUSE_EXTERNAL_TINYXML=ON \
                        -DUSE_EXTERNAL_YAML=ON \
                        -DOCIO_USE_SSE=OFF \
                        -DOCIO_BUILD_STATIC=OFF \
-                       -DOCIO_BUILD_APPS=OFF \
-                       -DCMAKE_SHARED_LINKER_FLAGS="-L${LOCALBASE}/lib" \
-                       -DCMAKE_EXE_LINKER_FLAGS="-L${LOCALBASE}/lib" \
-                       -DCMAKE_MODULE_LINKER_FLAGS="-L${LOCALBASE}/lib"
+                       -DOCIO_BUILD_APPS=OFF

  NO_TEST =             Yes

-WRKDIST =              ${WRKDIR}/OpenColorIO-master
-
-CFLAGS =               -pthread
+do-install:
+       ${INSTALL_DATA_DIR} ${PREFIX}/include/OpenColorIO
+       ${INSTALL_DATA} ${WRKBUILD}/export/*.h \
+                ${WRKSRC}/export/OpenColorIO/*.h \
+               ${PREFIX}/include/OpenColorIO
+       ${INSTALL_DATA_DIR} ${PREFIX}/lib/cmake/OpenColorIO
+       ${INSTALL_DATA} ${WRKBUILD}/CMakeFiles/Export/cmake/*.cmake \
+               ${WRKBUILD}/OpenColorIOConfig.cmake \
+               ${PREFIX}/lib/cmake/OpenColorIO
+       ${INSTALL_DATA} ${WRKBUILD}/src/core/libOpenColorIO.so.* \
+               ${PREFIX}/lib
+       ${INSTALL_DATA} ${WRKBUILD}/src/core/*.pc \
+               ${PREFIX}/lib/pkgconfig
+       ${INSTALL_DATA_DIR} ${PREFIX}/share/ocio
+       ${INSTALL_SCRIPT} ${WRKBUILD}/share/ocio/*.sh \
+               ${PREFIX}/share/ocio

  .include <bsd.port.mk>
Index: distinfo
===================================================================
RCS file: /cvs/ports/graphics/opencolorio/distinfo,v
retrieving revision 1.3
diff -u -p -r1.3 distinfo
--- distinfo    21 Nov 2014 17:25:49 -0000      1.3
+++ distinfo    5 Dec 2020 08:32:04 -0000
@@ -1,2 +1,2 @@
-SHA256 (OpenColorIO-20140911.tar.gz) = 
d1Y6L8peE+SuqhBv/lA5Km70hjRlyszFOuVFmNzlnpQ=
-SIZE (OpenColorIO-20140911.tar.gz) = 13742525
+SHA256 (opencolorio-1.1.1.tar.gz) = 
ybW53vkH4dr7KeNzNrcC//IsxjBtRFoTsWIbinVMFMg=
+SIZE (opencolorio-1.1.1.tar.gz) = 13828483
Index: patches/patch-CMakeLists_txt
===================================================================
RCS file: patches/patch-CMakeLists_txt
diff -N patches/patch-CMakeLists_txt
--- /dev/null   1 Jan 1970 00:00:00 -0000
+++ patches/patch-CMakeLists_txt        5 Dec 2020 08:32:04 -0000
@@ -0,0 +1,25 @@
+$OpenBSD$
+
+Fix macro redefined error
+
+--- CMakeLists.txt.orig        2018-01-05 12:08:27 UTC
++++ CMakeLists.txt
+@@ -218,8 +222,7 @@ else(USE_EXTERNAL_TINYXML)
+         ## => great news when build staticaly since we do not want another 
client project have to link also with tinyxml when he want to use this project
+         ## => could be problematic if the client project use another version 
of tinyxml... In this case build tinyxml as shared lib with all projects could be 
a solution
+         ## => TODO: so maybe provide a simple cmake way to build 3rdParty as 
shared and auto install with this project ?
+-        set_target_properties(TINYXML_LIB PROPERTIES COMPILE_FLAGS 
"-DTIXML_USE_STL -fPIC -fvisibility-inlines-hidden -fvisibility=hidden")
+-        add_definitions(-DTIXML_USE_STL) ## needed to build correctly, and 
also need to be propagated in child projects (client projects)
++        set_target_properties(TINYXML_LIB PROPERTIES COMPILE_FLAGS "-fPIC 
-fvisibility-inlines-hidden -fvisibility=hidden")
+         list(APPEND EXTERNAL_OBJECTS $<TARGET_OBJECTS:TINYXML_LIB>)
+     else()
+         find_package(Git REQUIRED) ## in order to apply patch (for 
crossplateform compatibility)
+@@ -384,7 +387,7 @@ else()
+     set(OCIO_INLINES_HIDDEN OFF)
+ endif()
+
+-set(EXTERNAL_COMPILE_FLAGS "-DTIXML_USE_STL ${YAML_CPP_COMPILE_FLAGS} 
${GCC_COMPILE_FLAGS}")
++set(EXTERNAL_COMPILE_FLAGS "${YAML_CPP_COMPILE_FLAGS} ${GCC_COMPILE_FLAGS}")
+
+ set(EXTERNAL_LINK_FLAGS "")
+ set(EXTERNAL_LIBRARY_DIRS ${PROJECT_BINARY_DIR}/ext/dist/lib)
Index: patches/patch-export_OpenColorIO_OpenColorABI_h_in
===================================================================
RCS file: patches/patch-export_OpenColorIO_OpenColorABI_h_in
diff -N patches/patch-export_OpenColorIO_OpenColorABI_h_in
--- patches/patch-export_OpenColorIO_OpenColorABI_h_in  21 Nov 2014 17:25:49 
-0000      1.2
+++ /dev/null   1 Jan 1970 00:00:00 -0000
@@ -1,12 +0,0 @@
-$OpenBSD: patch-export_OpenColorIO_OpenColorABI_h_in,v 1.2 2014/11/21 17:25:49 
pascal Exp $
---- export/OpenColorIO/OpenColorABI.h.in.orig  Thu Sep 11 21:08:18 2014
-+++ export/OpenColorIO/OpenColorABI.h.in       Fri Nov 21 13:29:20 2014
-@@ -69,7 +69,7 @@ OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY O
-
- // If supported, define OCIOEXPORT, OCIOHIDDEN
- // (used to choose which symbols to export from OpenColorIO)
--#if defined __linux__ || __APPLE__ || __FreeBSD__
-+#if defined __linux__ || __APPLE__ || __FreeBSD__ || __OpenBSD__
-     #if __GNUC__ >= 4
-         #define OCIOEXPORT __attribute__ ((visibility("default")))
-         #define OCIOHIDDEN __attribute__ ((visibility("hidden")))
Index: patches/patch-src_core_Config_cpp
===================================================================
RCS file: patches/patch-src_core_Config_cpp
diff -N patches/patch-src_core_Config_cpp
--- /dev/null   1 Jan 1970 00:00:00 -0000
+++ patches/patch-src_core_Config_cpp   5 Dec 2020 08:32:04 -0000
@@ -0,0 +1,15 @@
+$OpenBSD$
+
+Avoid assigning field to itself
+
+--- src/core/Config.cpp.orig   Thu Dec  3 22:17:51 2020
++++ src/core/Config.cpp        Thu Dec  3 22:18:10 2020
+@@ -330,7 +330,7 @@ OCIO_NAMESPACE_ENTER
+                 sanitytext_ = rhs.sanitytext_;
+
+                 cacheids_ = rhs.cacheids_;
+-                cacheidnocontext_ = cacheidnocontext_;
++                cacheidnocontext_ = rhs.cacheidnocontext_;
+             }
+             return *this;
+         }
Index: patches/patch-src_core_Platform_h
===================================================================
RCS file: /cvs/ports/graphics/opencolorio/patches/patch-src_core_Platform_h,v
retrieving revision 1.1.1.1
diff -u -p -r1.1.1.1 patch-src_core_Platform_h
--- patches/patch-src_core_Platform_h   25 Mar 2014 10:33:33 -0000      1.1.1.1
+++ patches/patch-src_core_Platform_h   5 Dec 2020 08:32:04 -0000
@@ -1,9 +1,9 @@
  $OpenBSD: patch-src_core_Platform_h,v 1.1.1.1 2014/03/25 10:33:33 pascal Exp $
---- src/core/Platform.h.orig   Wed Oct  9 00:59:34 2013
-+++ src/core/Platform.h        Mon Mar 24 22:58:05 2014
+--- src/core/Platform.h.orig   Thu Mar 28 10:42:57 2019
++++ src/core/Platform.h        Thu Dec  3 20:01:17 2020
  @@ -85,7 +85,7 @@ OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY O
+ // assume linux/unix/posix

- // linux/unix/posix
   #include <stdlib.h>
  -#if !defined(__FreeBSD__)
  +#if !defined(__FreeBSD__) && !defined(__OpenBSD__)
Index: pkg/PLIST
===================================================================
RCS file: /cvs/ports/graphics/opencolorio/pkg/PLIST,v
retrieving revision 1.1.1.1
diff -u -p -r1.1.1.1 PLIST
--- pkg/PLIST   25 Mar 2014 10:33:33 -0000      1.1.1.1
+++ pkg/PLIST   5 Dec 2020 08:32:04 -0000
@@ -1,13 +1,15 @@
-@comment $OpenBSD: PLIST,v 1.1.1.1 2014/03/25 10:33:33 pascal Exp $
+@comment $OpenBSD: PLIST,v$
  include/OpenColorIO/
  include/OpenColorIO/OpenColorABI.h
  include/OpenColorIO/OpenColorIO.h
  include/OpenColorIO/OpenColorTransforms.h
  include/OpenColorIO/OpenColorTypes.h
-include/PyOpenColorIO/
-include/PyOpenColorIO/PyOpenColorIO.h
+lib/cmake/
+lib/cmake/OpenColorIO/
+lib/cmake/OpenColorIO/OpenColorIO${MODCMAKE_BUILD_SUFFIX}
+lib/cmake/OpenColorIO/OpenColorIO.cmake
+lib/cmake/OpenColorIO/OpenColorIOConfig.cmake
  @lib lib/libOpenColorIO.so.${LIBOpenColorIO_VERSION}
  lib/pkgconfig/OpenColorIO.pc
-lib/python${MODPY_VERSION}/site-packages/PyOpenColorIO.so
  share/ocio/
  share/ocio/setup_ocio.sh



Reply via email to