diff options
author | Sam James <sam@gentoo.org> | 2023-08-19 05:31:19 +0100 |
---|---|---|
committer | Sam James <sam@gentoo.org> | 2023-08-19 05:31:19 +0100 |
commit | 491bf8bd3ef621da49b168736a04f23aa4c7719c (patch) | |
tree | 703dffbcc8d02d0f0e4cdfcce84d6bff4986a92e /sys-libs/zlib | |
parent | dev-db/mydumper: Stabilize 0.14.5.2 amd64, #912453 (diff) | |
download | gentoo-491bf8bd3ef621da49b168736a04f23aa4c7719c.tar.gz gentoo-491bf8bd3ef621da49b168736a04f23aa4c7719c.tar.bz2 gentoo-491bf8bd3ef621da49b168736a04f23aa4c7719c.zip |
sys-libs/zlib: drop macro renaming hacks
Renaming zlib's `OF()` macro has caused problems for various packages; fortunately
C23 kills K&R prototypes so its use should be dying out anyway. If anyone is still
using it, we should just patch it out entirely rather than renaming it.
Other distros have been coping fine without the rename anyway. The most important
point being that this is against POLA and leads to problems when people try to build
things using zlib that haven't been fixed yet.
See also the legendary godot patch to workaround this:
https://github.com/godotengine/godot/blob/93409b8e64a9bc3c271ab4a7489b59a43bc0d048/thirdparty/minizip/patches/unbreak-gentoo.patch.
As of zlib-1.3, zlib doesn't even use it by itself.
Bug: https://bugs.gentoo.org/383179
Signed-off-by: Sam James <sam@gentoo.org>
Diffstat (limited to 'sys-libs/zlib')
-rw-r--r-- | sys-libs/zlib/zlib-1.3-r1.ebuild (renamed from sys-libs/zlib/zlib-1.3.ebuild) | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/sys-libs/zlib/zlib-1.3.ebuild b/sys-libs/zlib/zlib-1.3-r1.ebuild index 6dbffd69d64f..22051ef75784 100644 --- a/sys-libs/zlib/zlib-1.3.ebuild +++ b/sys-libs/zlib/zlib-1.3-r1.ebuild @@ -130,12 +130,6 @@ multilib_src_compile() { use minizip && emake -C contrib/minizip } -sed_macros() { - # Clean up namespace a little, bug #383179 - # We do it here so we only have to tweak 2 files - sed -i -r 's:\<(O[FN])\>:_Z_\1:g' "$@" || die -} - multilib_src_install() { case ${CHOST} in *-mingw*|mingw*) |