GNU bug report logs - #25928
[PATCH] gnu: Add darktable.

Previous Next

Full log

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

From: Roel Janssen <roel <at> gnu.org>
To: Leo Famulari <leo <at> famulari.name>
Cc: 25928 <at> debbugs.gnu.org
Subject: Re: bug#25928: [PATCH] gnu: Add darktable.
Date: Wed, 08 Mar 2017 20:01:07 +0100

Leo Famulari writes:

> On Thu, Mar 02, 2017 at 01:04:33AM +0100, Roel Janssen wrote:
>> Subject: [PATCH] gnu: Add darktable.
>> 
>> * gnu/packages/photo.scm (darktable): New variable.
>
> Cool!
>
>> ---
>>  gnu/packages/photo.scm | 81 ++++++++++++++++++++++++++++++++++++++++++++++++++
>>  1 file changed, 81 insertions(+)
>> 
>> diff --git a/gnu/packages/photo.scm b/gnu/packages/photo.scm
>> index ce71d2a44..42da06080 100644
>> --- a/gnu/packages/photo.scm
>> +++ b/gnu/packages/photo.scm
>> @@ -31,17 +31,23 @@
>>    #:use-module (gnu packages base)
>>    #:use-module (gnu packages boost)
>>    #:use-module (gnu packages compression)
>> +  #:use-module (gnu packages curl)
>> +  #:use-module (gnu packages databases)
>>    #:use-module (gnu packages ghostscript)
>>    #:use-module (gnu packages gl)
>
> It's missing the (gnu packages glib) module, for dbus-glib.

Yes, the glib module is in another patch I which is a prerequisite for
Darktable.

>> +       ("fop" ,fop)
>
> And I can't test it further, because this package is missing.

I sent a separate patch for fop.

> Also, the patch as sent doesn't apply correctly with `git am`. I
> recommend attaching the patch if you are using the debbugs guix-patches.

Right.  I messed up the patch creation I guess.  I'll do better in the
future.

Kind regards,
Roel Janssen

Previous Next

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