guix-patches
[Top][All Lists]
Advanced

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

[bug#57337] [v2 1/2] gnu: Add miniz-cpp.


From: Ludovic Courtès
Subject: [bug#57337] [v2 1/2] gnu: Add miniz-cpp.
Date: Wed, 25 Jan 2023 23:17:18 +0100
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux)

Hi Wiktor,

Apologies for the delay.

Wiktor Żelazny <wz@freeshell.de> skribis:

> * gnu/packages/cpp.scm (miniz-cpp): New variable.
> ---
>  gnu/local.mk                                  |    1 +
>  gnu/packages/cpp.scm                          |   35 +
>  .../miniz-cpp-fixed-duplicated-symbols.patch  | 2734 +++++++++++++++++
>  3 files changed, 2770 insertions(+)
>  create mode 100644 
> gnu/packages/patches/miniz-cpp-fixed-duplicated-symbols.patch

That’s a pretty big patch, and we don’t usually take that big a patch
downstream, but…

> +                (patches (search-patches
> +                          ;; To be removed once 
> https://github.com/tfussell/miniz-cpp/pull/11
> +                          ;; gets merged.
> +                          "miniz-cpp-fixed-duplicated-symbols.patch"))))

Looking at the GitHub issue, it seems the solution could be merge soon.

How about (1) pinging upstream asking whether they could merge it, and
(2) referring to the commit of this pull request in the package
definition in the meantime, with a comment explaining why?

Thanks in advance,
Ludo’.





reply via email to

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