On Fri, 08 Sep 2017 18:07:01 +0200
ludo@gnu.org (Ludovic Courtès) wrote:

> Christopher Baines <mail@cbaines.net> skribis:
> 
> > Add several missing dependencies, particularly for the caff tool. It
> > now runs, but I've been unable to get it working fully yet, possibly
> > due to problems with the actual sending of the emails.
> >
> > Christopher Baines (12):
> >   gnu: Add perl-moox.
> >   gnu: Add perl-data-perl.
> >   gnu: Add perl-moox-handlesvia.
> >   gnu: Add perl-moox-late.
> >   gnu: Add perl-convert-binhex.
> >   gnu: Add perl-mailtools.
> >   gnu: Add perl-mime-tools.
> >   gnu: Add perl-net-idn-encode.
> >   gnu: Add perl-text-template.
> >   gnu: Add perl-gnupg-interface.
> >   gnu: Add libmd.
> >   gnu: Update signing-party.  
> 
> At first sight this looks good to me.

Great, thanks for reviewing :)

> Please use ‘license:perl-license’ instead of (package-license perl).

Within (gnu packages perl), it doesn't use a prefix for the licenses
module. I could switch it to use the license: prefix, or use just
perl-license within that module. Do you have a preference?