GNU bug report logs - #77350
[PATCH 0/6] More OpenEXR 2 to 3 migration

Previous Next

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

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

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

From: Vinicius Monego <monego <at> posteo.net>
To: guix-patches <at> gnu.org
Cc: Vinicius Monego <monego <at> posteo.net>
Subject: [PATCH 0/6] More OpenEXR 2 to 3 migration
Date: Sat, 29 Mar 2025 02:09:56 +0000

This is a continuation of 76712. It updates some packages and builds them with OpenEXR 3 instead of 2.

Vinicius Monego (6):
  gnu: vigra: Update to 1.12.2.
  gnu: enblend-enfuse: Build with OpenEXR 3.
  gnu: hugin: Update to 2024.0.1.
  gnu: synfig: Build with maintained inputs.
  gnu: Remove mlt-6.
  gnu: mlt: Do not override LDFLAGS.

 gnu/packages/animation.scm |  10 +--
 gnu/packages/image.scm     | 124 ++++++++++++++-----------------------
 gnu/packages/photo.scm     |  49 +++++++--------
 gnu/packages/video.scm     |  32 ----------
 4 files changed, 72 insertions(+), 143 deletions(-)


base-commit: a3674c059abf16256f549c4d558a0140b9cd8945
-- 
2.49.0

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

From: Vinicius Monego <monego <at> posteo.net>
To: 77350 <at> debbugs.gnu.org
Cc: Vinicius Monego <monego <at> posteo.net>
Subject: [PATCH 1/6] gnu: vigra: Update to 1.12.2.
Date: Sat, 29 Mar 2025 02:11:31 +0000

* gnu/packages/image.scm (vigra): Update to 1.12.2.
[arguments]<#:phases>: Delete the 'disable-broken-tests phase.
<#:configure-flags>: Don't pass "-DCMAKE_CXX_FLAGS", "-DCMAKE_C_FLAGS",
"-Wno-dev".
[inputs]: Remove package labels. Replace openexr-2 by openexr. Remove ilmbase.

Change-Id: Ie953a45f6b5e9da5985e307d4acf25a955e73539
---
 gnu/packages/image.scm | 124 +++++++++++++++--------------------------
 1 file changed, 45 insertions(+), 79 deletions(-)

diff --git a/gnu/packages/image.scm b/gnu/packages/image.scm
index 89e7ef83e6..41c1a55f4a 100644
--- a/gnu/packages/image.scm
+++ b/gnu/packages/image.scm
@@ -28,7 +28,7 @@
 ;;; Copyright © 2020, 2023 Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
 ;;; Copyright © 2020 Jan (janneke) Nieuwenhuizen <janneke <at> gnu.org>
 ;;; Copyright © 2020 Zhu Zihao <all_but_last <at> 163.com>
-;;; Copyright © 2020, 2021, 2022, 2023, 2024 Vinicius Monego <monego <at> posteo.net>
+;;; Copyright © 2020-2025 Vinicius Monego <monego <at> posteo.net>
 ;;; Copyright © 2021 Sharlatan Hellseher <sharlatanus <at> gmail.com>
 ;;; Copyright © 2021 Nicolò Balzarotti <nicolo <at> nixo.xyz>
 ;;; Copyright © 2021 Alexandr Vityazev <avityazev <at> posteo.org>
