GNU bug report logs - #72046
[core-updates] OpenImageIO fails to build

Previous Next

Package: guix;

Reported by: Leo Famulari <leo <at> famulari.name>

Date: Wed, 10 Jul 2024 20:27:16 UTC

Severity: normal

Done: Leo Famulari <leo <at> famulari.name>

Bug is archived. No further changes may be made.

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 72046 in the body.
You can then email your comments to 72046 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to bug-guix <at> gnu.org:
bug#72046; Package guix. (Wed, 10 Jul 2024 20:27:16 GMT) Full text and rfc822 format available.

Acknowledgement sent to Leo Famulari <leo <at> famulari.name>:
New bug report received and forwarded. Copy sent to bug-guix <at> gnu.org. (Wed, 10 Jul 2024 20:27:20 GMT) Full text and rfc822 format available.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Leo Famulari <leo <at> famulari.name>
To: bug-guix <at> gnu.org
Subject: [core-updates] OpenImageIO fails to build
Date: Wed, 10 Jul 2024 15:47:58 -0400
[Message part 1 (text/plain, inline)]
On core-updates commit 378e1d9b69b030a165, openimageio fails to build
like this:

------
[ 30%] Building CXX object src/libOpenImageIO/CMakeFiles/OpenImageIO.dir/imageio.cpp.o
cd /tmp/guix-build-openimageio-2.5.13.0.drv-0/build/src/libOpenImageIO && /gnu/store/86fc8bi3mciljxz7c79jx8zr4wsx7xw8-gcc-11.4.0/bin/c++ -DEMBED_PLUGINS=1 -DOIIO_FREETYPE_VERSION=\"\" -DOIIO_INTERNAL=1 -DOIIO_OPENEXR_CORE_DEFAULT=1 -DOIIO_OpenCV_VERSION=\"\" -DOIIO_PYTHON_VERSION=\"3.10.7\" -DOIIO_QT_VERSION=\"\" -DOIIO_TBB_VERSION=\"\" -DOIIO_USE_EXR_C_API=1 -DOpenImageIO_EXPORTS -DUSE_BOOST_FILESYSTEM -DUSE_EXTERNAL_PUGIXML=1 -DUSE_GIF -DUSE_HEIF=1 -DUSE_JPEG_TURBO=1 -D__STDC_CONSTANT_MACROS -D__STDC_LIMIT_MACROS -I/tmp/guix-build-openimageio-2.5.13.0.drv-0/build/include/OpenImageIO -I/tmp/guix-build-openimageio-2.5.13.0.drv-0/build/include -I/tmp/guix-build-openimageio-2.5.13.0.drv-0/build/src/include -I/tmp/guix-build-openimageio-2.5.13.0.drv-0/source/src/include -I/OpenEXR -isystem /gnu/store/mcyxqs4sxnb8r642q1l0gfmg86hrngqp-imath-3.1.3/include/Imath -isystem /gnu/store/p1rqc1ydnvcrn5ndjq9kn37l7fscilrv-openexr-3.2.4/include/OpenEXR -O2 -g -DNDEBUG -fPIC -fvisibility=hidden -Wall -Wno-unused-local-typedefs -Wno-unused-result -Wno-aligned-new -Wno-noexcept-type -fno-math-errno -std=c++14 -MD -MT src/libOpenImageIO/CMakeFiles/OpenImageIO.dir/imageio.cpp.o -MF CMakeFiles/OpenImageIO.dir/imageio.cpp.o.d -o CMakeFiles/OpenImageIO.dir/imageio.cpp.o -c /tmp/guix-build-openimageio-2.5.13.0.drv-0/source/src/libOpenImageIO/imageio.cpp
/tmp/guix-build-openimageio-2.5.13.0.drv-0/source/src/libOpenImageIO/imageio.cpp: In function ‘bool OpenImageIO_v2_5::getattribute(OpenImageIO_v2_5::string_view, OpenImageIO_v2_5::TypeDesc, void*)’:
/tmp/guix-build-openimageio-2.5.13.0.drv-0/source/src/libOpenImageIO/imageio.cpp:623:33: error: expected primary-expression before ‘(’ token
  623 |         *(ustring*)val = ustring(OIIO_ALL_BUILD_DEPS_FOUND);
      |                                 ^
In file included from /tmp/guix-build-openimageio-2.5.13.0.drv-0/source/src/libOpenImageIO/imageio.cpp:24:
/tmp/guix-build-openimageio-2.5.13.0.drv-0/build/include/imageio_pvt.h:229:377: error: expected ‘)’ before numeric constant
  229 | #define OIIO_ALL_BUILD_DEPS_FOUND "Boost 1.83.0;BZip2 1.0.8;DCMTK NONE;FFmpeg NONE;fmt 9.1.0;Freetype NONE;GIF 5.2.1;Libheif 1.17.6;libjpeg-turbo 2.1.4;LibRaw NONE;OpenColorIO NONE;OpenCV NONE;OpenEXR 3.2.4;OpenGL NONE;OpenJPEG NONE;PNG 1.6.39;Ptex NONE;Ptex NONE;pugixml 1.12;pybind11 2.8.1;Python 3.10.7;Robinmap;TBB NONE;TIFF 4.4.0;WebP NONE;ZLIB 1.3.#define ZLIB_VERSION "1.3""
      |                                                                                                                                                                                                                                                                                                                                                                                         ^~~
