GNU bug report logs - #41953
[PATCH] gnu: darktable: Update to 3.0.2.

Previous Next

Package: guix-patches;

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

Date: Fri, 19 Jun 2020 20:14:01 UTC

Severity: normal

Tags: patch

Done: Marius Bakke <marius <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 41953 in the body.
You can then email your comments to 41953 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#41953; Package guix-patches. (Fri, 19 Jun 2020 20:14:01 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. (Fri, 19 Jun 2020 20:14:01 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] gnu: darktable: Update to 3.0.2.
Date: Fri, 19 Jun 2020 17:11:20 -0300
* gnu/packages/photo.scm (darktable): Update to 3.0.2.
[arguments]: Compile with clang.
[arguments]: Wrap GSETTINGS_SCHEMAS_DIR.
[license]: Add license information for Rawspeed.
[native-inputs]: Add clang.
[native-inputs]: Add desktop-file-utils.
[native-inputs]: Add glib.
[native-inputs]: Add gobject-instrospection.
[native-inputs]: Add llvm.
[native-inputs]: Add po4a.
[native-inputs]: Reorder alphabetically.
[inputs]: Add colord-gtk.
[inputs]: Add cups.
[inputs]: Add graphicsmagick.
[inputs]: Add gsettings-desktop-schemas.
[inputs]: Add iso-codes.
[inputs]: Add libsecret.
[inputs]: Add lua.
[inputs]: Add openjpeg.
[inputs]: Add osm-gps-map.
[inputs]: Reorder alphabetically.
---
 gnu/packages/photo.scm | 80 ++++++++++++++++++++++++++++++------------
 1 file changed, 57 insertions(+), 23 deletions(-)

diff --git a/gnu/packages/photo.scm b/gnu/packages/photo.scm
index 4904fa1db1..890035f0c7 100644
--- a/gnu/packages/photo.scm
+++ b/gnu/packages/photo.scm
@@ -8,6 +8,7 @@
 ;;; Copyright © 2018 Leo Famulari <leo <at> famulari.name>
 ;;; Copyright © 2020 Sebastian Schott <sschott <at> mailbox.org>
 ;;; Copyright © 2020 Vincent Legoll <vincent.legoll <at> gmail.com>
+;;; Copyright © 2020 Vinicius Monego <monego <at> posteo.net>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -39,9 +40,11 @@
   #:use-module (gnu packages base)
   #:use-module (gnu packages boost)
   #:use-module (gnu packages compression)
+  #:use-module (gnu packages cups)
   #:use-module (gnu packages curl)
   #:use-module (gnu packages file)
   #:use-module (gnu packages freedesktop)
+  #:use-module (gnu packages geo)
   #:use-module (gnu packages gettext)
   #:use-module (gnu packages ghostscript)
   #:use-module (gnu packages gl)
@@ -52,9 +55,11 @@
   #:use-module (gnu packages gtk)
   #:use-module (gnu packages image)
   #:use-module (gnu packages imagemagick)
+  #:use-module (gnu packages iso-codes)
   #:use-module (gnu packages libcanberra)
   #:use-module (gnu packages libusb)
   #:use-module (gnu packages llvm)
+  #:use-module (gnu packages lua)
   #:use-module (gnu packages man)
   #:use-module (gnu packages maths)
   #:use-module (gnu packages perl)
