GNU bug report logs - #42807
[PATCH 1/2] gnu: darktable: Update to 3.2.1.

Previous Next

Package: guix-patches;

Reported by: Vinicius Monego <monego <at> posteo.net>

Date: Mon, 10 Aug 2020 23:24:02 UTC

Severity: normal

Tags: patch

Done: Ludovic Courtès <ludo <at> gnu.org>

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 42807 in the body.
You can then email your comments to 42807 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 guix-patches <at> gnu.org:
bug#42807; Package guix-patches. (Mon, 10 Aug 2020 23:24:02 GMT) Full text and rfc822 format available.

Acknowledgement sent to Vinicius Monego <monego <at> posteo.net>:
New bug report received and forwarded. Copy sent to guix-patches <at> gnu.org. (Mon, 10 Aug 2020 23:24:02 GMT) Full text and rfc822 format available.

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 1/2] gnu: darktable: Update to 3.2.1.
Date: Mon, 10 Aug 2020 20:23:25 -0300
* gnu/packages/photo.scm (darktable): update to 3.2.1.
---
 gnu/packages/photo.scm | 29 +++++++++++++++--------------
 1 file changed, 15 insertions(+), 14 deletions(-)

diff --git a/gnu/packages/photo.scm b/gnu/packages/photo.scm
index abdb751a5a..6be0094a8d 100644
--- a/gnu/packages/photo.scm
+++ b/gnu/packages/photo.scm
@@ -465,7 +465,7 @@ photographic equipment.")
 (define-public darktable
   (package
     (name "darktable")
-    (version "3.0.2")
+    (version "3.2.1")
     (source
      (origin
        (method url-fetch)
@@ -473,7 +473,7 @@ photographic equipment.")
              "https://github.com/darktable-org/darktable/releases/"
              "download/release-" version "/darktable-" version ".tar.xz"))
        (sha256
-        (base32 "1yrnkw8c47kmy2x6m1xp69hwyk02xyc8pd9kvcmyj54lzrhzdfka"))))
+        (base32 "035rvqmw386hm0jpi14lf4dnpr5rjkalzjkyprqh42nwi3m86dkf"))))
     (build-system cmake-build-system)
     (arguments
      `(#:tests? #f                      ; there are no tests
@@ -482,6 +482,7 @@ photographic equipment.")
        (modify-phases %standard-phases
          (add-before 'configure 'prepare-build-environment
            (lambda* (#:key inputs #:allow-other-keys)
+             ;; Build fails with every version of GCC.
              (setenv "CC" "clang") (setenv "CXX" "clang++")
              ;; Darktable looks for opencl-c.h in the LLVM dir. Guix installs
              ;; it to the Clang dir. We fix this by patching CMakeLists.txt.
@@ -524,35 +525,35 @@ photographic equipment.")
     (inputs
      `(("cairo" ,cairo)
        ("colord-gtk" ,colord-gtk)
-       ("cups" ,cups)
+       ("cups" ,cups) ;optional, for printing support
        ("curl" ,curl)
        ("dbus-glib" ,dbus-glib)
        ("exiv2" ,exiv2)
        ("freeimage" ,freeimage)
-       ("gmic" ,gmic)
+       ("gmic" ,gmic) ;optional
        ("graphicsmagick" ,graphicsmagick)
        ("gsettings-desktop-schemas" ,gsettings-desktop-schemas)
        ("gtk+" ,gtk+)
        ("ilmbase" ,ilmbase)
-       ("iso-codes" ,iso-codes)
+       ("iso-codes" ,iso-codes) ;optional
        ("json-glib" ,json-glib)
        ("lcms" ,lcms)
-       ("lensfun" ,lensfun)
-       ("libgphoto2" ,libgphoto2)
+       ("lensfun" ,lensfun) ;optional
+       ("libgphoto2" ,libgphoto2) ;optional
        ("libjpeg" ,libjpeg-turbo)
        ("libomp" ,libomp)
        ("libpng" ,libpng)
        ("librsvg" ,librsvg)
-       ("libsecret" ,libsecret)
-       ("libsoup" ,libsoup)
+       ("libsecret" ,libsecret) ;optional
+       ("libsoup" ,libsoup) ;optional
        ("libtiff" ,libtiff)
-       ("libwebp" ,libwebp)
-       ("libxml2" ,libxml2)
+       ("libwebp" ,libwebp) ;optional, for WebP support
+       ("libxml2" ,libxml2) ;optional, for validation of data/cameras.xml
        ("libxslt" ,libxslt)
-       ("lua" ,lua) ;for plugins
+       ("lua" ,lua) ;optional, for plugins
        ("openexr" ,openexr)
-       ("openjpeg" ,openjpeg)
-       ("osm-gps-map" ,osm-gps-map)
+       ("openjpeg" ,openjpeg) ;optional
+       ("osm-gps-map" ,osm-gps-map) ;optional,
        ("pugixml" ,pugixml)
        ("python-jsonschema" ,python-jsonschema)
        ("sqlite" ,sqlite)))