/tmp/guix-build-openimageio-2.5.13.0.drv-0/source/src/libOpenImageIO/imageio.cpp:623:34: note: in expansion of macro ‘OIIO_ALL_BUILD_DEPS_FOUND’
  623 |         *(ustring*)val = ustring(OIIO_ALL_BUILD_DEPS_FOUND);
      |                                  ^~~~~~~~~~~~~~~~~~~~~~~~~
/tmp/guix-build-openimageio-2.5.13.0.drv-0/source/src/libOpenImageIO/imageio.cpp:623:33: note: to match this ‘(’
  623 |         *(ustring*)val = ustring(OIIO_ALL_BUILD_DEPS_FOUND);
      |                                 ^
make[2]: *** [src/libOpenImageIO/CMakeFiles/OpenImageIO.dir/build.make:387: src/libOpenImageIO/CMakeFiles/OpenImageIO.dir/imageio.cpp.o] Error 1
make[2]: Leaving directory '/tmp/guix-build-openimageio-2.5.13.0.drv-0/build'
make[1]: *** [CMakeFiles/Makefile2:1703: src/libOpenImageIO/CMakeFiles/OpenImageIO.dir/all] Error 2
make[1]: Leaving directory '/tmp/guix-build-openimageio-2.5.13.0.drv-0/build'
make: *** [Makefile:169: all] Error 2
error: in phase 'build': uncaught exception:
%exception #<&invoke-error program: "make" arguments: ("-j" "1") exit-status: 2 term-signal: #f stop-signal: #f> 
phase `build' failed after 426.2 seconds
command "make" "-j" "1" failed with status 2
build process 14 exited with status 256
builder for `/gnu/store/zbb1ycg7q7iqsp8pa1dyvbsxvh2v94lx-openimageio-2.5.13.0.drv' failed with exit code 1
------

I can't find anything online related to this. Any ideas? Full log
attached.
[b1ycg7q7iqsp8pa1dyvbsxvh2v94lx-openimageio-2.5.13.0.drv.gz (application/x-gunzip, attachment)]

Information forwarded to bug-guix <at> gnu.org:
bug#72046; Package guix. (Thu, 11 Jul 2024 06:31:01 GMT) Full text and rfc822 format available.

Message #8 received at 72046 <at> debbugs.gnu.org (full text, mbox):

From: Zheng Junjie <zhengjunjie <at> iscas.ac.cn>
To: Leo Famulari <leo <at> famulari.name>
Cc: 72046 <at> debbugs.gnu.org
Subject: Re: bug#72046: [core-updates] OpenImageIO fails to build
Date: Thu, 11 Jul 2024 14:29:59 +0800
[Message part 1 (text/plain, inline)]
Leo Famulari <leo <at> famulari.name> writes:

> On core-updates commit 378e1d9b69b030a165, openimageio fails to build
> like this:
>
> ------
> [ 30%] Building CXX object src/libOpenImageIO/CMakeFiles/OpenImageIO.dir/imageio.cpp.o
> cd /tmp/guix-build-openimageio-2.5.13.0.drv-0/build/src/libOpenImageIO
> && /gnu/store/86fc8bi3mciljxz7c79jx8zr4wsx7xw8-gcc-11.4.0/bin/c++
> -DEMBED_PLUGINS=1 -DOIIO_FREETYPE_VERSION=\"\" -DOIIO_INTERNAL=1
> -DOIIO_OPENEXR_CORE_DEFAULT=1 -DOIIO_OpenCV_VERSION=\"\"
> -DOIIO_PYTHON_VERSION=\"3.10.7\" -DOIIO_QT_VERSION=\"\"
> -DOIIO_TBB_VERSION=\"\" -DOIIO_USE_EXR_C_API=1 -DOpenImageIO_EXPORTS
> -DUSE_BOOST_FILESYSTEM -DUSE_EXTERNAL_PUGIXML=1 -DUSE_GIF -DUSE_HEIF=1
> -DUSE_JPEG_TURBO=1 -D__STDC_CONSTANT_MACROS -D__STDC_LIMIT_MACROS
> -I/tmp/guix-build-openimageio-2.5.13.0.drv-0/build/include/OpenImageIO
> -I/tmp/guix-build-openimageio-2.5.13.0.drv-0/build/include
> -I/tmp/guix-build-openimageio-2.5.13.0.drv-0/build/src/include
> -I/tmp/guix-build-openimageio-2.5.13.0.drv-0/source/src/include
> -I/OpenEXR -isystem
> /gnu/store/mcyxqs4sxnb8r642q1l0gfmg86hrngqp-imath-3.1.3/include/Imath
> -isystem
> /gnu/store/p1rqc1ydnvcrn5ndjq9kn37l7fscilrv-openexr-3.2.4/include/OpenEXR
> -O2 -g -DNDEBUG -fPIC -fvisibility=hidden -Wall
> -Wno-unused-local-typedefs -Wno-unused-result -Wno-aligned-new
> -Wno-noexcept-type -fno-math-errno -std=c++14 -MD -MT
> src/libOpenImageIO/CMakeFiles/OpenImageIO.dir/imageio.cpp.o -MF
> CMakeFiles/OpenImageIO.dir/imageio.cpp.o.d -o
> CMakeFiles/OpenImageIO.dir/imageio.cpp.o -c
> /tmp/guix-build-openimageio-2.5.13.0.drv-0/source/src/libOpenImageIO/imageio.cpp
> /tmp/guix-build-openimageio-2.5.13.0.drv-0/source/src/libOpenImageIO/imageio.cpp:
> In function ‘bool
> OpenImageIO_v2_5::getattribute(OpenImageIO_v2_5::string_view,
> OpenImageIO_v2_5::TypeDesc, void*)’:
> /tmp/guix-build-openimageio-2.5.13.0.drv-0/source/src/libOpenImageIO/imageio.cpp:623:33:
> error: expected primary-expression before ‘(’ token
>   623 |         *(ustring*)val = ustring(OIIO_ALL_BUILD_DEPS_FOUND);
>       |                                 ^
> In file included from /tmp/guix-build-openimageio-2.5.13.0.drv-0/source/src/libOpenImageIO/imageio.cpp:24:
> /tmp/guix-build-openimageio-2.5.13.0.drv-0/build/include/imageio_pvt.h:229:377:
> error: expected ‘)’ before numeric constant
>   229 | #define OIIO_ALL_BUILD_DEPS_FOUND "Boost 1.83.0;BZip2
> 1.0.8;DCMTK NONE;FFmpeg NONE;fmt 9.1.0;Freetype NONE;GIF 5.2.1;Libheif
> 1.17.6;libjpeg-turbo 2.1.4;LibRaw NONE;OpenColorIO NONE;OpenCV
> NONE;OpenEXR 3.2.4;OpenGL NONE;OpenJPEG NONE;PNG 1.6.39;Ptex NONE;Ptex
> NONE;pugixml 1.12;pybind11 2.8.1;Python 3.10.7;Robinmap;TBB NONE;TIFF
> 4.4.0;WebP NONE;ZLIB 1.3.#define ZLIB_VERSION "1.3""
>       |                                                                                                                                                                                                                                                                                                                                                                                         ^~~
> /tmp/guix-build-openimageio-2.5.13.0.drv-0/source/src/libOpenImageIO/imageio.cpp:623:34:
> note: in expansion of macro ‘OIIO_ALL_BUILD_DEPS_FOUND’
>   623 |         *(ustring*)val = ustring(OIIO_ALL_BUILD_DEPS_FOUND);
>       |                                  ^~~~~~~~~~~~~~~~~~~~~~~~~
> /tmp/guix-build-openimageio-2.5.13.0.drv-0/source/src/libOpenImageIO/imageio.cpp:623:33: note: to match this ‘(’
>   623 |         *(ustring*)val = ustring(OIIO_ALL_BUILD_DEPS_FOUND);
>       |                                 ^
> make[2]: ***
> [src/libOpenImageIO/CMakeFiles/OpenImageIO.dir/build.make:387:
> src/libOpenImageIO/CMakeFiles/OpenImageIO.dir/imageio.cpp.o] Error 1
> make[2]: Leaving directory '/tmp/guix-build-openimageio-2.5.13.0.drv-0/build'
> make[1]: *** [CMakeFiles/Makefile2:1703: src/libOpenImageIO/CMakeFiles/OpenImageIO.dir/all] Error 2
> make[1]: Leaving directory '/tmp/guix-build-openimageio-2.5.13.0.drv-0/build'
> make: *** [Makefile:169: all] Error 2
> error: in phase 'build': uncaught exception:
> %exception #<&invoke-error program: "make" arguments: ("-j" "1") exit-status: 2 term-signal: #f stop-signal: #f> 
> phase `build' failed after 426.2 seconds
> command "make" "-j" "1" failed with status 2
> build process 14 exited with status 256
> builder for `/gnu/store/zbb1ycg7q7iqsp8pa1dyvbsxvh2v94lx-openimageio-2.5.13.0.drv' failed with exit code 1
> ------
>
> I can't find anything online related to this. Any ideas? Full log
> attached.
>
> [2. application/x-gunzip; b1ycg7q7iqsp8pa1dyvbsxvh2v94lx-openimageio-2.5.13.0.drv.gz]...

see patch, as i known, because ours cmake too old, have a bug about zlib.

> Found ZLIB 1.3.#define ZLIB_VERSION "1.3"

``` C
#define OIIO_ALL_BUILD_DEPS_FOUND "Boost 1.83.0;BZip2 1.0.8;DCMTK NONE;FFmpeg NONE;fmt 9.1.0;Freetype NONE;GIF 5.2.1;Libheif 1.17.6;libjpeg-turbo 2.1.4;LibRaw 0.20.0;OpenColorIO NONE;OpenCV NONE;OpenEXR 3.2.4;OpenGL NONE;OpenJPEG NONE;PNG 1.6.39;Ptex NONE;Ptex NONE;pugixml 1.12;pybind11 2.8.1;Python 3.10.7;Robinmap;TBB NONE;TIFF 4.4.0;WebP;ZLIB 1.3.#define ZLIB_VERSION "1.3""
```

shulde be

``` C
#define OIIO_ALL_BUILD_DEPS_FOUND "Boost 1.83.0;BZip2 1.0.8;DCMTK NONE;FFmpeg NONE;fmt 9.1.0;Freetype NONE;GIF 5.2.1;Libheif 1.17.6;libjpeg-turbo 2.1.4;LibRaw 0.20.0;OpenColorIO NONE;OpenCV NONE;OpenEXR 3.2.4;OpenGL NONE;OpenJPEG NONE;PNG 1.6.39;Ptex NONE;Ptex NONE;pugixml 1.12;pybind11 2.8.1;Python 3.10.7;Robinmap;TBB NONE;TIFF 4.4.0;WebP;ZLIB 1.3."
```

so is not a Not a qualified c file.

[0001-gnu-openimageio-Fix-build.patch (text/x-patch, attachment)]
[signature.asc (application/pgp-signature, inline)]

Information forwarded to bug-guix <at> gnu.org:
bug#72046; Package guix. (Thu, 11 Jul 2024 15:13:02 GMT) Full text and rfc822 format available.

Message #11 received at 72046 <at> debbugs.gnu.org (full text, mbox):

From: Leo Famulari <leo <at> famulari.name>
To: Zheng Junjie <zhengjunjie <at> iscas.ac.cn>
Cc: 72046 <at> debbugs.gnu.org
Subject: Re: bug#72046: [core-updates] OpenImageIO fails to build
Date: Thu, 11 Jul 2024 11:10:10 -0400
[Message part 1 (text/plain, inline)]
On Thu, Jul 11, 2024 at 02:29:59PM +0800, Zheng Junjie wrote:
> see patch, as i known, because ours cmake too old, have a bug about zlib.

So, it's a problem with cmake?

Do you know if they've fixed it in later versions?

> +           #:phases #~(modify-phases %standard-phases
> +                        (add-after 'configure 'fix-zlib-version
> +                          (lambda _
> +                            (substitute* "include/imageio_pvt.h"
> +                              (("#define ZLIB_VERSION \"1\\.3\"")
> +                               "")))))))

