[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’.
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- [bug#57337] [v2 1/2] gnu: Add miniz-cpp.,
Ludovic Courtès <=