Subject: Re: bug#72046: [core-updates] OpenImageIO fails to build

From: Zheng Junjie <[email-masked]>
Date: Fri, 12 Jul 2024 11:26:16 +0800

Leo Famulari <[email-masked]> writes:

> On Thu, Jul 11, 2024 at 11:32:53AM -0400, Leo Famulari wrote:
>> Okay, great. I confirm your patch fixes the problem. I'll push it to
>> core-updates on your behalf.
>
> Pushed as 50243774824597dbd141a074a7be0117dc450cef
>
> Thanks for your help!

i think also need this patch.

From f781494940c801db6c63e22ea12c43ffec547456 Mon Sep 17 00:00:00 2001
Message-ID: <f78[email-masked]>
From: Zheng Junjie <[email-masked]>
Date: Fri, 12 Jul 2024 11:23:34 +0800
Subject: [PATCH] gnu: opencolorio: Fix cmake file.

* gnu/packages/image-processing.scm (opencolorio)[arguments]: Add
fix-OpenColorIOConfig phase.

Change-Id: I317870801f77dc59bcd07b0d11a750bbbb63b327
---
 gnu/packages/image-processing.scm | 13 ++++++++++++-
 1 file changed, 12 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/image-processing.scm b/gnu/packages/image-processing.scm
index 32d5b8bb5c..4e83e5c26c 100644
--- a/gnu/packages/image-processing.scm
+++ b/gnu/packages/image-processing.scm
@@ -276,7 +276,18 @@ (define-public opencolorio
     (build-system cmake-build-system)
     (arguments
      ;; XXX: GPU tests are failing.
-     (list #:configure-flags #~(list "-DOCIO_BUILD_GPU_TESTS=false")))
+     (list #:configure-flags #~(list "-DOCIO_BUILD_GPU_TESTS=false")
+           #:phases #~(modify-phases %standard-phases
+                        (add-after 'install 'fix-OpenColorIOConfig
+                          (lambda _
+                            ;; Work around a CMake Zlib-detection bug:
+                            ;; https://gitlab.kitware.com/cmake/cmake/-/issues/25200
+                            ;; make OpenColorIOConfig.cmake is a normal cmake file
+                            (substitute*
+                                (string-append #$output
+                                               "/lib/cmake/OpenColorIO/OpenColorIOConfig.cmake")
+                              (("\\.#define ZLIB_VERSION \"1\\.3\"")
+                               "")))))))
     (native-inputs
      ;; XXX: OCIO has unit tests for OpenShadingLanguage, but they fail.
      ;; They also require OIIO, but OCIO is an optional dependency to it.

base-commit: 50243774824597dbd141a074a7be0117dc450cef
prerequisite-patch-id: 54dff8d31eca386fe3e0f90879bf9e98dade94d7
-- 
2.45.2