Would this remove zlib support from openimageio?
[signature.asc (application/pgp-signature, inline)]

Information forwarded to bug-guix <at> gnu.org:
bug#72046; Package guix. (Thu, 11 Jul 2024 15:27:01 GMT) Full text and rfc822 format available.

Message #14 received at 72046 <at> debbugs.gnu.org (full text, mbox):

From: Zheng Junjie <zhengjunjie <at> iscas.ac.cn>
To: Leo Famulari <leo <at> famulari.name>
Cc: 72046 <at> debbugs.gnu.org
Subject: Re: bug#72046: [core-updates] OpenImageIO fails to build
Date: Thu, 11 Jul 2024 23:25:34 +0800
[Message part 1 (text/plain, inline)]
Leo Famulari <leo <at> famulari.name> writes:

> On Thu, Jul 11, 2024 at 02:29:59PM +0800, Zheng Junjie wrote:
>> see patch, as i known, because ours cmake too old, have a bug about zlib.
>
> So, it's a problem with cmake?
>
> Do you know if they've fixed it in later versions?

see https://gitlab.kitware.com/cmake/cmake/-/issues/25200

>
>> +           #:phases #~(modify-phases %standard-phases
>> +                        (add-after 'configure 'fix-zlib-version
>> +                          (lambda _
>> +                            (substitute* "include/imageio_pvt.h"
>> +                              (("#define ZLIB_VERSION \"1\\.3\"")
>> +                               "")))))))
>
> Would this remove zlib support from openimageio?

