GNU bug report logs - #57034
[PATCH] gnu: packages: Add miniz.

Previous Next

Package: guix-patches;

Reported by: "(" <paren <at> disroot.org>

Date: Sun, 7 Aug 2022 12:03:01 UTC

Severity: normal

Tags: patch

To reply to this bug, email your comments to 57034 AT debbugs.gnu.org.

Toggle the display of automated, internal messages from the tracker.

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to guix-patches <at> gnu.org:
bug#57034; Package guix-patches. (Sun, 07 Aug 2022 12:03:02 GMT) Full text and rfc822 format available.

Acknowledgement sent to "(" <paren <at> disroot.org>:
New bug report received and forwarded. Copy sent to guix-patches <at> gnu.org. (Sun, 07 Aug 2022 12:03:02 GMT) Full text and rfc822 format available.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: "(" <paren <at> disroot.org>
To: guix-patches <at> gnu.org
Cc: "\(" <paren <at> disroot.org>
Subject: [PATCH] gnu: packages: Add miniz.
Date: Sun,  7 Aug 2022 13:02:01 +0100
* gnu/packages/compression.scm (miniz): 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 6070c5beb1..cb1283c9a1 100644
--- a/gnu/packages/compression.scm
+++ b/gnu/packages/compression.scm
@@ -35,6 +35,7 @@
 ;;; Copyright © 2021 Simon Tournier <zimon.toutoune <at> gmail.com>
 ;;; Copyright © 2021 Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
 ;;; Copyright © 2021 Ahmad Jarara <git <at> ajarara.io>
+;;; Copyright © 2022 ( <paren <at> disroot.org>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -2784,3 +2785,34 @@ (define-public fcrackzip
     (synopsis "Zip password cracker")
     (description "Fcrackzip is a Zip file password cracker.")
     (license license:gpl2+)))
