bug#25918: [PATCH 1/3] gnu: Add libmesode.

[Catonano]

2017-03-02 10:13 GMT+01:00 Ricardo Wurmus <address@hidden>:

Hi Mekeor,

thanks for the patch!

This is almost ready to push, and I could make the needed changes
myself, but maybe you’d like to handle them yourself.

Oh gosh

this had been reviewed already :-/
 

>>From aff05409f65025c007d7a50f5d4e90b301ce3b18 Mon Sep 17 00:00:00 2001
> From: Mekeor Melire <address@hidden>
> Date: Wed, 1 Mar 2017 20:08:04 +0100
> Subject: [PATCH 1/3] gnu: Add libmesode.
>
> * gnu/packages/messaging.scm (libmesode): New variable.
> ---

[…]

> +(define-public libmesode
> +  (package
> +    (name "libmesode")
> +    (version "0.9.1")
> +    (source (origin
> +              (method url-fetch)
> +              (uri (string-append "https://github.com/boothj5/libmesode/archive/"
> +                                  version ".tar.gz")

Please add a “file-name” _expression_ here to give the archive a proper
name in the store.  We do this for all tarballs that don’t include the
name of the package.

> +              (sha256
> +               (base32
> +                "0iaj56fkd5bjvqpvq3324ni895rmbj1akbfqipjydnghfwaym4z6"))))
> +    (build-system gnu-build-system)
> +    (arguments
> +     `(#:phases
> +       (modify-phases %standard-phases
> +         (add-before 'configure 'bootstrap
> +           (lambda _
> +             (zero? (system* "./bootstrap.sh")))))))
> +    (inputs
> +     `(("expat" ,expat)
> +       ("openssl" ,openssl)))
> +    (native-inputs
> +     `(("autoconf" ,autoconf)
> +       ("automake" ,automake)
> +       ("libtool" ,libtool)
> +       ("pkg-config" ,pkg-config)))
> +    (synopsis "A simple, lightweight C library for writing XMPP
> clients")

Please remove the “A” from the beginning.  (I think “guix lint” also
complains about it.)