From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from lists.gentoo.org (pigeon.gentoo.org [208.92.234.80]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by finch.gentoo.org (Postfix) with ESMTPS id EE95A158451 for ; Mon, 8 Jan 2024 03:34:16 +0000 (UTC) Received: from pigeon.gentoo.org (localhost [127.0.0.1]) by pigeon.gentoo.org (Postfix) with SMTP id 37C5C2BC01B; Mon, 8 Jan 2024 03:34:16 +0000 (UTC) Received: from smtp.gentoo.org (smtp.gentoo.org [IPv6:2001:470:ea4a:1:5054:ff:fec7:86e4]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by pigeon.gentoo.org (Postfix) with ESMTPS id 1917B2BC01B for ; Mon, 8 Jan 2024 03:34:16 +0000 (UTC) Received: from oystercatcher.gentoo.org (oystercatcher.gentoo.org [148.251.78.52]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by smtp.gentoo.org (Postfix) with ESMTPS id 286EB3432DA for ; Mon, 8 Jan 2024 03:34:15 +0000 (UTC) Received: from localhost.localdomain (localhost [IPv6:::1]) by oystercatcher.gentoo.org (Postfix) with ESMTP id 89CE995D for ; Mon, 8 Jan 2024 03:34:13 +0000 (UTC) From: "Göktürk Yüksek" To: gentoo-commits@lists.gentoo.org Content-Transfer-Encoding: 8bit Content-type: text/plain; charset=UTF-8 Reply-To: gentoo-dev@lists.gentoo.org, "Göktürk Yüksek" Message-ID: <1704684817.6df45df54e7fd8e2c9151d333b5404dc88b9e96e.gokturk@gentoo> Subject: [gentoo-commits] repo/gentoo:master commit in: sys-fs/ext4magic/, sys-fs/ext4magic/files/ X-VCS-Repository: repo/gentoo X-VCS-Files: sys-fs/ext4magic/ext4magic-0.3.2-r2.ebuild sys-fs/ext4magic/files/ext4magic-0.3.2-fix-sys-types-h.patch X-VCS-Directories: sys-fs/ext4magic/files/ sys-fs/ext4magic/ X-VCS-Committer: gokturk X-VCS-Committer-Name: Göktürk Yüksek X-VCS-Revision: 6df45df54e7fd8e2c9151d333b5404dc88b9e96e X-VCS-Branch: master Date: Mon, 8 Jan 2024 03:34:13 +0000 (UTC) Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-Id: Gentoo Linux mail X-BeenThere: gentoo-commits@lists.gentoo.org X-Auto-Response-Suppress: DR, RN, NRN, OOF, AutoReply X-Archives-Salt: 20ba4872-cedd-4ecf-ac67-a68bb8126a30 X-Archives-Hash: 8fd0dc585a810a8c3b33f0d893fc1d1e commit: 6df45df54e7fd8e2c9151d333b5404dc88b9e96e Author: Göktürk Yüksek gentoo org> AuthorDate: Mon Jan 8 03:22:27 2024 +0000 Commit: Göktürk Yüksek gentoo org> CommitDate: Mon Jan 8 03:33:37 2024 +0000 URL: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=6df45df5 sys-fs/ext4magic: fix building with musl #716136 Also update EAPI 6 -> 8, and update LICENSES. Closes: https://bugs.gentoo.org/716136 Signed-off-by: Göktürk Yüksek gentoo.org> sys-fs/ext4magic/ext4magic-0.3.2-r2.ebuild | 37 +++++ .../files/ext4magic-0.3.2-fix-sys-types-h.patch | 183 +++++++++++++++++++++ 2 files changed, 220 insertions(+) diff --git a/sys-fs/ext4magic/ext4magic-0.3.2-r2.ebuild b/sys-fs/ext4magic/ext4magic-0.3.2-r2.ebuild new file mode 100644 index 000000000000..1ddc9bc13dec --- /dev/null +++ b/sys-fs/ext4magic/ext4magic-0.3.2-r2.ebuild @@ -0,0 +1,37 @@ +# Copyright 1999-2024 Gentoo Authors +# Distributed under the terms of the GNU General Public License v2 + +EAPI=8 + +DESCRIPTION="Tool for recovery of deleted or overwritten files on ext3/ext4 filesystems" +HOMEPAGE="https://sourceforge.net/projects/ext4magic/" +SRC_URI="mirror://sourceforge/${PN}/${P}.tar.gz" + +LICENSE="BSD-2 FSFAP GPL-2 GPL-2+" +SLOT="0" +KEYWORDS="~amd64 ~arm64 ~x86" +IUSE="debug +expert-mode file-attr" + +RDEPEND="app-arch/bzip2 + >=sys-apps/file-5.04 + sys-apps/util-linux + >=sys-fs/e2fsprogs-1.41.9 + sys-libs/zlib" +DEPEND="${RDEPEND}" + +DOCS="AUTHORS ChangeLog NEWS README TODO" + +PATCHES=( + "${FILESDIR}"/${PN}-0.3.2-sysmacros.patch + "${FILESDIR}"/${PN}-0.3.2-i_dir_acl.patch + "${FILESDIR}"/${PN}-0.3.2-fix-sys-types-h.patch +) + +src_configure() { + # build-system incorrectly recognizes '--disable-feature' options as enabled! + econf \ + $(usex debug '--enable-debug' '') \ + $(usex debug '--enable-debug-magic' '') \ + $(usex expert-mode '--enable-expert-mode' '') \ + $(usex file-attr '--enable-file-attr' '') +} diff --git a/sys-fs/ext4magic/files/ext4magic-0.3.2-fix-sys-types-h.patch b/sys-fs/ext4magic/files/ext4magic-0.3.2-fix-sys-types-h.patch new file mode 100644 index 000000000000..f1e1d1c91716 --- /dev/null +++ b/sys-fs/ext4magic/files/ext4magic-0.3.2-fix-sys-types-h.patch @@ -0,0 +1,183 @@ +From 22960546489285b03857609c5c3f6c25ae91a6cd Mon Sep 17 00:00:00 2001 +From: =?UTF-8?q?G=C3=B6kt=C3=BCrk=20Y=C3=BCksek?= +Date: Sun, 7 Jan 2024 16:59:09 -0800 +Subject: [PATCH 1/1] Include config.h in various files before any other + include statements +MIME-Version: 1.0 +Content-Type: text/plain; charset=UTF-8 +Content-Transfer-Encoding: 8bit + +This is an indirect way to fix a compilation failure with musl +libc. Glibc includes sys/stat.h as part of stdlib.h[1], which +eventually typedefs dev_t[2]. Musl libc doesn't seem to pull that in +like glibc does. The header file ext2fs.h uses dev_t in some of its +function signatures[3], but the inclusion of sys/types.h is guarded by +the macro `#ifdef HAVE_SYS_TYPES_H`[4]. + +So every time ext4magic tries to include ext2fs.h while building with musl, it fails with: + +``` +In file included from hard_link_stack.h:23, +from hard_link_stack.c:25: +/usr/include/ext2fs/ext2fs.h:1402:39: error: unknown type name 'dev_t'; did you mean 'div_t'? +1402 | extern char *ext2fs_find_block_device(dev_t device); +| ^~~~~ +| div_t +/usr/include/ext2fs/ext2fs.h:1822:62: error: unknown type name 'mode_t' +1822 | extern int ext2fs_open_file(const char *pathname, int flags, mode_t mode); +| ^~~~~~ +make[2]: *** [Makefile:467: ext4magic-hard_link_stack.o] Error 1 +``` + +Autoconf actually declares HAVE_SYS_TYPES_H during ext4magic build, we +just need to include config.h in the relevant C files so that the +macro propagates. The alternative would be to pass -DHAVE_SYS_TYPES_H +to gcc but that solution seems more of a hack compared to including +config.h as it's meant to be included. + +[1] https://sourceware.org/git/?p=glibc.git;a=blob;f=include/stdlib.h;h=580da9be15adf0c1034986f62dd89aaaf6498c3f;hb=HEAD#l20 +[2] https://sourceware.org/git/?p=glibc.git;a=blob;f=io/sys/stat.h;h=1fa6d6e62ecb2e4b4f0039d0154307a6c27e3fa9;hb=HEAD#l40 +[3] https://git.kernel.org/pub/scm/fs/ext2/e2fsprogs.git/tree/lib/ext2fs/ext2fs.h#n1402 +[4] https://git.kernel.org/pub/scm/fs/ext2/e2fsprogs.git/tree/lib/ext2fs/ext2fs.h#n68 + +Bug: https://bugs.gentoo.org/716136 +Signed-off-by: Göktürk Yüksek +--- + src/block.c | 4 ++++ + src/file_type.c | 5 +++++ + src/hard_link_stack.c | 4 ++++ + src/imap_search.c | 4 ++++ + src/inode.c | 5 +++++ + src/lookup_local.c | 5 +++++ + src/magic_block_scan.c | 5 +++++ + src/util.c | 5 +++++ + 8 files changed, 37 insertions(+) + +diff --git a/src/block.c b/src/block.c +index 84fc1bb..4681d8e 100644 +--- a/src/block.c ++++ b/src/block.c +@@ -12,6 +12,10 @@ + * %End-Header% + */ + ++#ifdef HAVE_CONFIG_H ++#include ++#endif ++ + /* + This is a workaround to allow compilation, but the one line that uses + this constant will never run because we open the fs read-only. +diff --git a/src/file_type.c b/src/file_type.c +index a1396c1..cf1790a 100644 +--- a/src/file_type.c ++++ b/src/file_type.c +@@ -17,6 +17,11 @@ + * * + * C Implementation: file_type * + ***************************************************************************/ ++ ++#ifdef HAVE_CONFIG_H ++#include ++#endif ++ + #include + #include + #include +diff --git a/src/hard_link_stack.c b/src/hard_link_stack.c +index abef55e..df7666f 100644 +--- a/src/hard_link_stack.c ++++ b/src/hard_link_stack.c +@@ -16,6 +16,10 @@ + * along with this program; if not, see . * + ***************************************************************************/ + ++#ifdef HAVE_CONFIG_H ++#include ++#endif ++ + //construct for global collect of hardlinks + + #include +diff --git a/src/imap_search.c b/src/imap_search.c +index 3c9c180..b8025c9 100644 +--- a/src/imap_search.c ++++ b/src/imap_search.c +@@ -19,6 +19,10 @@ + * C Implementation: imap_search * + ***************************************************************************/ + ++#ifdef HAVE_CONFIG_H ++#include ++#endif ++ + //header util.h + + #include "util.h" +diff --git a/src/inode.c b/src/inode.c +index 5cbf52e..db8c915 100644 +--- a/src/inode.c ++++ b/src/inode.c +@@ -15,6 +15,11 @@ + * You should have received a copy of the GNU General Public License * + * along with this program; if not, see . * + ***************************************************************************/ ++ ++#ifdef HAVE_CONFIG_H ++#include ++#endif ++ + #include + #include + #include +diff --git a/src/lookup_local.c b/src/lookup_local.c +index c566809..9dcd8aa 100644 +--- a/src/lookup_local.c ++++ b/src/lookup_local.c +@@ -15,6 +15,11 @@ + * You should have received a copy of the GNU General Public License * + * along with this program; if not, see . * + ***************************************************************************/ ++ ++#ifdef HAVE_CONFIG_H ++#include ++#endif ++ + //header util.h + + #include +diff --git a/src/magic_block_scan.c b/src/magic_block_scan.c +index 000cf80..d0844e1 100644 +--- a/src/magic_block_scan.c ++++ b/src/magic_block_scan.c +@@ -18,6 +18,11 @@ + * * + * C Implementation: magic_block_scan * + ***************************************************************************/ ++ ++#ifdef HAVE_CONFIG_H ++#include ++#endif ++ + #include + #include + #include +diff --git a/src/util.c b/src/util.c +index b148baa..99a58fc 100644 +--- a/src/util.c ++++ b/src/util.c +@@ -15,6 +15,11 @@ + * You should have received a copy of the GNU General Public License * + * along with this program; if not, see . * + ***************************************************************************/ ++ ++#ifdef HAVE_CONFIG_H ++#include ++#endif ++ + #include + #include + #include +-- +2.43.0 +