guix-devel
[Top][All Lists]
Advanced

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

Re: [PATCH] gnu: Add xinetd.


From: Tobias Geerinckx-Rice
Subject: Re: [PATCH] gnu: Add xinetd.
Date: Thu, 26 Jan 2017 14:58:53 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.5.1

Thomas,

On 26/01/17 11:43, Thomas Danckaert wrote:
> This patch provides the xinetd executables[.]

Thanks!

> + (origin
> +   (method url-fetch)
> +   (uri "https://github.com/xinetd-org/xinetd/archive/\
>           \xinetd-2-3-15.tar.gz")
> +   (file-name (string-append name "-" version))

file-name is used mainly to avoid polluting the store with meaningless
(tobias.gr/download.php?foo=123) or misleading file names.

It shouldn't be needed here, and would require its own ".tar.gz" suffix
otherwise.

> + #:phases
> + (modify-phases %standard-phases (delete 'check)))) ; doesn't\
> \provide tests

Would the simpler (and more idiomatic)

  #:tests? #f ... ; no tests

work?

> + (license (l:non-copyleft "file://COPYRIGHT"))))

Interesting licence. :-) Against my intuition, it appears to be
FSDG-compatible[0].

Kind regards,

T G-R

[0]:
http://metadata.ftp-master.debian.org/changelogs/main/x/xinetd/xinetd_2.3.15-7_copyright

Attachment: signature.asc
Description: OpenPGP digital signature


reply via email to

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