From: Eric Bavier <ericbavier@openmailbox.org>
To: Tobias Geerinckx-Rice <me@tobias.gr>
Cc: guix-devel@gnu.org
Subject: Re: [PATCH] gnu: Add lrzip.
Date: Thu, 21 Jul 2016 00:24:36 -0500 [thread overview]
Message-ID: <20160721002436.13651c1f@openmailbox.org> (raw)
In-Reply-To: <20160720225413.2903-1-me@tobias.gr>
Hello Tobias,
Thanks for the patch. A few comments:
On Thu, 21 Jul 2016 00:54:13 +0200
Tobias Geerinckx-Rice <me@tobias.gr> wrote:
> * gnu/packages/compression.scm (lrzip): New variable.
> ---
> gnu/packages/compression.scm | 32 ++++++++++++++++++++++++++++++++
> 1 file changed, 32 insertions(+)
>
> diff --git a/gnu/packages/compression.scm b/gnu/packages/compression.scm
> index 6a17003..26467a4 100644
> --- a/gnu/packages/compression.scm
> +++ b/gnu/packages/compression.scm
> @@ -36,6 +36,7 @@
> #:use-module (guix build-system gnu)
> #:use-module (guix build-system perl)
> #:use-module (gnu packages)
> + #:use-module (gnu packages assembly)
> #:use-module (gnu packages autotools)
> #:use-module (gnu packages backup)
> #:use-module (gnu packages base)
> @@ -833,3 +834,34 @@ also be used to apply such patches. xdelta is similar to @command{diff} and
> @command{patch}, but is not limited to plain text and does not generate
> human-readable output.")
> (license license:asl2.0)))
> +
> +(define-public lrzip
> + (package
> + (name "lrzip")
> + (version "0.630")
> + (source
> + (origin
> + (method url-fetch)
> + (uri (string-append
> + "http://ck.kolivas.org/apps/lrzip/lrzip-" version ".tar.bz2"))
> + (sha256
> + (base32
> + "01ykxliqw4cavx9f2gawxfa9wf52cjy1qx28cnkrh6i3lfzzcq94"))))
> + (build-system gnu-build-system)
> + (native-inputs
> + `(("nasm" ,nasm)
ASM support seems to be supported only on x86/32-bit systems. Could
you make this input conditional on the system type? An example would
be the "lame" package in gnu/packages/mp3.scm.
> + ("perl" ,perl)))
> + (inputs
> + `(("bzip2" ,bzip2)
Configure checks for libbz2 and the build instruction mention it as a
requirement, but there are no references retained in the output and I
don't see any references to libbz2 in the Makefile's. Any ideas?
> + ("lzo" ,lzo)
> + ("zlib" ,zlib)))
> + (home-page "http://ck.kolivas.org/apps/lrzip/")
> + (synopsis "Large file compressor with a very high compression ratio")
> + (description "lrzip is a compression utility that uses long-range
> +redundancy reduction to improve the subsequent compression ratio of
> +larger files. It can then further compress the result with the ZPAQ or
> +LZMA algorithms for maximum compression, or LZO for maximum speed. This
> +choice between size or speed allows for either better compression than
> +even LZMA can provide, or a higher speed than gzip while compressing as
> +well as bzip2.")
> + (license license:gpl2+)))
The license should probably be "(list license:gpl3+
license:public-domain)" since there is at least one file, md5.c,
that is borrowed from glibc and is under a gpl3+ license; and several
of the files in the "lzma" directory have been released into the public
domain.
Could you send an updated patch?
`~Eric
next prev parent reply other threads:[~2016-07-21 5:24 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-07-20 22:54 [PATCH] gnu: Add lrzip Tobias Geerinckx-Rice
2016-07-21 5:24 ` Eric Bavier [this message]
2016-07-21 17:46 ` Tobias Geerinckx-Rice
2016-07-21 20:21 ` Tobias Geerinckx-Rice
2016-07-21 20:24 ` [PATCH v2] " Tobias Geerinckx-Rice
2016-07-21 20:31 ` Tobias Geerinckx-Rice
2016-07-22 13:13 ` Ludovic Courtès
2016-07-23 10:45 ` Ludovic Courtès
2016-07-23 19:05 ` Tobias Geerinckx-Rice
2016-07-24 21:36 ` Ludovic Courtès
2016-07-24 21:56 ` Leo Famulari
2016-07-25 8:20 ` Ludovic Courtès
2016-07-24 22:34 ` Tobias Geerinckx-Rice
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20160721002436.13651c1f@openmailbox.org \
--to=ericbavier@openmailbox.org \
--cc=guix-devel@gnu.org \
--cc=me@tobias.gr \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/guix.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.