-- 
2.20.1





Information forwarded to guix-patches <at> gnu.org:
bug#42807; Package guix-patches. (Mon, 10 Aug 2020 23:26:01 GMT) Full text and rfc822 format available.

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

From: Vinicius Monego <monego <at> posteo.net>
To: 42807 <at> debbugs.gnu.org
Cc: Vinicius Monego <monego <at> posteo.net>
Subject: [PATCH 2/2] gnu: darktable: Add dependency on libavif, ocl-icd.
Date: Mon, 10 Aug 2020 20:25:15 -0300
* gnu/packages/photo.scm (darktable)[inputs]: Add libavif, ocl-icd.
[arguments]: Set LD_LIBRARY_PATH for OpenCL.
[supported-systems]: Remove i686 which is not supported anymore.
---
 gnu/packages/photo.scm | 26 ++++++++++++++++----------
 1 file changed, 16 insertions(+), 10 deletions(-)

diff --git a/gnu/packages/photo.scm b/gnu/packages/photo.scm
index 6be0094a8d..1b82cef8df 100644
--- a/gnu/packages/photo.scm
+++ b/gnu/packages/photo.scm
@@ -502,15 +502,19 @@ photographic equipment.")
                      (string-append (assoc-ref inputs "ilmbase")
                                     "/include/OpenEXR:" (or (getenv "CPATH") "")))
              #t))