@@ -1365,87 +1365,53 @@ (define-public freeimage
     (home-page "https://freeimage.sourceforge.io/")))
 
 (define-public vigra
-    (let ((commit "9b514fa00a136f5fd81bb57ee9f6293c333ffc1f")
-          (revision "0"))
-    (package
-     (name "vigra")
-     (version (git-version "1.11.1" revision commit))
-     (source
-      (origin
-        ;; The last release is 1.11.1, from 2017. It's becoming more and more
-        ;; difficult to build this old release, and the upstream developers
-        ;; suggest on their home page to build from the Git repo, saying "It is
-        ;; generally safe to use the 'master' branch of the development snapshot,
-        ;; as we avoid uploading untested or incompatible changes to this branch."
-        (method git-fetch)
-        (uri (git-reference
-               (url "https://github.com/ukoethe/vigra")
-               (commit commit)))
-        (file-name (git-file-name name version))
-        (sha256 (base32
-                  "1vzlypviala109imwxkp46lqhhxszf79ypfb8wxg6z7g02j7mm73"))))
-     (build-system cmake-build-system)
-     (inputs
-      `(("boost" ,boost)
-        ("fftw" ,fftw)
-        ("fftwf" ,fftwf)
-        ("hdf5" ,hdf5)
-        ("ilmbase" ,ilmbase) ; propagated by openexr, but needed explicitly
-                             ; to create a configure-flag
-        ("libjpeg" ,libjpeg-turbo)
-        ("libpng" ,libpng)
-        ("libtiff" ,libtiff)
-        ("openexr" ,openexr-2)
-        ("python" ,python-wrapper)
-        ;("python-numpy" ,python-numpy)
-        ("zlib" ,zlib)))
-     (native-inputs
-      `(("doxygen" ,doxygen)
-        ("python-nose" ,python-nose)
-        ("sphinx" ,python-sphinx)))
-     (arguments
-      `(#:test-target "check"
-        #:phases
-        (modify-phases %standard-phases
-          (add-after 'unpack 'disable-broken-tests
-            (lambda _
-              ;; See https://github.com/ukoethe/vigra/issues/432
-              (substitute* "test/fourier/CMakeLists.txt"
-                (("VIGRA_ADD_TEST.*") ""))
-              ;; This test fails with Numpy 1.15:
-              ;; <https://github.com/ukoethe/vigra/issues/436>.
-              (substitute* "vigranumpy/test/CMakeLists.txt"
-                (("test1\\.py") ""))
-              #t)))
-        #:configure-flags
-          (list "-Wno-dev" ; suppress developer mode with lots of warnings
-                (string-append "-DVIGRANUMPY_INSTALL_DIR="
-                               (assoc-ref %outputs "out")
-                               "/lib/python"
-                               ,(version-major+minor (package-version python))
-                               "/site-packages")
-                ;; Vigranumpy isn't compatible with numpy >= 1.20.
-                "-DWITH_VIGRANUMPY=0"
-                ;; OpenEXR is not enabled by default.
-                "-DWITH_OPENEXR=1"
-                ;; Fix rounding error on 32-bit machines
-                "-DCMAKE_C_FLAGS=-ffloat-store"
-                ;; The header files of ilmbase are not found when included
-                ;; by the header files of openexr, and an explicit flag
-                ;; needs to be set.
-                (string-append "-DCMAKE_CXX_FLAGS=-I"
-                               (assoc-ref %build-inputs "ilmbase")
-                               "/include/OpenEXR"
-                               " -ffloat-store"))))
-     (synopsis "Computer vision library")
-     (description
-      "VIGRA stands for Vision with Generic Algorithms.  It is an image
+  (package
+    (name "vigra")
+    (version "1.12.2")
+    (source
+     (origin
+       (method git-fetch)
+       (uri (git-reference
+             (url "https://github.com/ukoethe/vigra")
+             (commit "Version-1-12-1")))
+       (file-name (git-file-name name version))
+       (sha256
+        (base32 "1pshpdylji9yz6n3h6rc6njgjih4cas8cf8jpd1c185j2kaf6qb6"))))
+    (build-system cmake-build-system)
+    (arguments
+     `(#:test-target "check"
+       #:configure-flags
+       (list (string-append "-DVIGRANUMPY_INSTALL_DIR="
+                            (assoc-ref %outputs "out")
+                            "/lib/python"
+                            ,(version-major+minor (package-version python))
+                            "/site-packages")
+             ;; Vigranumpy isn't compatible with numpy >= 1.20.
+             "-DWITH_VIGRANUMPY=0"
+             ;; OpenEXR is not enabled by default.
+             "-DWITH_OPENEXR=1")))
+    (native-inputs (list doxygen
+                         python-nose
+                         python-sphinx))
+    (inputs (list boost
+                  fftw
+                  fftwf
+                  hdf5
+                  libjpeg-turbo
+                  libpng
+                  libtiff
+                  openexr
+                  python-wrapper
+                  zlib))
+    (synopsis "Computer vision library")
+    (description
+     "VIGRA stands for Vision with Generic Algorithms.  It is an image
   processing and analysis library that puts its main emphasis on customizable
   algorithms and data structures.  It is particularly strong for
   multi-dimensional image processing.")
-     (license license:expat)
-     (home-page "https://ukoethe.github.io/vigra/")
-     (properties '((max-silent-time . 7200)))))) ;2 hours, to avoid timing out
+    (license license:expat)
+    (home-page "https://ukoethe.github.io/vigra/")
+    (properties '((max-silent-time . 7200))))) ;2 hours, to avoid timing out
 
 (define-public vigra-c
   (let* ((commit "66ff4fa5a7d4a77415caa676a45c2c6ea16562e7")
-- 
2.49.0

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

From: Vinicius Monego <monego <at> posteo.net>
To: 77350 <at> debbugs.gnu.org
Cc: Vinicius Monego <monego <at> posteo.net>
Subject: [PATCH 2/6] gnu: enblend-enfuse: Build with OpenEXR 3.
Date: Sat, 29 Mar 2025 02:11:32 +0000

* gnu/packages/photo.scm (enblend-enfuse)[inputs]: Replace openexr-2 by
openexr.

Change-Id: Ie9bbc0bb5067e3141284ae5061743dc59aca45ca
---
 gnu/packages/photo.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gnu/packages/photo.scm b/gnu/packages/photo.scm
index a8401db67f..5b34cd135c 100644
--- a/gnu/packages/photo.scm
+++ b/gnu/packages/photo.scm
@@ -431,7 +431,7 @@ (define-public enblend-enfuse
            libjpeg-turbo
            libpng
            libtiff
-           openexr-2
+           openexr
            vigra
            zlib))
     (arguments
-- 
2.49.0

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

From: Vinicius Monego <monego <at> posteo.net>
To: 77350 <at> debbugs.gnu.org
Cc: Vinicius Monego <monego <at> posteo.net>
Subject: [PATCH 3/6] gnu: hugin: Update to 2024.0.1.
Date: Sat, 29 Mar 2025 02:11:33 +0000

* gnu/packages/photo.scm (hugin): Update to 2024.0.1.
[arguments]: Use G-Expressions.
<#:configure-flags>: Unset "-DCMAKE_CXX_FLAGS".
[inputs]: Replace openexr-2 by openexr.

Change-Id: I5703ef40301b72a54865e11ef560ccc55c657b53
---
 gnu/packages/photo.scm | 47 +++++++++++++++++++-----------------------
 1 file changed, 21 insertions(+), 26 deletions(-)

diff --git a/gnu/packages/photo.scm b/gnu/packages/photo.scm
index 5b34cd135c..c71d498a5a 100644
--- a/gnu/packages/photo.scm
+++ b/gnu/packages/photo.scm
@@ -752,7 +752,7 @@ (define-public entangle
 (define-public hugin
   (package
     (name "hugin")
-    (version "2021.0.0")
+    (version "2024.0.1")
     (source (origin
               (method url-fetch)
               (uri (string-append "mirror://sourceforge/hugin/hugin/hugin-"
@@ -760,7 +760,7 @@ (define-public hugin
                                   "/hugin-" version ".tar.bz2"))
               (sha256
                (base32
-                "1ngadsv22ii05kmvpzdivhwlks4pnv9ijz7j9srl8y54gy5flyh4"))))
+                "1r57bgq9dr2wi182vl6qm2kgaz2f6wz8sxikr14k3djfxgg0rv0k"))))
     (build-system cmake-build-system)
     (native-inputs
      (list gettext-minimal pkg-config))
@@ -780,35 +780,30 @@ (define-public hugin
            libxi
            libxmu
            mesa
-           openexr-2
+           openexr
            sqlite
            vigra
            wxwidgets
            zlib))
     (arguments
-     `(#:tests? #f                      ; no check target
-       #:configure-flags
-       (list
-        ;; The header files of ilmbase (propagated by openexr) are not found
-        ;; when included by the header files of openexr, and an explicit
-        ;; flag needs to be set.
-        (string-append "-DCMAKE_CXX_FLAGS=-I"
-                       (assoc-ref %build-inputs "ilmbase")
-                       "/include/OpenEXR")
-        ;; Disable installation of the Python scripting interface.
-        ;; It would require the additional inputs python and swig.
-        ;; Installation would need to be tweaked, as it tries to install
-        ;; into the python directory.
-        "-DBUILD_HSI=OFF")
-       #:phases
-       (modify-phases %standard-phases
-         (add-before 'configure 'substitute
-           (lambda _
-             (substitute* "src/hugin1/base_wx/StitchingExecutor.cpp"
-               (("wxT\\(\"enblend\"\\)")
-                (string-append "wxT(\"" (which "enblend") "\")"))
-               (("wxT\\(\"enfuse\"\\)")
-                (string-append "wxT(\"" (which "enfuse") "\")"))))))))
+     (list
+      #:tests? #f                      ; no check target
+      #:configure-flags
+      #~(list
+         ;; Disable installation of the Python scripting interface.
+         ;; It would require the additional inputs python and swig.
+         ;; Installation would need to be tweaked, as it tries to install
+         ;; into the python directory.
+         "-DBUILD_HSI=OFF")
+      #:phases
+      #~(modify-phases %standard-phases
+          (add-before 'configure 'substitute
+            (lambda _
+              (substitute* "src/hugin1/base_wx/StitchingExecutor.cpp"
+                (("wxT\\(\"enblend\"\\)")
+                 (string-append "wxT(\"" (which "enblend") "\")"))
+                (("wxT\\(\"enfuse\"\\)")
+                 (string-append "wxT(\"" (which "enfuse") "\")"))))))))
     (home-page "https://hugin.sourceforge.net/")
     (synopsis "Panorama photo stitcher")
     (description
-- 
2.49.0

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

From: Vinicius Monego <monego <at> posteo.net>
To: 77350 <at> debbugs.gnu.org
Cc: Vinicius Monego <monego <at> posteo.net>
Subject: [PATCH 4/6] gnu: synfig: Build with maintained inputs.
Date: Sat, 29 Mar 2025 02:11:34 +0000

* gnu/packages/animation.scm (synfig)[inputs]: Replace ffmpeg-4 by
ffmpeg, mlt-6 by mlt, openexr-2 by openexr.

Change-Id: I6c99cfbf7b828ad790e10a4eebf9103e15ffb4e9
---
 gnu/packages/animation.scm | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/gnu/packages/animation.scm b/gnu/packages/animation.scm
index b2e33a71a0..3db0539ad6 100644
--- a/gnu/packages/animation.scm
+++ b/gnu/packages/animation.scm
@@ -146,16 +146,16 @@ (define-public synfig
     (propagated-inputs
      (list cairo
            etl
-           ffmpeg-4
+           ffmpeg
            fftw
            fontconfig
            freetype
-           glibmm-2.66
+           glibmm-2.66 ; latest possible version as of synfig 1.5.3
            imagemagick
-           libxml++-2
+           libxml++-2 ; libxml <= 2 required as of synfig 1.5.3
            libsigc++
-           mlt-6
-           openexr-2
+           mlt
+           openexr
            pango))
     (native-inputs
      (list intltool pkg-config))
-- 
2.49.0

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

From: Vinicius Monego <monego <at> posteo.net>
To: 77350 <at> debbugs.gnu.org
Cc: Vinicius Monego <monego <at> posteo.net>
Subject: [PATCH 5/6] gnu: Remove mlt-6.
Date: Sat, 29 Mar 2025 02:11:35 +0000

This is a deprecated version of MLT which had synfig as its last
dependent. Synfig now builds with MLT 7, making mlt-6 obsolete.

* gnu/packages/video.scm (mlt-6): Delete variable.

Change-Id: I9bbe57d89b1a66ffdcc1d4f3f520c72d57923770
---
 gnu/packages/video.scm | 28 ----------------------------
 1 file changed, 28 deletions(-)

diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
index 10df7e908f..98a64dd879 100644
--- a/gnu/packages/video.scm
+++ b/gnu/packages/video.scm
@@ -3964,34 +3964,6 @@ (define-public mlt
 tools, XML authoring components, and an extensible plug-in based API.")
     (license license:lgpl2.1+)))
 
-(define-public mlt-6
-  (package
-    (inherit mlt)
-    (name "mlt")
-    (version "6.26.1")
-    (source
-     (origin
-       (method git-fetch)
-       (uri (git-reference
-             (url "https://github.com/mltframework/mlt")
-             (commit (string-append "v" version))))
-       (file-name (git-file-name name version))
-       (sha256
-        (base32 "1gz79xvs5jrzqhwhfk0dqdd3xiavnjp4q957h7nb02rij32byb39"))))
-    (arguments
-     `(#:configure-flags
-       (list (string-append "-DGTK2_GDKCONFIG_INCLUDE_DIR="
-                            (assoc-ref %build-inputs "gtk+")
-                             "/lib/gtk-2.0/include")
-             (string-append "-DGTK2_GLIBCONFIG_INCLUDE_DIR="
-                            (assoc-ref %build-inputs "glib")
-                            "/lib/glib-2.0/include"))
-       ,@(package-arguments mlt)))
-    (inputs
-     (modify-inputs (package-inputs mlt)
-       (replace "ffmpeg" ffmpeg-4)
-       (replace "gtk+" gtk+-2)))))
-
 (define-public v4l-utils
   (package
     (name "v4l-utils")
-- 
2.49.0

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

From: Vinicius Monego <monego <at> posteo.net>
To: 77350 <at> debbugs.gnu.org
Cc: Vinicius Monego <monego <at> posteo.net>
Subject: [PATCH 6/6] gnu: mlt: Do not override LDFLAGS.
Date: Sat, 29 Mar 2025 02:11:36 +0000

* gnu/packages/video.scm (mlt)[arguments]<#:phases>: Remove obsolete
'override-LDFLAGS phase.

Change-Id: Ib34138ab85dab20897d06f08697e99358954050f
---
 gnu/packages/video.scm | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
index 98a64dd879..d15825dcf7 100644
--- a/gnu/packages/video.scm
+++ b/gnu/packages/video.scm
@@ -3907,10 +3907,6 @@ (define-public mlt
       #:tests? #f                       ;requires "Kwalify"
       #:phases
       #~(modify-phases %standard-phases
-          (add-before 'configure 'override-LDFLAGS
-            (lambda _
-              (setenv "LDFLAGS"
-                      (string-append "-Wl,-rpath=" #$output "/lib"))))
           (add-after 'install 'wrap-executable
             (lambda _
               (let* ((frei0r #$(this-package-input "frei0r-plugins"))
-- 
2.49.0

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

From: Andreas Enge <andreas <at> enge.fr>
To: Vinicius Monego <monego <at> posteo.net>
Cc: 77350 <at> debbugs.gnu.org
Subject: Re: More OpenEXR 2 to 3 migration
Date: Wed, 2 Apr 2025 00:16:19 +0200

Hello Vinicius,

I have independently worked on vigra/hugin during the last days, and
actually pushed an update of vigra to 1.11.2 today (I wanted to do the
most conservative update possible to ensure depending packages still
build and not go down the rabbit hole).

Right now I am updating to 1.12.2 and will push once I have made sure
that all dependents still build. The aarch64 package also builds.
I have not moved to openexr <at> 3 and have also not removed ilmbase; what is
the motivation for this latest change? However I am building with
python-numpy <at> 2, which has been made possible in the 1.12 release.
Could you wait a day or two and then maybe adapt your changes to the new
package?

I also tried to update hugin, but gave up when I noticed that it would not
even start. Actually the current hugin with the previous vigra also does
not start (it opens a window with an error message). Could you maybe
give it a try?

The issue has been treated by QA, and there is an openexr related
problem in vigra-c:
   https://bordeaux.guix.gnu.org/build/5b70b1d1-2bba-4e60-b131-743f037c197f/log
This project has seen its latest commit in 2022, and may not be ready
for a newer openexr version. We might consider removing it, but there is
a depending package, guile-cv. This package is also not very actively
developed:
   https://git.savannah.gnu.org/cgit/guile-cv.git
Removing these two packages might be an option, but would make sense
mainly if it enables us to get rid of openexr <at> 2.

Thanks,

Andreas

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

From: Andreas Enge <andreas <at> enge.fr>
To: Vinicius Monego <monego <at> posteo.net>
Cc: 77350 <at> debbugs.gnu.org
Subject: Re: More OpenEXR 2 to 3 migration
Date: Wed, 2 Apr 2025 09:57:17 +0200

Am Wed, Apr 02, 2025 at 12:16:19AM +0200 schrieb Andreas Enge:
> Right now I am updating to 1.12.2 and will push once I have made sure
> that all dependents still build. The aarch64 package also builds.

Done! Please feel free to work on this package complex, I am not going
to do so for the time being, but will be available for discussions.

Andreas

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

From: Vinicius Monego <monego <at> posteo.net>
To: Andreas Enge <andreas <at> enge.fr>
Cc: 77350 <at> debbugs.gnu.org
Subject: Re: More OpenEXR 2 to 3 migration
Date: Mon,  7 Apr 2025 11:17:13 +0000

Em 01/04/2025 19:16, Andreas Enge escreveu:
> Hello Vinicius,
Hello Andreas,
>
> I have independently worked on vigra/hugin during the last days, and
> actually pushed an update of vigra to 1.11.2 today (I wanted to do the
> most conservative update possible to ensure depending packages still
> build and not go down the rabbit hole).
>
> Right now I am updating to 1.12.2 and will push once I have made sure
> that all dependents still build. The aarch64 package also builds.
> I have not moved to openexr <at> 3 and have also not removed ilmbase; what is
> the motivation for this latest change?

The OpenEXR 2 situation may be comparable to GTK2. While not officially 
deprecated (AFAIK), it won't receive any feature updates and I don't 
know whether it is still receiving security updates.

Image processing software (which often handles complex numerical 
operations and data structures) is more likely to have security issues 
related to overflows and openexr has had some in its past. 'guix lint -c 
cve openexr <at> 2' reports one CVE and future CVEs are unlikely to be fixed. 
Upstream also recommends against using version 2: 
https://openexr.com/en/latest/install.html

For guix, it also simplifies the package graph as Hugin depends on both 
openexr 3 and 2 by different inputs.

> However I am building with
> python-numpy <at> 2, which has been made possible in the 1.12 release.
> Could you wait a day or two and then maybe adapt your changes to the new
> package?
>
> I also tried to update hugin, but gave up when I noticed that it would not
> even start. Actually the current hugin with the previous vigra also does
> not start (it opens a window with an error message). Could you maybe
> give it a try?
I did try and had the same result. The current build is also broken 
although for a different reason (I am greeted by a GTK debug screen). I 
am unfamiliar with the program, so I don't know what to expect when it 
works.
>
> The issue has been treated by QA, and there is an openexr related
> problem in vigra-c:
>     https://bordeaux.guix.gnu.org/build/5b70b1d1-2bba-4e60-b131-743f037c197f/log
> This project has seen its latest commit in 2022, and may not be ready
> for a newer openexr version. We might consider removing it, but there is
> a depending package, guile-cv. This package is also not very actively
> developed:
>     https://git.savannah.gnu.org/cgit/guile-cv.git
> Removing these two packages might be an option, but would make sense
> mainly if it enables us to get rid of openexr <at> 2.
>
> Thanks,
>
> Andreas
>
Currently there are only 15 packages having openexr-2 as input. I also 
sent a patch series at 76712 to migrate some more. I have not checked 
all 15, but I estimate less than 5 of them have to depend on openexr-2.

Will have a look at what is happening to vigra-c whenever possible.

Vinicius

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

From: Andreas Enge <andreas <at> enge.fr>
To: Vinicius Monego <monego <at> posteo.net>
Cc: 77350 <at> debbugs.gnu.org
Subject: Re: More OpenEXR 2 to 3 migration
Date: Mon, 7 Apr 2025 13:29:14 +0200

Hello Vinicius,

Am Mon, Apr 07, 2025 at 11:17:13AM +0000 schrieb Vinicius Monego:
> > Right now I am updating to 1.12.2 and will push once I have made sure
> > that all dependents still build. The aarch64 package also builds.
> > I have not moved to openexr <at> 3 and have also not removed ilmbase; what is
> > the motivation for this latest change?
> The OpenEXR 2 situation may be comparable to GTK2. While not officially
> deprecated (AFAIK), it won't receive any feature updates and I don't know
> whether it is still receiving security updates.

there has been a misunderstanding; I suppose that moving from @2 to @3
is a good thing, but why remove ilmbase at the same time? Is ilmbase
related to openexr <at> 2? (I know nothing about these packages, I have just
used hugin in the past, so my question may well be naive.)

Thanks for your effort in updating packages! I think you can push at
your own discretion once it passes QA, or after a waiting period of one
week if it has not been picked up by QA and all dependent packages build
locally on x86_64.

> For guix, it also simplifies the package graph as Hugin depends on both
> openexr 3 and 2 by different inputs.

Ah, that sounds bad, and like a source of bugs indeed!

Andreas

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

From: Vinicius Monego <monego <at> posteo.net>
To: Andreas Enge <andreas <at> enge.fr>
Cc: 77350 <at> debbugs.gnu.org
Subject: Re: More OpenEXR 2 to 3 migration
Date: Mon,  7 Apr 2025 11:45:18 +0000

Em 07/04/2025 08:29, Andreas Enge escreveu:
> Hello Vinicius,
>
> Am Mon, Apr 07, 2025 at 11:17:13AM +0000 schrieb Vinicius Monego:
>>> Right now I am updating to 1.12.2 and will push once I have made sure
>>> that all dependents still build. The aarch64 package also builds.
>>> I have not moved to openexr <at> 3 and have also not removed ilmbase; what is
>>> the motivation for this latest change?
>> The OpenEXR 2 situation may be comparable to GTK2. While not officially
>> deprecated (AFAIK), it won't receive any feature updates and I don't know
>> whether it is still receiving security updates.
> there has been a misunderstanding; I suppose that moving from @2 to @3
> is a good thing, but why remove ilmbase at the same time? Is ilmbase
> related to openexr <at> 2? (I know nothing about these packages, I have just
> used hugin in the past, so my question may well be naive.)

Oh, I see.

Yes, it is documented in the porting guide:

https://openexr.com/en/latest/PortingGuide.html

> Thus, with the 2.4 release, the “IlmBase” libraries are no longer 
distributed in a form that is readily separable from the rest of OpenEXR.

IlmBase had part of it moved into the 'imath' package and other 
remaining functionality was merged into the openexr tree.

Vinicius

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

From: Andreas Enge <andreas <at> enge.fr>
To: Vinicius Monego <monego <at> posteo.net>
Cc: 77350 <at> debbugs.gnu.org
Subject: Re: More OpenEXR 2 to 3 migration
Date: Mon, 7 Apr 2025 13:49:11 +0200

Am Mon, Apr 07, 2025 at 11:45:18AM +0000 schrieb Vinicius Monego:
> Yes, it is documented in the porting guide:
> https://openexr.com/en/latest/PortingGuide.html
> > Thus, with the 2.4 release, the “IlmBase” libraries are no longer
> distributed in a form that is readily separable from the rest of OpenEXR.
> IlmBase had part of it moved into the 'imath' package and other remaining
> functionality was merged into the openexr tree.

Okay, I see! Then removing ilmbase will be a goal at the same time as
removing openexr <at> 2. Right now ilmbase has 186 dependents (including
gnome), but this should go down with the openexr <at> 2 to @3 migration,
or maybe even disappear.

Andreas

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

From: Vinicius Monego <monego <at> posteo.net>
To: Andreas Enge <andreas <at> enge.fr>
Cc: 77350 <at> debbugs.gnu.org
Subject: Re: More OpenEXR 2 to 3 migration
Date: Mon,  7 Apr 2025 11:59:15 +0000

Em 07/04/2025 08:49, Andreas Enge escreveu:
> Am Mon, Apr 07, 2025 at 11:45:18AM +0000 schrieb Vinicius Monego:
>> Yes, it is documented in the porting guide:
>> https://openexr.com/en/latest/PortingGuide.html
>>> Thus, with the 2.4 release, the “IlmBase” libraries are no longer
>> distributed in a form that is readily separable from the rest of OpenEXR.
>> IlmBase had part of it moved into the 'imath' package and other remaining
>> functionality was merged into the openexr tree.
> Okay, I see! Then removing ilmbase will be a goal at the same time as
> removing openexr <at> 2. Right now ilmbase has 186 dependents (including
> gnome), but this should go down with the openexr <at> 2 to @3 migration,
> or maybe even disappear.
>
> Andreas
>
Most of them are because of freeimage, which I am trying to solve in 76354.

I also noticed that ilmbase is an input to opencv, but that might be 
another unused input.

Vinicius

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

From: Andreas Enge <andreas <at> enge.fr>
To: Vinicius Monego <monego <at> posteo.net>
Cc: 77350-done <at> debbugs.gnu.org
Subject: Re: [PATCH 1/6] gnu: vigra: Update to 1.12.2.
Date: Thu, 1 May 2025 09:06:04 +0200

Hello Vinicius,

I have pushed all commits except for the obsolete vigra one.

Hugin segfaults directly upon starting, but a similar error also
occurred for the previous version, so there is no regression...
At least we could start debugging from the latest version.

Andreas

Previous Next

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