no, just fix macro OIIO_ALL_BUILD_DEPS_FOUND to a right string.
[signature.asc (application/pgp-signature, inline)]

Information forwarded to bug-guix <at> gnu.org:
bug#72046; Package guix. (Thu, 11 Jul 2024 15:34:02 GMT) Full text and rfc822 format available.

Message #17 received at 72046 <at> debbugs.gnu.org (full text, mbox):

From: Leo Famulari <leo <at> famulari.name>
To: Zheng Junjie <zhengjunjie <at> iscas.ac.cn>
Cc: 72046 <at> debbugs.gnu.org
Subject: Re: bug#72046: [core-updates] OpenImageIO fails to build
Date: Thu, 11 Jul 2024 11:32:53 -0400
[Message part 1 (text/plain, inline)]
On Thu, Jul 11, 2024 at 11:25:34PM +0800, Zheng Junjie wrote:
> > So, it's a problem with cmake?
> >
> > Do you know if they've fixed it in later versions?
> 
> see https://gitlab.kitware.com/cmake/cmake/-/issues/25200

Thanks!

> > Would this remove zlib support from openimageio?
> 
> no, just fix macro OIIO_ALL_BUILD_DEPS_FOUND to a right string.

Okay, great. I confirm your patch fixes the problem. I'll push it to
core-updates on your behalf.
[signature.asc (application/pgp-signature, inline)]

