[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH 1/2] gnu: Add mbedtls.

From: Leo Famulari
Subject: Re: [PATCH 1/2] gnu: Add mbedtls.
Date: Wed, 25 Jan 2017 22:11:05 -0500
User-agent: Mutt/1.7.2 (2016-11-26)

On Wed, Jan 25, 2017 at 12:25:17PM +0000, address@hidden wrote:
> * gnu/packages/tls.scm (mbedtls): New variable.

> +       ;; FIXME: Tarball on can no be fetched by our downloader.

Please add a comment explaining what goes wrong.

> +    (native-inputs
> +     `(("perl" ,perl) ; Tests
> +       ("doxygen" ,doxygen)
> +       ("graphviz" ,graphviz)))
> +    (inputs
> +     `(("zlib" ,zlib)
> +       ("openssl" ,openssl)))

The only dependency it needs to build is Perl.

The rest can be missing and mbedtls will still pass its test suite.

Doxygen seems unused; no documentation is installed when its available.

If they are available, zlib and openssl are not referred to by the output
(checked with `guix gc --references`).

reply via email to

[Prev in Thread] Current Thread [Next in Thread]