@@ -457,7 +462,7 @@ photographic equipment.")
 (define-public darktable
   (package
     (name "darktable")
-    (version "2.6.3")
+    (version "3.0.2")
     (source
      (origin
        (method url-fetch)
@@ -465,13 +470,16 @@ photographic equipment.")
              "https://github.com/darktable-org/darktable/releases/"
              "download/release-" version "/darktable-" version ".tar.xz"))
        (sha256
-        (base32 "1w3q3dhcxa0bs590zbsj61ap8z84wmn04xs5q3gjwisqhjf9j655"))))
+        (base32 "1yrnkw8c47kmy2x6m1xp69hwyk02xyc8pd9kvcmyj54lzrhzdfka"))))
     (build-system cmake-build-system)
     (arguments
      `(#:tests? #f                      ; there are no tests
        #:configure-flags '("-DBINARY_PACKAGE_BUILD=On")
        #:phases
        (modify-phases %standard-phases
+         (add-before 'configure 'prepare-build-environment
+           (lambda* (#:key inputs #:allow-other-keys)
+             (setenv "CC" "clang") (setenv "CXX" "clang++")))
          (add-before 'configure 'set-LDFLAGS-and-CPATH
            (lambda* (#:key inputs outputs #:allow-other-keys)
              (setenv "LDFLAGS"
@@ -483,35 +491,60 @@ photographic equipment.")
              (setenv "CPATH"
                      (string-append (assoc-ref inputs "ilmbase")
                                     "/include/OpenEXR:" (or (getenv "CPATH") "")))
-             #t)))))
+             #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)))))
     (native-inputs
-     `(("intltool" ,intltool)
+     `(("clang" ,clang-9)
+       ("desktop-file-utils" ,desktop-file-utils)
+       ("glib:bin" ,glib "bin")
+       ("gobject-introspection" ,gobject-introspection)
+       ("intltool" ,intltool)
+       ("llvm" ,llvm)
        ("perl" ,perl)
-       ("pkg-config" ,pkg-config)))
+       ("pkg-config" ,pkg-config)
+       ("po4a" ,po4a)))
     (inputs
-     `(("libxslt" ,libxslt)
-       ("libxml2" ,libxml2)
-       ("pugixml" ,pugixml)
+     `(("cairo" ,cairo)
+       ("colord-gtk" ,colord-gtk)
+       ("cups" ,cups)
+       ("curl" ,curl)
+       ("dbus-glib" ,dbus-glib)
+       ("exiv2" ,exiv2)
+       ("freeimage" ,freeimage)
+       ("graphicsmagick" ,graphicsmagick)
+       ("gsettings-desktop-schemas" ,gsettings-desktop-schemas)
        ("gtk+" ,gtk+)
-       ("sqlite" ,sqlite)
+       ("ilmbase" ,ilmbase)
+       ("iso-codes" ,iso-codes)
+       ("json-glib" ,json-glib)
+       ("lcms" ,lcms)
+       ("lensfun" ,lensfun)
+       ("libgphoto2" ,libgphoto2)
        ("libjpeg" ,libjpeg-turbo)
+       ("libomp" ,libomp)
        ("libpng" ,libpng)
-       ("cairo" ,cairo)
-       ("lcms" ,lcms)
-       ("exiv2" ,exiv2)
+       ("librsvg" ,librsvg)
+       ("libsecret" ,libsecret)
+       ("libsoup" ,libsoup)
        ("libtiff" ,libtiff)
-       ("curl" ,curl)
-       ("libgphoto2" ,libgphoto2)
-       ("dbus-glib" ,dbus-glib)
+       ("libwebp" ,libwebp)
+       ("libxml2" ,libxml2)
+       ("libxslt" ,libxslt)
+       ("lua" ,lua)
        ("openexr" ,openexr)
-       ("ilmbase" ,ilmbase)
-       ("libsoup" ,libsoup)
+       ("openjpeg" ,openjpeg)
+       ("osm-gps-map" ,osm-gps-map)
+       ("pugixml" ,pugixml)
        ("python-jsonschema" ,python-jsonschema)
-       ("libwebp" ,libwebp)
-       ("lensfun" ,lensfun)
-       ("librsvg" ,librsvg)
-       ("json-glib" ,json-glib)
-       ("freeimage" ,freeimage)))
+       ("sqlite" ,sqlite)))
     (home-page "https://www.darktable.org")
     (synopsis "Virtual lighttable and darkroom for photographers")
     (description "Darktable is a photography workflow application and RAW
@@ -520,7 +553,8 @@ 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"))
-    (license license:gpl3+)))
+    (license (list license:gpl3+ ;; Darktable itself.
+                   license:lgpl2.1+)))) ;; Rawspeed library.
 
 (define-public hugin
   (package
-- 
2.20.1





Reply sent to Marius Bakke <marius <at> gnu.org>:
You have taken responsibility. (Mon, 22 Jun 2020 21:12:02 GMT) Full text and rfc822 format available.

Notification sent to Vinicius Monego <monego <at> posteo.net>:
bug acknowledged by developer. (Mon, 22 Jun 2020 21:12:02 GMT) Full text and rfc822 format available.

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

From: Marius Bakke <marius <at> gnu.org>
To: Vinicius Monego <monego <at> posteo.net>, 41953-done <at> debbugs.gnu.org
Cc: Vinicius Monego <monego <at> posteo.net>
Subject: Re: [bug#41953] [PATCH] gnu: darktable: Update to 3.0.2.
Date: Mon, 22 Jun 2020 23:11:13 +0200
[Message part 1 (text/plain, inline)]
Vinicius Monego <monego <at> posteo.net> writes:

> * gnu/packages/photo.scm (darktable): Update to 3.0.2.
> [arguments]: Compile with clang.
> [arguments]: Wrap GSETTINGS_SCHEMAS_DIR.
> [license]: Add license information for Rawspeed.
> [native-inputs]: Add clang.
> [native-inputs]: Add desktop-file-utils.
> [native-inputs]: Add glib.
> [native-inputs]: Add gobject-instrospection.
> [native-inputs]: Add llvm.
> [native-inputs]: Add po4a.
> [native-inputs]: Reorder alphabetically.
> [inputs]: Add colord-gtk.
> [inputs]: Add cups.
> [inputs]: Add graphicsmagick.
> [inputs]: Add gsettings-desktop-schemas.
> [inputs]: Add iso-codes.
> [inputs]: Add libsecret.
> [inputs]: Add lua.
> [inputs]: Add openjpeg.
> [inputs]: Add osm-gps-map.
> [inputs]: Reorder alphabetically.

Thanks!  I flattened this commit message so there is just a single entry
for each of the changed fields.

>         (modify-phases %standard-phases
> +         (add-before 'configure 'prepare-build-environment
> +           (lambda* (#:key inputs #:allow-other-keys)
> +             (setenv "CC" "clang") (setenv "CXX" "clang++")))

I tried building with GCC, and indeed it failed.  First because ISL was
not enabled, so I added 'gcc-9' which has it and adjusted
CPLUS_INCLUDE_PATH accordingly, but still ran into problems.

It would be good to move to GCC eventually because Clang is not nearly
as well tested.  :-)

Anyway thanks for the patch, applied!
[signature.asc (application/pgp-signature, inline)]

Information forwarded to guix-patches <at> gnu.org:
bug#41953; Package guix-patches. (Mon, 22 Jun 2020 21:39:01 GMT) Full text and rfc822 format available.

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

From: Vinicius Monego <monego <at> posteo.net>
To: Marius Bakke <marius <at> gnu.org>, 41953-done <at> debbugs.gnu.org
Subject: Re: [bug#41953] [PATCH] gnu: darktable: Update to 3.0.2.
Date: Mon, 22 Jun 2020 18:37:58 -0300
Em seg, 2020-06-22 às 23:11 +0200, Marius Bakke escreveu:
> Vinicius Monego <monego <at> posteo.net> writes:
> 
> > * gnu/packages/photo.scm (darktable): Update to 3.0.2.
> > [arguments]: Compile with clang.
> > [arguments]: Wrap GSETTINGS_SCHEMAS_DIR.
> > [license]: Add license information for Rawspeed.
> > [native-inputs]: Add clang.
> > [native-inputs]: Add desktop-file-utils.
> > [native-inputs]: Add glib.
> > [native-inputs]: Add gobject-instrospection.
> > [native-inputs]: Add llvm.
> > [native-inputs]: Add po4a.
> > [native-inputs]: Reorder alphabetically.
> > [inputs]: Add colord-gtk.
> > [inputs]: Add cups.
> > [inputs]: Add graphicsmagick.
> > [inputs]: Add gsettings-desktop-schemas.
> > [inputs]: Add iso-codes.
> > [inputs]: Add libsecret.
> > [inputs]: Add lua.
> > [inputs]: Add openjpeg.
> > [inputs]: Add osm-gps-map.
> > [inputs]: Reorder alphabetically.
> 
> Thanks!  I flattened this commit message so there is just a single
> entry
> for each of the changed fields.
> 
> >         (modify-phases %standard-phases
> > +         (add-before 'configure 'prepare-build-environment
> > +           (lambda* (#:key inputs #:allow-other-keys)
> > +             (setenv "CC" "clang") (setenv "CXX" "clang++")))
> 
> I tried building with GCC, and indeed it failed.  First because ISL
> was
> not enabled, so I added 'gcc-9' which has it and adjusted
> CPLUS_INCLUDE_PATH accordingly, but still ran into problems.

A few hours ago I was splitting this patch into shorter commits and the
build began to fail. Weird, because it compiled succesfully for about 7
times before I submitted this patch.

> 
> It would be good to move to GCC eventually because Clang is not
> nearly
> as well tested.  :-)
> 

The reason I chose clang is that CMake was very loud about LLVM not
being found (it's an optional dependency to run OpenCL tests, which is
one of the few features still missing from the package). When LLVM is
found, CMake will recommend clang. That made me guess it was the
recommended compiler.

> Anyway thanks for the patch, applied!

I didn't expect it to be approved at v1. Thanks :-)





Information forwarded to guix-patches <at> gnu.org:
bug#41953; Package guix-patches. (Mon, 22 Jun 2020 21:59:01 GMT) Full text and rfc822 format available.

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

From: Marius Bakke <marius <at> gnu.org>
To: Vinicius Monego <monego <at> posteo.net>, 41953-done <at> debbugs.gnu.org
Subject: Re: [bug#41953] [PATCH] gnu: darktable: Update to 3.0.2.
Date: Mon, 22 Jun 2020 23:58:10 +0200
[Message part 1 (text/plain, inline)]
Vinicius Monego <monego <at> posteo.net> writes:

>> >         (modify-phases %standard-phases
>> > +         (add-before 'configure 'prepare-build-environment
>> > +           (lambda* (#:key inputs #:allow-other-keys)
>> > +             (setenv "CC" "clang") (setenv "CXX" "clang++")))
>> 
>> I tried building with GCC, and indeed it failed.  First because ISL
>> was
>> not enabled, so I added 'gcc-9' which has it and adjusted
>> CPLUS_INCLUDE_PATH accordingly, but still ran into problems.
>
> A few hours ago I was splitting this patch into shorter commits and the
> build began to fail. Weird, because it compiled succesfully for about 7
> times before I submitted this patch.

Oh, I thought all these changes were necessary for the upgrade.  :-)

If you have further improvements to make, don't hesitate to send them.

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

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

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

Previous Next


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