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.

> +       ("fop" ,fop)

And I can't test it further, because this package is missing.

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.