-          (add-after 'install 'wrap-program
-            (lambda* (#:key inputs outputs #:allow-other-keys)
-              (wrap-program (string-append (assoc-ref outputs "out")
-                                           "/bin/darktable")
-                ;; For GtkFileChooserDialog.
-                `("GSETTINGS_SCHEMA_DIR" =
-                  (,(string-append (assoc-ref inputs "gtk+")
-                                   "/share/glib-2.0/schemas"))))
-              #t)))))
+         (add-after 'install 'wrap-program
+           (lambda* (#:key inputs outputs #:allow-other-keys)
+             (wrap-program (string-append (assoc-ref outputs "out")
+                                          "/bin/darktable")
+               ;; For GtkFileChooserDialog.
+               `("GSETTINGS_SCHEMA_DIR" =
+                 (,(string-append (assoc-ref inputs "gtk+")
+                                  "/share/glib-2.0/schemas")))
+               ;; For libOpenCL.so.
+               `("LD_LIBRARY_PATH" =
+                 (,(string-append (assoc-ref inputs "ocl-icd")
+                                  "/lib"))))
+             #t)))))
     (native-inputs
      `(("clang" ,clang-9)
        ("desktop-file-utils" ,desktop-file-utils)
@@ -539,6 +543,7 @@ photographic equipment.")
        ("json-glib" ,json-glib)
        ("lcms" ,lcms)
        ("lensfun" ,lensfun) ;optional
+       ("libavif" ,libavif) ;optional, for AVIF support
        ("libgphoto2" ,libgphoto2) ;optional
        ("libjpeg" ,libjpeg-turbo)
        ("libomp" ,libomp)
@@ -551,6 +556,7 @@ photographic equipment.")
        ("libxml2" ,libxml2) ;optional, for validation of data/cameras.xml
        ("libxslt" ,libxslt)
        ("lua" ,lua) ;optional, for plugins
+       ("ocl-icd" ,ocl-icd) ;optional, for OpenCL support
        ("openexr" ,openexr)
        ("openjpeg" ,openjpeg) ;optional
        ("osm-gps-map" ,osm-gps-map) ;optional,
@@ -564,7 +570,7 @@ developer.  It manages your digital negatives in a database, lets you view
 them through a zoomable lighttable and enables you to develop raw images
 and enhance them.")
     ;; See src/is_supported_platform.h for supported platforms.
-    (supported-systems '("i686-linux" "x86_64-linux" "aarch64-linux"))
+    (supported-systems '("x86_64-linux" "aarch64-linux"))
     (license (list license:gpl3+ ;; Darktable itself.
                    license:lgpl2.1+)))) ;; Rawspeed library.
 
-- 
2.20.1





Information forwarded to guix-patches <at> gnu.org:
bug#42807; Package guix-patches. (Mon, 10 Aug 2020 23:32:01 GMT) Full text and rfc822 format available.

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

From: Brett Gilio <brettg <at> gnu.org>
To: Vinicius Monego <monego <at> posteo.net>
Cc: 42807 <at> debbugs.gnu.org
Subject: Re: [bug#42807] [PATCH 1/2] gnu: darktable: Update to 3.2.1.
Date: Mon, 10 Aug 2020 18:31:21 -0500
Vinicius Monego <monego <at> posteo.net> writes:

--8<---------------cut here---------------start------------->8---
> +             ;; Build fails with every version of GCC.
>               (setenv "CC" "clang") (setenv "CXX" "clang++")
--8<---------------cut here---------------end--------------->8---

Hey, is this issue with GCC reported upstream? I'm not opposed to
compiling against clang<++>, especially since it is already in the
closure for the package. But if there is a reported issue for this, the
ticket number should be referenced in the comment ideally. Otherwise,
could you report it?

Thanks!

Brett Gilio




Information forwarded to guix-patches <at> gnu.org:
bug#42807; Package guix-patches. (Tue, 11 Aug 2020 01:14:02 GMT) Full text and rfc822 format available.

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

From: Vinicius Monego <monego <at> posteo.net>
To: Brett Gilio <brettg <at> gnu.org>
Cc: 42807 <at> debbugs.gnu.org
Subject: Re: [bug#42807] [PATCH 1/2] gnu: darktable: Update to 3.2.1.
Date: Mon, 10 Aug 2020 22:12:52 -0300
Brett Gilio writes:

> Hey, is this issue with GCC reported upstream? I'm not opposed 
> to
> compiling against clang<++>, especially since it is already in 
> the
> closure for the package. But if there is a reported issue for 
> this, the
> ticket number should be referenced in the comment 
> ideally. Otherwise,
> could you report it?

I debugged the build with -K and the problem seems to be this tiny 
detail:

`ld: cannot find -lpthreads`

From a quick read online, -lpthreads is Clang syntax. In GCC it's 
only
-lpthread or -pthread. That flag is not changed for GCC. Some 
other flags are
also using -s termination which GCC doesn't recognize.

I'm not sure who's at fault here. I may be missing something in 
the
build steps, but I don't have enough time to investigate this.

The ticket/issue that more closely matches this problem is #5185 
and GCC 8+ is
officially supported by Darktable.




Information forwarded to guix-patches <at> gnu.org:
bug#42807; Package guix-patches. (Wed, 12 Aug 2020 01:03:02 GMT) Full text and rfc822 format available.

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

From: Vinicius Monego <monego <at> posteo.net>
To: Brett Gilio <brettg <at> gnu.org>
Cc: 42807 <at> debbugs.gnu.org
Subject: Re: [bug#42807] [PATCH 1/2] gnu: darktable: Update to 3.2.1.
Date: Tue, 11 Aug 2020 22:01:51 -0300
Vinicius Monego writes:

> I debugged the build with -K and the problem seems to be this 
> tiny
> detail:
>
> `ld: cannot find -lpthreads`

To correct myself, that's only normal CMake check, nothing wrong
there. Pthread is passed correctly to make.

The actual failure in my build log is something to do with OpenMP 
(libgomp):

VC5Decompressor.cpp:744: undefined reference to 
`GOMP_loop_nonmonotonic_dynamic_next'
collect2: error: ld returned 1 exit status

Could this be merged as is for now? I may send a v2 with better
comments and leave GCC for another patch. I'd make sure that the 
issue
is with upstream before submitting an issue there.




Information forwarded to guix-patches <at> gnu.org:
bug#42807; Package guix-patches. (Thu, 13 Aug 2020 15:01:01 GMT) Full text and rfc822 format available.

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

From: Vinicius Monego <monego <at> posteo.net>
To: 42807 <at> debbugs.gnu.org
Cc: Vinicius Monego <monego <at> posteo.net>
Subject: [PATCH v2 1/2] gnu: darktable: Update to 3.2.1.
Date: Thu, 13 Aug 2020 11:59:15 -0300
* gnu/packages/photo.scm (darktable): Update to 3.2.1.
---
Edited description for GCC error and added more comments about optional dependencies
 gnu/packages/photo.scm | 30 ++++++++++++++++--------------
 1 file changed, 16 insertions(+), 14 deletions(-)

diff --git a/gnu/packages/photo.scm b/gnu/packages/photo.scm
index abdb751a5a..688de5c460 100644
--- a/gnu/packages/photo.scm
+++ b/gnu/packages/photo.scm
@@ -465,7 +465,7 @@ photographic equipment.")
 (define-public darktable
   (package
     (name "darktable")
-    (version "3.0.2")
+    (version "3.2.1")
     (source
      (origin
        (method url-fetch)
@@ -473,7 +473,7 @@ photographic equipment.")
              "https://github.com/darktable-org/darktable/releases/"
              "download/release-" version "/darktable-" version ".tar.xz"))
        (sha256
-        (base32 "1yrnkw8c47kmy2x6m1xp69hwyk02xyc8pd9kvcmyj54lzrhzdfka"))))
+        (base32 "035rvqmw386hm0jpi14lf4dnpr5rjkalzjkyprqh42nwi3m86dkf"))))
     (build-system cmake-build-system)
     (arguments
      `(#:tests? #f                      ; there are no tests
@@ -482,6 +482,8 @@ photographic equipment.")
        (modify-phases %standard-phases
          (add-before 'configure 'prepare-build-environment
            (lambda* (#:key inputs #:allow-other-keys)
+             ;; Rawspeed fails to build with GCC due to OpenMP error:
+             ;; "undefined reference to `GOMP_loop_nonmonotonic_dynamic_next'"
              (setenv "CC" "clang") (setenv "CXX" "clang++")
              ;; Darktable looks for opencl-c.h in the LLVM dir. Guix installs
              ;; it to the Clang dir. We fix this by patching CMakeLists.txt.
@@ -523,36 +525,36 @@ photographic equipment.")
        ("po4a" ,po4a)))
     (inputs
      `(("cairo" ,cairo)
-       ("colord-gtk" ,colord-gtk)
-       ("cups" ,cups)
+       ("colord-gtk" ,colord-gtk) ;optional, for color profile support
+       ("cups" ,cups) ;optional, for printing support
        ("curl" ,curl)
        ("dbus-glib" ,dbus-glib)
        ("exiv2" ,exiv2)
        ("freeimage" ,freeimage)
-       ("gmic" ,gmic)
+       ("gmic" ,gmic) ;optional, for HaldcLUT support
        ("graphicsmagick" ,graphicsmagick)
        ("gsettings-desktop-schemas" ,gsettings-desktop-schemas)
        ("gtk+" ,gtk+)
        ("ilmbase" ,ilmbase)
-       ("iso-codes" ,iso-codes)
+       ("iso-codes" ,iso-codes) ;optional, for language names in the preferences
        ("json-glib" ,json-glib)
        ("lcms" ,lcms)
-       ("lensfun" ,lensfun)
-       ("libgphoto2" ,libgphoto2)
+       ("lensfun" ,lensfun) ;optional, for the lens distortion plugin
+       ("libgphoto2" ,libgphoto2) ;optional, for camera tethering
        ("libjpeg" ,libjpeg-turbo)
        ("libomp" ,libomp)
        ("libpng" ,libpng)
        ("librsvg" ,librsvg)
-       ("libsecret" ,libsecret)
+       ("libsecret" ,libsecret) ;optional, for storing passwords
        ("libsoup" ,libsoup)
        ("libtiff" ,libtiff)
-       ("libwebp" ,libwebp)
+       ("libwebp" ,libwebp) ;optional, for WebP support
        ("libxml2" ,libxml2)
        ("libxslt" ,libxslt)
-       ("lua" ,lua) ;for plugins
-       ("openexr" ,openexr)
-       ("openjpeg" ,openjpeg)
-       ("osm-gps-map" ,osm-gps-map)
+       ("lua" ,lua) ;optional, for plugins
+       ("openexr" ,openexr) ;optional, for EXR import/export
+       ("openjpeg" ,openjpeg) ;optional, for JPEG2000 export
+       ("osm-gps-map" ,osm-gps-map) ;optional, for geotagging view
        ("pugixml" ,pugixml)
        ("python-jsonschema" ,python-jsonschema)
        ("sqlite" ,sqlite)))
-- 
2.20.1





Information forwarded to guix-patches <at> gnu.org:
bug#42807; Package guix-patches. (Thu, 13 Aug 2020 15:01:02 GMT) Full text and rfc822 format available.

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

From: Vinicius Monego <monego <at> posteo.net>
To: 42807 <at> debbugs.gnu.org
Cc: Vinicius Monego <monego <at> posteo.net>
Subject: [PATCH v2 2/2] gnu: darktable: Add dependency on libavif, ocl-icd.
Date: Thu, 13 Aug 2020 11:59:16 -0300
* gnu/packages/photo.scm (darktable)[inputs]: Add libavif, ocl-icd.
[arguments]: Set LD_LIBRARY_PATH for OpenCL.
[supported-systems]: Remove i686 which is not supported anymore.
---
 gnu/packages/photo.scm | 26 ++++++++++++++++----------
 1 file changed, 16 insertions(+), 10 deletions(-)

diff --git a/gnu/packages/photo.scm b/gnu/packages/photo.scm
index 688de5c460..2684ac294d 100644
--- a/gnu/packages/photo.scm
+++ b/gnu/packages/photo.scm
@@ -503,15 +503,19 @@ photographic equipment.")
                      (string-append (assoc-ref inputs "ilmbase")
                                     "/include/OpenEXR:" (or (getenv "CPATH") "")))
              #t))
-          (add-after 'install 'wrap-program
-            (lambda* (#:key inputs outputs #:allow-other-keys)
-              (wrap-program (string-append (assoc-ref outputs "out")
-                                           "/bin/darktable")
-                ;; For GtkFileChooserDialog.
-                `("GSETTINGS_SCHEMA_DIR" =
-                  (,(string-append (assoc-ref inputs "gtk+")
-                                   "/share/glib-2.0/schemas"))))
-              #t)))))
+         (add-after 'install 'wrap-program
+           (lambda* (#:key inputs outputs #:allow-other-keys)
+             (wrap-program (string-append (assoc-ref outputs "out")
+                                          "/bin/darktable")
+               ;; For GtkFileChooserDialog.
+               `("GSETTINGS_SCHEMA_DIR" =
+                 (,(string-append (assoc-ref inputs "gtk+")
+                                  "/share/glib-2.0/schemas")))
+               ;; For libOpenCL.so.
+               `("LD_LIBRARY_PATH" =
+                 (,(string-append (assoc-ref inputs "ocl-icd")
+                                  "/lib"))))
+             #t)))))
     (native-inputs
      `(("clang" ,clang-9)
        ("desktop-file-utils" ,desktop-file-utils)
@@ -541,6 +545,7 @@ photographic equipment.")
        ("lcms" ,lcms)
        ("lensfun" ,lensfun) ;optional, for the lens distortion plugin
        ("libgphoto2" ,libgphoto2) ;optional, for camera tethering
+       ("libavif" ,libavif) ;optional, for AVIF support
        ("libjpeg" ,libjpeg-turbo)
        ("libomp" ,libomp)
        ("libpng" ,libpng)
@@ -552,6 +557,7 @@ photographic equipment.")
        ("libxml2" ,libxml2)
        ("libxslt" ,libxslt)
        ("lua" ,lua) ;optional, for plugins
+       ("ocl-icd" ,ocl-icd) ;optional, for OpenCL support
        ("openexr" ,openexr) ;optional, for EXR import/export
        ("openjpeg" ,openjpeg) ;optional, for JPEG2000 export
        ("osm-gps-map" ,osm-gps-map) ;optional, for geotagging view
@@ -565,7 +571,7 @@ developer.  It manages your digital negatives in a database, lets you view
 them through a zoomable lighttable and enables you to develop raw images
 and enhance them.")
     ;; See src/is_supported_platform.h for supported platforms.
-    (supported-systems '("i686-linux" "x86_64-linux" "aarch64-linux"))
+    (supported-systems '("x86_64-linux" "aarch64-linux"))
     (license (list license:gpl3+ ;; Darktable itself.
                    license:lgpl2.1+)))) ;; Rawspeed library.
 
-- 
2.20.1





Information forwarded to guix-patches <at> gnu.org:
bug#42807; Package guix-patches. (Fri, 04 Sep 2020 09:07:02 GMT) Full text and rfc822 format available.

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

From: Ludovic Courtès <ludo <at> gnu.org>
To: Vinicius Monego <monego <at> posteo.net>
Cc: 42807 <at> debbugs.gnu.org, Brett Gilio <brettg <at> gnu.org>
Subject: Re: [bug#42807] [PATCH 1/2] gnu: darktable: Update to 3.2.1.
Date: Fri, 04 Sep 2020 11:06:36 +0200
Hi,

Vinicius Monego <monego <at> posteo.net> skribis:

> The actual failure in my build log is something to do with OpenMP
> (libgomp):
>
> VC5Decompressor.cpp:744: undefined reference to
> `GOMP_loop_nonmonotonic_dynamic_next'
> collect2: error: ld returned 1 exit status

It looks like a case where ‘-fopenmp’ is used as compile time but
omitted at link time.  The fix is to add it at link time (to the LDFLAGS
of the executable at hand).

It would be nice to see eventually if this can be fixed so we can switch
back to GCC.

Thanks,
Ludo’.




Reply sent to Ludovic Courtès <ludo <at> gnu.org>:
You have taken responsibility. (Fri, 04 Sep 2020 09:07:02 GMT) Full text and rfc822 format available.

Notification sent to Vinicius Monego <monego <at> posteo.net>:
bug acknowledged by developer. (Fri, 04 Sep 2020 09:07:02 GMT) Full text and rfc822 format available.

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

From: Ludovic Courtès <ludo <at> gnu.org>
To: Vinicius Monego <monego <at> posteo.net>
Cc: 42807-done <at> debbugs.gnu.org
Subject: Re: [bug#42807] [PATCH v2 1/2] gnu: darktable: Update to 3.2.1.
Date: Fri, 04 Sep 2020 11:06:14 +0200
Hi,

Vinicius Monego <monego <at> posteo.net> skribis:

> * gnu/packages/photo.scm (darktable): Update to 3.2.1.

[...]

> * gnu/packages/photo.scm (darktable)[inputs]: Add libavif, ocl-icd.
> [arguments]: Set LD_LIBRARY_PATH for OpenCL.
> [supported-systems]: Remove i686 which is not supported anymore.

Thanks, applied!

> +             (wrap-program (string-append (assoc-ref outputs "out")
> +                                          "/bin/darktable")
> +               ;; For GtkFileChooserDialog.
> +               `("GSETTINGS_SCHEMA_DIR" =
> +                 (,(string-append (assoc-ref inputs "gtk+")
> +                                  "/share/glib-2.0/schemas")))
> +               ;; For libOpenCL.so.
> +               `("LD_LIBRARY_PATH" =
> +                 (,(string-append (assoc-ref inputs "ocl-icd")
> +                                  "/lib"))))

It’s weird that LD_LIBRARY_PATH needs to be specified here.  Isn’t the
binary already linked against libOpenCL?  If so, it should have it in
as NEEDed and its its RUNPATH, no?

Ludo’.




Information forwarded to guix-patches <at> gnu.org:
bug#42807; Package guix-patches. (Sat, 05 Sep 2020 18:19:01 GMT) Full text and rfc822 format available.

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

From: Vinicius Monego <monego <at> posteo.net>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: 42807-done <at> debbugs.gnu.org
Subject: Re: [bug#42807] [PATCH v2 1/2] gnu: darktable: Update to 3.2.1.
Date: Sat, 05 Sep 2020 15:17:27 -0300
Ludovic Courtès writes:

> Isn’t the binary already linked against libOpenCL?

It's not linked. libOpenCL is dynamically loaded. See
src/common/dlopencl.c.




bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Sun, 04 Oct 2020 11:24:04 GMT) Full text and rfc822 format available.

This bug report was last modified 3 years and 177 days ago.

Previous Next


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