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

From: Ludovic Courtès <[email-masked]>
Date: Wed, 25 Jan 2023 23:17:18 +0100

Hi Wiktor,

Apologies for the delay.

Wiktor Żelazny <[email-masked]> 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’.