Reply sent to Leo Famulari <leo <at> famulari.name>:
You have taken responsibility. (Thu, 11 Jul 2024 15:51:01 GMT) Full text and rfc822 format available.

Notification sent to Leo Famulari <leo <at> famulari.name>:
bug acknowledged by developer. (Thu, 11 Jul 2024 15:51:02 GMT) Full text and rfc822 format available.

Message #22 received at 72046-done <at> debbugs.gnu.org (full text, mbox):

From: Leo Famulari <leo <at> famulari.name>
To: Zheng Junjie <zhengjunjie <at> iscas.ac.cn>
Cc: 72046-done <at> debbugs.gnu.org
Subject: Re: bug#72046: [core-updates] OpenImageIO fails to build
Date: Thu, 11 Jul 2024 11:50:24 -0400
[Message part 1 (text/plain, inline)]
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!
[signature.asc (application/pgp-signature, inline)]

Information forwarded to bug-guix <at> gnu.org:
bug#72046; Package guix. (Fri, 12 Jul 2024 03:27:02 GMT) Full text and rfc822 format available.

Message #25 received at 72046 <at> debbugs.gnu.org (full text, mbox):

From: Zheng Junjie <zhengjunjie <at> iscas.ac.cn>
To: 72046 <at> debbugs.gnu.org
Cc: leo <at> famulari.name
Subject: Re: bug#72046: [core-updates] OpenImageIO fails to build
Date: Fri, 12 Jul 2024 11:26:16 +0800
[Message part 1 (text/plain, inline)]
Leo Famulari <leo <at> famulari.name> 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.