+
+(define-public miniz
+  ;; Last release was on 27-06-2021; there have been many commits since
+  ;; then.
+  (let ((commit "be92575500f71dfb5ac7b0c5bb47b6e736c23039")
+        (revision "0"))
+    (package
+     (name "miniz")
+     (version (git-version "2.2.0" revision commit))
+     (source (origin
+              (method git-fetch)
+              (uri (git-reference
+                    (url "https://github.com/richgel999/miniz")
+                    (commit commit)))
+              (file-name (git-file-name name version))
+              (sha256
+               (base32
+                "0w42vagcv4b0hcf0jk7vamrcpmw62dblx9856j98msdfm3qv1xzi"))))
+     (build-system cmake-build-system)
+     (arguments
+      (list #:configure-flags
+            #~(list "-DBUILD_SHARED_LIBS=ON")
+            ;; No test target.
+            #:tests? #f))
+     (home-page "https://github.com/richgel999/miniz")
+     (synopsis "Simple replacement for @code{zlib}")
+     (description
+      "@code{miniz} is a small, simple alternative to the @code{zlib}
+data compression library for manipulating zlib, DEFLATE, PNG, and ZIP
+files from C.")
+     (license license:expat))))
-- 
2.37.1





Information forwarded to guix-patches <at> gnu.org:
bug#57034; Package guix-patches. (Sun, 07 Aug 2022 16:43:02 GMT) Full text and rfc822 format available.

Message #8 received at 57034 <at> debbugs.gnu.org (full text, mbox):

From: Liliana Marie Prikler <liliana.prikler <at> gmail.com>
To: "(" <paren <at> disroot.org>, 57034 <at> debbugs.gnu.org
Subject: Re: [PATCH] gnu: packages: Add miniz.
Date: Sun, 07 Aug 2022 18:42:35 +0200
Hi (,

last time I checked, the template was "gnu: Add PACKAGE", not "gnu:
packages: Add PACKAGE."

Am Sonntag, dem 07.08.2022 um 13:02 +0100 schrieb (:
> * gnu/packages/compression.scm (miniz): 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 6070c5beb1..cb1283c9a1 100644
> --- a/gnu/packages/compression.scm
> +++ b/gnu/packages/compression.scm
> @@ -35,6 +35,7 @@
>  ;;; Copyright © 2021 Simon Tournier <zimon.toutoune <at> gmail.com>
>  ;;; Copyright © 2021 Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
>  ;;; Copyright © 2021 Ahmad Jarara <git <at> ajarara.io>
> +;;; Copyright © 2022 ( <paren <at> disroot.org>
>  ;;;
>  ;;; This file is part of GNU Guix.
>  ;;;
> @@ -2784,3 +2785,34 @@ (define-public fcrackzip
>      (synopsis "Zip password cracker")
>      (description "Fcrackzip is a Zip file password cracker.")
>      (license license:gpl2+)))
> +
> +(define-public miniz
> +  ;; Last release was on 27-06-2021; there have been many commits
> since
> +  ;; then.
And there might still be many more until a release.  I don't think
using untagged commits "just because" if the latest release is barely a
year old is a good idea.
> +  (let ((commit "be92575500f71dfb5ac7b0c5bb47b6e736c23039")
> +        (revision "0"))
> +    (package
> +     (name "miniz")
> +     (version (git-version "2.2.0" revision commit))
> +     (source (origin
> +              (method git-fetch)
> +              (uri (git-reference
> +                    (url "https://github.com/richgel999/miniz")
> +                    (commit commit)))
> +              (file-name (git-file-name name version))
> +              (sha256
> +               (base32
> +               
> "0w42vagcv4b0hcf0jk7vamrcpmw62dblx9856j98msdfm3qv1xzi"))))
> +     (build-system cmake-build-system)
> +     (arguments
> +      (list #:configure-flags
> +            #~(list "-DBUILD_SHARED_LIBS=ON")
> +            ;; No test target.
> +            #:tests? #f))
There is a test script you could use.
> +     (home-page "https://github.com/richgel999/miniz")
> +     (synopsis "Simple replacement for @code{zlib}")
> +     (description
> +      "@code{miniz} is a small, simple alternative to the
> @code{zlib}
> +data compression library for manipulating zlib, DEFLATE, PNG, and
> ZIP
> +files from C.")
Avoid marketing terms :)




Information forwarded to guix-patches <at> gnu.org:
bug#57034; Package guix-patches. (Sun, 07 Aug 2022 17:00:01 GMT) Full text and rfc822 format available.

Message #11 received at 57034 <at> debbugs.gnu.org (full text, mbox):

From: "(" <paren <at> disroot.org>
To: "Liliana Marie Prikler" <liliana.prikler <at> gmail.com>,
 <57034 <at> debbugs.gnu.org>
Subject: Re: [PATCH] gnu: packages: Add miniz.
Date: Sun, 07 Aug 2022 17:59:19 +0100
On Sun Aug 7, 2022 at 5:42 PM BST, Liliana Marie Prikler wrote:
> last time I checked, the template was "gnu: Add PACKAGE", not "gnu:
> packages: Add PACKAGE."
Okay, I'll remember that for next time :)

> And there might still be many more until a release.  I don't think
> using untagged commits "just because" if the latest release is barely a
> year old is a good idea.

> > +  (let ((commit "be92575500f71dfb5ac7b0c5bb47b6e736c23039")
> > +        (revision "0"))
> > +    (package
> > +     (name "miniz")
> > +     (version (git-version "2.2.0" revision commit))
> > +     (source (origin
> > +              (method git-fetch)
> > +              (uri (git-reference
> > +                    (url "https://github.com/richgel999/miniz")
> > +                    (commit commit)))
> > +              (file-name (git-file-name name version))
> > +              (sha256
> > +               (base32
> > +               
> > "0w42vagcv4b0hcf0jk7vamrcpmw62dblx9856j98msdfm3qv1xzi"))))
> > +     (build-system cmake-build-system)
> > +     (arguments
> > +      (list #:configure-flags
> > +            #~(list "-DBUILD_SHARED_LIBS=ON")
> > +            ;; No test target.
> > +            #:tests? #f))
> There is a test script you could use.
It doesn't work; I tried it.

> > +      "@code{miniz} is a small, simple alternative to the
> > @code{zlib}
> > +data compression library for manipulating zlib, DEFLATE, PNG, and
> > ZIP
> > +files from C.")
> Avoid marketing terms :)
Do 'small' and 'simple' count as marketing terms? 'smaller than zlib'
is objective fact, and 'simpler than zlib' is subjective but pretty
obvious. Maybe I could rewrite it to say 'smaller, simpler alternative'
instead?

    -- (




Information forwarded to guix-patches <at> gnu.org:
bug#57034; Package guix-patches. (Sun, 07 Aug 2022 17:02:01 GMT) Full text and rfc822 format available.

Message #14 received at 57034 <at> debbugs.gnu.org (full text, mbox):

From: "(" <paren <at> disroot.org>
To: "Liliana Marie Prikler" <liliana.prikler <at> gmail.com>,
 <57034 <at> debbugs.gnu.org>
Subject: Re: [PATCH] gnu: packages: Add miniz.
Date: Sun, 07 Aug 2022 18:01:46 +0100
On Sun Aug 7, 2022 at 5:42 PM BST, Liliana Marie Prikler wrote:
> And there might still be many more until a release.  I don't think
> using untagged commits "just because" if the latest release is barely a
> year old is a good idea.
Okay, I'll update it to use the latest release.

    -- (




Information forwarded to guix-patches <at> gnu.org:
bug#57034; Package guix-patches. (Sun, 07 Aug 2022 17:21:01 GMT) Full text and rfc822 format available.

Message #17 received at 57034 <at> debbugs.gnu.org (full text, mbox):

From: Liliana Marie Prikler <liliana.prikler <at> gmail.com>
To: "(" <paren <at> disroot.org>, 57034 <at> debbugs.gnu.org
Subject: Re: [PATCH] gnu: packages: Add miniz.
Date: Sun, 07 Aug 2022 19:19:57 +0200
Am Sonntag, dem 07.08.2022 um 17:59 +0100 schrieb (:
> On Sun Aug 7, 2022 at 5:42 PM BST, Liliana Marie Prikler wrote:
> > last time I checked, the template was "gnu: Add PACKAGE", not "gnu:
> > packages: Add PACKAGE."
> Okay, I'll remember that for next time :)
> 
> > And there might still be many more until a release.  I don't think
> > using untagged commits "just because" if the latest release is
> > barely a
> > year old is a good idea.
> 
> > > +  (let ((commit "be92575500f71dfb5ac7b0c5bb47b6e736c23039")
> > > +        (revision "0"))
> > > +    (package
> > > +     (name "miniz")
> > > +     (version (git-version "2.2.0" revision commit))
> > > +     (source (origin
> > > +              (method git-fetch)
> > > +              (uri (git-reference
> > > +                    (url "https://github.com/richgel999/miniz")
> > > +                    (commit commit)))
> > > +              (file-name (git-file-name name version))
> > > +              (sha256
> > > +               (base32
> > > +               
> > > "0w42vagcv4b0hcf0jk7vamrcpmw62dblx9856j98msdfm3qv1xzi"))))
> > > +     (build-system cmake-build-system)
> > > +     (arguments
> > > +      (list #:configure-flags
> > > +            #~(list "-DBUILD_SHARED_LIBS=ON")
> > > +            ;; No test target.
> > > +            #:tests? #f))
> > There is a test script you could use.
> It doesn't work; I tried it.
> 
> > > +      "@code{miniz} is a small, simple alternative to the
> > > @code{zlib}
> > > +data compression library for manipulating zlib, DEFLATE, PNG,
> > > and
> > > ZIP
> > > +files from C.")
> > Avoid marketing terms :)
> Do 'small' and 'simple' count as marketing terms? 'smaller than zlib'
> is objective fact, and 'simpler than zlib' is subjective but pretty
> obvious. Maybe I could rewrite it to say 'smaller, simpler
> alternative'
> instead?
"subjective, but pretty obvious", how so?  I think the fact that it
doesn't have its own documentation and you need to instead refer to
zlib kinda defeats that point.  Several of its README claims seem
either outdated already (the 550 lines one for example), or difficult
to guarantee forever.  I think the only claim we can reasonably make is
that it is an alternative zlib implementation, and that it supports
zlib, deflate, png and zip.

Cheers




Information forwarded to guix-patches <at> gnu.org:
bug#57034; Package guix-patches. (Sun, 07 Aug 2022 17:23:01 GMT) Full text and rfc822 format available.

Message #20 received at 57034 <at> debbugs.gnu.org (full text, mbox):

From: "(" <paren <at> disroot.org>
To: "Liliana Marie Prikler" <liliana.prikler <at> gmail.com>,
 <57034 <at> debbugs.gnu.org>
Subject: Re: [PATCH] gnu: packages: Add miniz.
Date: Sun, 07 Aug 2022 18:22:28 +0100
On Sun Aug 7, 2022 at 6:19 PM BST, Liliana Marie Prikler wrote:
> "subjective, but pretty obvious", how so?  I think the fact that it
> doesn't have its own documentation and you need to instead refer to
> zlib kinda defeats that point.  Several of its README claims seem
> either outdated already (the 550 lines one for example), or difficult
> to guarantee forever.  I think the only claim we can reasonably make is
> that it is an alternative zlib implementation, and that it supports
> zlib, deflate, png and zip.
Okay. (There is a little documentation in the headers, by the way.) :)

    -- (




Information forwarded to guix-patches <at> gnu.org:
bug#57034; Package guix-patches. (Mon, 08 Aug 2022 00:37:01 GMT) Full text and rfc822 format available.

Message #23 received at 57034 <at> debbugs.gnu.org (full text, mbox):

From: Tobias Geerinckx-Rice <me <at> tobias.gr>
To: guix-patches <at> gnu.org, Liliana Marie Prikler <liliana.prikler <at> gmail.com>,
 "(" <paren <at> disroot.org>, 57034 <at> debbugs.gnu.org
Subject: Re: [bug#57034] [PATCH] gnu: packages: Add miniz.
Date: Mon, 08 Aug 2022 00:36:20 +0000
On 7 August 2022 16:42:35 UTC, Liliana Marie Prikler <liliana.prikler <at> gmail.com> wrote:
>> +     (synopsis "Simple replacement for @code{zlib}")
>> +     (description
>> +      "@code{miniz} is a small, simple alternative to the
>> @code{zlib}
>> +data compression library for manipulating zlib, DEFLATE, PNG, and
>> ZIP
>> +files from C.")
>Avoid marketing terms :)
 
(Small and simple are marketing terms?  Be right back; I need to update my profile...)

...no: avoid misleading or meaningless drivel, puffery, and redundancy.  Do, however, point out some basic features that might help users decide which of our oofteen deflate libraries fits their needs.  Add -er if you think it helps, or add some examples.

Otherwise, a common-sense guideline risks devolving into 'avoid adjectives; you might accidentally write a useful description'.



Kind regards,

T G-R

Sent on the go.  Excuse or enjoy my brevity.




Information forwarded to guix-patches <at> gnu.org:
bug#57034; Package guix-patches. (Mon, 08 Aug 2022 00:38:01 GMT) Full text and rfc822 format available.

Information forwarded to guix-patches <at> gnu.org:
bug#57034; Package guix-patches. (Mon, 08 Aug 2022 04:30:02 GMT) Full text and rfc822 format available.

Message #29 received at 57034 <at> debbugs.gnu.org (full text, mbox):

From: Liliana Marie Prikler <liliana.prikler <at> gmail.com>
To: Tobias Geerinckx-Rice <me <at> tobias.gr>, "(" <paren <at> disroot.org>, 
 57034 <at> debbugs.gnu.org
Subject: Re: [bug#57034] [PATCH] gnu: packages: Add miniz.
Date: Mon, 08 Aug 2022 06:29:23 +0200
Am Montag, dem 08.08.2022 um 00:36 +0000 schrieb Tobias Geerinckx-Rice:
> On 7 August 2022 16:42:35 UTC, Liliana Marie Prikler
> <liliana.prikler <at> gmail.com> wrote:
> > > +     (synopsis "Simple replacement for @code{zlib}")
> > > +     (description
> > > +      "@code{miniz} is a small, simple alternative to the
> > > @code{zlib}
> > > +data compression library for manipulating zlib, DEFLATE, PNG,
> > > and
> > > ZIP
> > > +files from C.")
> > Avoid marketing terms :)
>  
> (Small and simple are marketing terms?  Be right back; I need to
> update my profile...)
> 
> ...no: avoid misleading or meaningless drivel, puffery, and
> redundancy.  Do, however, point out some basic features that might
> help users decide which of our oofteen deflate libraries fits their
> needs.  Add -er if you think it helps, or add some examples.
Maybe not always, but I think in this context they are.  Especially
"simple", who decides that?  Pardon my cynicism here, but my personal
opinion w.r.t. compression libraries is that projects tend to pick the
one they're already contractually obligated to use FSVO contractually
obligated.




Information forwarded to guix-patches <at> gnu.org:
bug#57034; Package guix-patches. (Mon, 12 Dec 2022 22:46:02 GMT) Full text and rfc822 format available.

Message #32 received at 57034 <at> debbugs.gnu.org (full text, mbox):

From: "(" <paren <at> disroot.org>
To: 57034 <at> debbugs.gnu.org
Cc: "\(" <paren <at> disroot.org>
Subject: [PATCH v2] gnu: packages: Add miniz.
Date: Mon, 12 Dec 2022 22:45:04 +0000
* gnu/packages/compression.scm (miniz): New variable.
---
 gnu/packages/compression.scm | 28 ++++++++++++++++++++++++++++
 1 file changed, 28 insertions(+)

diff --git a/gnu/packages/compression.scm b/gnu/packages/compression.scm
index d2a1ed36f7..aa6404d1a3 100644
--- a/gnu/packages/compression.scm
+++ b/gnu/packages/compression.scm
@@ -36,6 +36,7 @@
 ;;; Copyright © 2021 Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
 ;;; Copyright © 2021 Ahmad Jarara <git <at> ajarara.io>
 ;;; Copyright © 2022 Zhu Zihao <all_but_last <at> 163.com>
+;;; Copyright © 2022 ( <paren <at> disroot.org>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -2820,3 +2821,30 @@ (define-public fcrackzip
     (synopsis "Zip password cracker")
     (description "Fcrackzip is a Zip file password cracker.")
     (license license:gpl2+)))
+
+(define-public miniz
+  (package
+    (name "miniz")
+    (version "3.0.1")
+    (source (origin
+              (method git-fetch)
+              (uri (git-reference
+                    (url "https://github.com/richgel999/miniz")
+                    (commit commit)))
+              (file-name (git-file-name name version))
+              (sha256
+               (base32
+                "0lchj2yhvaiy0bc28xqnjqjhv4yj4kffcgxgfiikn634milwd5b4"))))
+     (build-system cmake-build-system)
+     (arguments
+      (list #:configure-flags
+            #~(list "-DBUILD_SHARED_LIBS=ON")
+            ;; No test target.
+            #:tests? #f))
+     (home-page "https://github.com/richgel999/miniz")
+     (synopsis "Alternative to zlib")
+     (description
+      "miniz is an alternative to, and in many cases drop-in replacement for, the
+zlib data compression libraries, which provides APIs for manipulating zlib, DEFLATE,
+PNG, and ZIP files from C.")
+     (license license:expat)))

base-commit: 1d160f41f5e973ea2bbbf8bc80d216b5d9b0974c
-- 
2.38.1





This bug report was last modified 1 year and 106 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.