GNU bug report logs - #27249
[PATCH] gnu: gnome: Add deja-dup.

Previous Next

Full log

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

From: Christopher Baines <mail <at> cbaines.net>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: 27249 <at> debbugs.gnu.org
Subject: Re: bug#27249: [PATCH] gnu: gnome: Add deja-dup.
Date: Thu, 8 Jun 2017 21:50:25 +0100

[Message part 1 (text/plain, inline)]

On 07/06/17 14:01, Ludovic Courtès wrote:
> Christopher Baines <mail <at> cbaines.net> skribis:
> 
>> * gnu/packages/gnome.scm (deja-dup): New variable.
> 
> There’s a couple of lint warnings fixed by the patch below:
> 
> 
> 
> diff --git a/gnu/packages/gnome.scm b/gnu/packages/gnome.scm
> index ab0c57d31..cd78e6015 100644
> --- a/gnu/packages/gnome.scm
> +++ b/gnu/packages/gnome.scm
> @@ -252,19 +252,19 @@ features to enable users to create their discs easily and quickly.")
>         ("libnotify" ,libnotify)
>         ("libpeas" ,libpeas)
>         ("libsecret" ,libsecret)
> -       ("itstool" ,itstool)
>         ("packagekit" ,packagekit)))
>      (native-inputs
>       `(("pkg-config" ,pkg-config)
>         ("vala" ,vala)
>         ("gettext" ,gettext-minimal)
> +       ("itstool" ,itstool)
>         ("intltool" ,intltool)
>         ("cmake", cmake)))
>      (home-page "https://launchpad.net/deja-dup")
>      (synopsis "Simple backup tool, for regular encrypted backups")
>      (description
> -     "Déjà Dup is a simple backup tool, for regular encrypted backups. It uses
> -duplicity as the backend, which supports incremental backups and storage
> +     "Déjà Dup is a simple backup tool, for regular encrypted backups.  It
> +uses duplicity as the backend, which supports incremental backups and storage
>  either on a local, or remote machine via a number of methods.")
>      (license license:gpl3+)))
>  
> 
> 
> 
> 
> Regarding propagated inputs:
> 
>> +    (propagated-inputs
>> +     `(("gsettings-desktop-schemas" ,gsettings-desktop-schemas)
>> +       ("gobject-introspection" ,gobject-introspection)
>> +       ("duplicity" ,duplicity)
>> +       ("python" ,python2-minimal)
>> +       ("python-pygobject" ,python2-pygobject)))
> 
> Could you add a comment explaining why all this needs to be propagated?
> If there’s a way to avoid it, it’s even better (for instance, if
> deja-dup execs duplicity, we could hard-code the absolute file name of
> duplicity.)

Thanks for your review Ludo, hopefully I can get around to looking at
deja-dup again at the weekend :)

[signature.asc (application/pgp-signature, attachment)]

Previous Next

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