[0001-gnu-opencolorio-Fix-cmake-file.patch (text/x-patch, inline)]
From f781494940c801db6c63e22ea12c43ffec547456 Mon Sep 17 00:00:00 2001
Message-ID: <f781494940c801db6c63e22ea12c43ffec547456.1720754660.git.zhengjunjie <at> iscas.ac.cn>
From: Zheng Junjie <zhengjunjie <at> iscas.ac.cn>
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

[signature.asc (application/pgp-signature, inline)]

Information forwarded to bug-guix <at> gnu.org:
bug#72046; Package guix. (Fri, 12 Jul 2024 03:44:02 GMT) Full text and rfc822 format available.

Message #28 received at 72046 <at> debbugs.gnu.org (full text, mbox):

From: Zheng Junjie <zhengjunjie <at> iscas.ac.cn>
To: 72046 <at> debbugs.gnu.org
Cc: leo <at> famulari.name
Subject: Re: bug#72046: [core-updates] OpenImageIO fails to build
Date: Fri, 12 Jul 2024 11:43:02 +0800
[Message part 1 (text/plain, inline)]
Zheng Junjie <zhengjunjie <at> iscas.ac.cn> writes:

> Leo Famulari <leo <at> famulari.name> 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: <f781494940c801db6c63e22ea12c43ffec547456.1720754660.git.zhengjunjie <at> iscas.ac.cn>
> From: Zheng Junjie <zhengjunjie <at> iscas.ac.cn>
> 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

and this

[0001-gnu-openimageio-Fix-cmake-file.patch (text/x-patch, inline)]
From dcf47a64e364fb3c59561d535325020368644223 Mon Sep 17 00:00:00 2001
Message-ID: <dcf47a64e364fb3c59561d535325020368644223.1720755723.git.zhengjunjie <at> iscas.ac.cn>
From: Zheng Junjie <zhengjunjie <at> iscas.ac.cn>
Date: Fri, 12 Jul 2024 11:41:21 +0800
Subject: [PATCH] gnu: openimageio: Fix cmake file.

* gnu/packages/graphics.scm (openimageio)[arguments]: Add
fix-OpenImageIOConfig phase.

Change-Id: I0a923c8513069dde5cd3e42fde9279cbd177a10a
---
 gnu/packages/graphics.scm | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/gnu/packages/graphics.scm b/gnu/packages/graphics.scm
index 22ef878972..7286d27b27 100644
--- a/gnu/packages/graphics.scm
+++ b/gnu/packages/graphics.scm
@@ -1359,6 +1359,13 @@ (define-public openimageio
                (add-after 'configure 'fix-zlib-version
                  (lambda _
                    (substitute* "include/imageio_pvt.h"
+                     (("#define ZLIB_VERSION \"1\\.3\"")
+                      ""))))
+               (add-after 'install 'fix-OpenImageIOConfig
+                 (lambda _
+                   (substitute* (string-append
+                                 #$output
+                                 "/lib/cmake/OpenImageIO/OpenImageIOConfig.cmake")
                      (("#define ZLIB_VERSION \"1\\.3\"")
                       "")))))))
     (native-inputs

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

[signature.asc (application/pgp-signature, inline)]

Information forwarded to bug-guix <at> gnu.org:
bug#72046; Package guix. (Fri, 12 Jul 2024 15:19:02 GMT) Full text and rfc822 format available.

Message #31 received at 72046-done <at> debbugs.gnu.org (full text, mbox):

From: Ricardo Wurmus <rekado <at> elephly.net>
To: 72046-done <at> debbugs.gnu.org
Subject: [core-updates] OpenImageIO fails to build
Date: Fri, 12 Jul 2024 17:18:19 +0200
Thank you for the additional patches.  I've applied them and confirm
that Blender now builds fine.

-- 
Ricardo




Information forwarded to bug-guix <at> gnu.org:
bug#72046; Package guix. (Fri, 12 Jul 2024 21:16:02 GMT) Full text and rfc822 format available.

Message #34 received at 72046 <at> debbugs.gnu.org (full text, mbox):

From: "Leo Famulari" <leo <at> famulari.name>
To: "Zheng Junjie" <zhengjunjie <at> iscas.ac.cn>
Cc: 72046 <at> debbugs.gnu.org
Subject: Re: bug#72046: [core-updates] OpenImageIO fails to build
Date: Fri, 12 Jul 2024 17:15:04 -0400
On Thu, Jul 11, 2024, at 11:32, Leo Famulari wrote:
> On Thu, Jul 11, 2024 at 11:25:34PM +0800, Zheng Junjie wrote:
>> > So, it's a problem with cmake?
>> >
>> > Do you know if they've fixed it in later versions?
>> 
>> see https://gitlab.kitware.com/cmake/cmake/-/issues/25200
>
> Thanks!
>
>> > Would this remove zlib support from openimageio?
>> 
>> no, just fix macro OIIO_ALL_BUILD_DEPS_FOUND to a right string.
>
> Okay, great. I confirm your patch fixes the problem. I'll push it to
> core-updates on your behalf.

I made a mistake when I thought that this first revision of the patch fixed the build. I'm not sure what I did wrong, but I'm sorry to cause confusion.

Leo

>
> Attachments:
> * signature.asc




bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Sat, 10 Aug 2024 11:24:09 GMT) Full text and rfc822 format available.

This bug report was last modified 212 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.