From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gentoo-commits+bounces-1350683-garchives=archives.gentoo.org@lists.gentoo.org>
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 5F9A9158087
	for <garchives@archives.gentoo.org>; Sat, 18 Dec 2021 04:39:44 +0000 (UTC)
Received: from pigeon.gentoo.org (localhost [127.0.0.1])
	by pigeon.gentoo.org (Postfix) with SMTP id 477172BC02F;
	Sat, 18 Dec 2021 04:39:43 +0000 (UTC)
Received: from smtp.gentoo.org (woodpecker.gentoo.org [140.211.166.183])
	(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 B2CFB2BC031
	for <gentoo-commits@lists.gentoo.org>; Sat, 18 Dec 2021 04:39:42 +0000 (UTC)
Received: from oystercatcher.gentoo.org (unknown [IPv6:2a01:4f8:202:4333:225:90ff:fed9:fc84])
	(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 3A72F3431A9
	for <gentoo-commits@lists.gentoo.org>; Sat, 18 Dec 2021 04:39:39 +0000 (UTC)
Received: from localhost.localdomain (localhost [IPv6:::1])
	by oystercatcher.gentoo.org (Postfix) with ESMTP id A8B3824E
	for <gentoo-commits@lists.gentoo.org>; Sat, 18 Dec 2021 04:39:37 +0000 (UTC)
From: "Sam James" <sam@gentoo.org>
To: gentoo-commits@lists.gentoo.org
Content-Transfer-Encoding: 8bit
Content-type: text/plain; charset=UTF-8
Reply-To: gentoo-dev@lists.gentoo.org, "Sam James" <sam@gentoo.org>
Message-ID: <1639802367.fa280d0503125ef715e6f52bbeae4c0ab2d09aea.sam@gentoo>
Subject: [gentoo-commits] repo/gentoo:master commit in: sys-fs/eudev/
X-VCS-Repository: repo/gentoo
X-VCS-Files: sys-fs/eudev/eudev-3.2.11.ebuild sys-fs/eudev/eudev-9999.ebuild
X-VCS-Directories: sys-fs/eudev/
X-VCS-Committer: sam
X-VCS-Committer-Name: Sam James
X-VCS-Revision: fa280d0503125ef715e6f52bbeae4c0ab2d09aea
X-VCS-Branch: master
Date: Sat, 18 Dec 2021 04:39:37 +0000 (UTC)
Precedence: bulk
List-Post: <mailto:gentoo-commits@lists.gentoo.org>
List-Help: <mailto:gentoo-commits+help@lists.gentoo.org>
List-Unsubscribe: <mailto:gentoo-commits+unsubscribe@lists.gentoo.org>
List-Subscribe: <mailto:gentoo-commits+subscribe@lists.gentoo.org>
List-Id: Gentoo Linux mail <gentoo-commits.gentoo.org>
X-BeenThere: gentoo-commits@lists.gentoo.org
X-Auto-Response-Suppress: DR, RN, NRN, OOF, AutoReply
X-Archives-Salt: b862fce3-f996-4541-9f97-883bb3fd8d96
X-Archives-Hash: 9ea474d0f2225ced24645c7d7b9725bf

commit:     fa280d0503125ef715e6f52bbeae4c0ab2d09aea
Author:     Sam James <sam <AT> gentoo <DOT> org>
AuthorDate: Sat Dec 18 03:44:34 2021 +0000
Commit:     Sam James <sam <AT> gentoo <DOT> org>
CommitDate: Sat Dec 18 04:39:27 2021 +0000
URL:        https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=fa280d05

sys-fs/eudev: modernise for EAPI 7

Signed-off-by: Sam James <sam <AT> gentoo.org>

 sys-fs/eudev/eudev-3.2.11.ebuild | 68 +++++++++++++++++++++-------------------
 sys-fs/eudev/eudev-9999.ebuild   | 68 +++++++++++++++++++++-------------------
 2 files changed, 72 insertions(+), 64 deletions(-)

diff --git a/sys-fs/eudev/eudev-3.2.11.ebuild b/sys-fs/eudev/eudev-3.2.11.ebuild
index f482a0348910..cf9f8b3b41a6 100644
--- a/sys-fs/eudev/eudev-3.2.11.ebuild
+++ b/sys-fs/eudev/eudev-3.2.11.ebuild
@@ -3,7 +3,7 @@
 
 EAPI=7
 
-KV_min=2.6.39
+KV_MIN=2.6.39
 
 inherit autotools linux-info multilib multilib-minimal toolchain-funcs
 
@@ -23,7 +23,8 @@ SLOT="0"
 IUSE="+kmod introspection rule-generator selinux static-libs test"
 RESTRICT="!test? ( test )"
 
-COMMON_DEPEND=">=sys-apps/util-linux-2.20
+DEPEND=">=sys-apps/util-linux-2.20
+	>=sys-kernel/linux-headers-${KV_MIN}
 	virtual/libcrypt:=
 	introspection? ( >=dev-libs/gobject-introspection-1.38 )
 	kmod? ( >=sys-apps/kmod-16 )
@@ -31,15 +32,7 @@ COMMON_DEPEND=">=sys-apps/util-linux-2.20
 	!<sys-libs/glibc-2.11
 	!sys-apps/gentoo-systemd-integration
 	!sys-apps/systemd"
-DEPEND="${COMMON_DEPEND}
-	dev-util/gperf
-	virtual/os-headers
-	virtual/pkgconfig
-	>=sys-devel/make-3.82-r4
-	>=sys-kernel/linux-headers-${KV_min}
-	test? ( app-text/tree dev-lang/perl )"
-
-RDEPEND="${COMMON_DEPEND}
+RDEPEND="${DEPEND}
 	acct-group/input
 	acct-group/kvm
 	acct-group/render
@@ -48,7 +41,11 @@ RDEPEND="${COMMON_DEPEND}
 	!sys-fs/udev
 	!sys-apps/systemd
 	!sys-apps/hwids[udev]"
-
+BDEPEND="dev-util/gperf
+	virtual/os-headers
+	virtual/pkgconfig
+	>=sys-devel/make-3.82-r4
+	test? ( app-text/tree dev-lang/perl )"
 PDEPEND=">=sys-fs/udev-init-scripts-26"
 
 MULTILIB_WRAPPED_HEADERS=(
@@ -75,10 +72,10 @@ pkg_setup() {
 
 	# These are required kernel options, but we don't error out on them
 	# because you can build under one kernel and run under another.
-	if kernel_is lt ${KV_min//./ }; then
+	if kernel_is lt ${KV_MIN//./ }; then
 		ewarn
 		ewarn "Your current running kernel version ${KV_FULL} is too old to run ${P}."
-		ewarn "Make sure to run udev under kernel version ${KV_min} or above."
+		ewarn "Make sure to run udev under kernel version ${KV_MIN} or above."
 		ewarn
 	fi
 }
@@ -86,15 +83,17 @@ pkg_setup() {
 src_prepare() {
 	# change rules back to group uucp instead of dialout for now
 	sed -e 's/GROUP="dialout"/GROUP="uucp"/' -i rules/*.rules \
-	|| die "failed to change group dialout to uucp"
+		|| die "failed to change group dialout to uucp"
 
-	eapply_user
+	default
 	eautoreconf
 }
 
 multilib_src_configure() {
-	tc-export CC #463846
-	export cc_cv_CFLAGS__flto=no #502950
+	# bug #463846
+	tc-export CC
+	# bug #502950
+	export cc_cv_CFLAGS__flto=no
 
 	# Keep sorted by ./configure --help and only pass --disable flags
 	# when *required* to avoid external deps or unnecessary compile
@@ -133,8 +132,10 @@ multilib_src_configure() {
 			--disable-kmod
 			--disable-selinux
 			--disable-rule-generator
+			--disable-hwdb
 		)
 	fi
+
 	ECONF_SOURCE="${S}" econf "${econf_args[@]}"
 }
 
@@ -147,14 +148,6 @@ multilib_src_compile() {
 	fi
 }
 
-multilib_src_install() {
-	if multilib_is_native_abi; then
-		emake DESTDIR="${D}" install
-	else
-		emake -C src/libudev DESTDIR="${D}" install
-	fi
-}
-
 multilib_src_test() {
 	# make sandbox get out of the way
 	# these are safe because there is a fake root filesystem put in place,
@@ -165,12 +158,22 @@ multilib_src_test() {
 		addread /sys
 		addwrite /dev
 		addwrite /run
+
 		default_src_test
 	fi
 }
 
+
+multilib_src_install() {
+	if multilib_is_native_abi; then
+		emake DESTDIR="${D}" install
+	else
+		emake -C src/libudev DESTDIR="${D}" install
+	fi
+}
+
 multilib_src_install_all() {
-	find "${D}" -name '*.la' -delete || die
+	find "${ED}" -name '*.la' -delete || die
 
 	insinto /lib/udev/rules.d
 	doins "${FILESDIR}"/40-gentoo.rules
@@ -182,12 +185,12 @@ multilib_src_install_all() {
 }
 
 pkg_postinst() {
-	mkdir -p "${EROOT}"run
+	mkdir -p "${EROOT}"/run
 
 	# "losetup -f" is confused if there is an empty /dev/loop/, Bug #338766
 	# So try to remove it here (will only work if empty).
-	rmdir "${EROOT}"dev/loop 2>/dev/null
-	if [[ -d ${EROOT}dev/loop ]]; then
+	rmdir "${EROOT}"/dev/loop 2>/dev/null
+	if [[ -d ${EROOT}/dev/loop ]]; then
 		ewarn "Please make sure your remove /dev/loop,"
 		ewarn "else losetup may be confused when looking for unused devices."
 	fi
@@ -207,15 +210,16 @@ pkg_postinst() {
 	done
 
 	if has_version 'sys-apps/hwids[udev]'; then
-		udevadm hwdb --update --root="${ROOT%/}"
+		udevadm hwdb --update --root="${ROOT}"
 
 		# https://cgit.freedesktop.org/systemd/systemd/commit/?id=1fab57c209035f7e66198343074e9cee06718bda
 		# reload database after it has be rebuilt, but only if we are not upgrading
 		# also pass if we are -9999 since who knows what hwdb related changes there might be
-		if [[ ${rvres} == doit* ]] && [[ ${ROOT%/} == "" ]] && [[ ${PV} != "9999" ]]; then
+		if [[ ${rvres} == doit* ]] && [[ -z ${ROOT} ]] && [[ ${PV} != "9999" ]]; then
 			udevadm control --reload
 		fi
 	fi
+
 	if [[ ${rvres} != doitnew ]]; then
 		ewarn
 		ewarn "You need to restart eudev as soon as possible to make the"

diff --git a/sys-fs/eudev/eudev-9999.ebuild b/sys-fs/eudev/eudev-9999.ebuild
index d7b7155f78de..99ca4c45aba6 100644
--- a/sys-fs/eudev/eudev-9999.ebuild
+++ b/sys-fs/eudev/eudev-9999.ebuild
@@ -3,7 +3,7 @@
 
 EAPI=7
 
-KV_min=2.6.39
+KV_MIN=2.6.39
 
 inherit autotools linux-info multilib multilib-minimal toolchain-funcs
 
@@ -23,7 +23,8 @@ SLOT="0"
 IUSE="+kmod introspection rule-generator selinux static-libs test"
 RESTRICT="!test? ( test )"
 
-COMMON_DEPEND=">=sys-apps/util-linux-2.20
+DEPEND=">=sys-apps/util-linux-2.20
+	>=sys-kernel/linux-headers-${KV_MIN}
 	virtual/libcrypt:=
 	introspection? ( >=dev-libs/gobject-introspection-1.38 )
 	kmod? ( >=sys-apps/kmod-16 )
@@ -31,15 +32,7 @@ COMMON_DEPEND=">=sys-apps/util-linux-2.20
 	!<sys-libs/glibc-2.11
 	!sys-apps/gentoo-systemd-integration
 	!sys-apps/systemd"
-DEPEND="${COMMON_DEPEND}
-	dev-util/gperf
-	virtual/os-headers
-	virtual/pkgconfig
-	>=sys-devel/make-3.82-r4
-	>=sys-kernel/linux-headers-${KV_min}
-	test? ( app-text/tree dev-lang/perl )"
-
-RDEPEND="${COMMON_DEPEND}
+RDEPEND="${DEPEND}
 	acct-group/input
 	acct-group/kvm
 	acct-group/render
@@ -48,7 +41,11 @@ RDEPEND="${COMMON_DEPEND}
 	!sys-fs/udev
 	!sys-apps/systemd
 	!sys-apps/hwids[udev]"
-
+BDEPEND="dev-util/gperf
+	virtual/os-headers
+	virtual/pkgconfig
+	>=sys-devel/make-3.82-r4
+	test? ( app-text/tree dev-lang/perl )"
 PDEPEND=">=sys-fs/udev-init-scripts-26"
 
 MULTILIB_WRAPPED_HEADERS=(
@@ -75,10 +72,10 @@ pkg_setup() {
 
 	# These are required kernel options, but we don't error out on them
 	# because you can build under one kernel and run under another.
-	if kernel_is lt ${KV_min//./ }; then
+	if kernel_is lt ${KV_MIN//./ }; then
 		ewarn
 		ewarn "Your current running kernel version ${KV_FULL} is too old to run ${P}."
-		ewarn "Make sure to run udev under kernel version ${KV_min} or above."
+		ewarn "Make sure to run udev under kernel version ${KV_MIN} or above."
 		ewarn
 	fi
 }
@@ -86,15 +83,17 @@ pkg_setup() {
 src_prepare() {
 	# change rules back to group uucp instead of dialout for now
 	sed -e 's/GROUP="dialout"/GROUP="uucp"/' -i rules/*.rules \
-	|| die "failed to change group dialout to uucp"
+		|| die "failed to change group dialout to uucp"
 
-	eapply_user
+	default
 	eautoreconf
 }
 
 multilib_src_configure() {
-	tc-export CC #463846
-	export cc_cv_CFLAGS__flto=no #502950
+	# bug #463846
+	tc-export CC
+	# bug #502950
+	export cc_cv_CFLAGS__flto=no
 
 	# Keep sorted by ./configure --help and only pass --disable flags
 	# when *required* to avoid external deps or unnecessary compile
@@ -133,8 +132,10 @@ multilib_src_configure() {
 			--disable-kmod
 			--disable-selinux
 			--disable-rule-generator
+			--disable-hwdb
 		)
 	fi
+
 	ECONF_SOURCE="${S}" econf "${econf_args[@]}"
 }
 
@@ -147,14 +148,6 @@ multilib_src_compile() {
 	fi
 }
 
-multilib_src_install() {
-	if multilib_is_native_abi; then
-		emake DESTDIR="${D}" install
-	else
-		emake -C src/libudev DESTDIR="${D}" install
-	fi
-}
-
 multilib_src_test() {
 	# make sandbox get out of the way
 	# these are safe because there is a fake root filesystem put in place,
@@ -165,12 +158,22 @@ multilib_src_test() {
 		addread /sys
 		addwrite /dev
 		addwrite /run
+
 		default_src_test
 	fi
 }
 
+
+multilib_src_install() {
+	if multilib_is_native_abi; then
+		emake DESTDIR="${D}" install
+	else
+		emake -C src/libudev DESTDIR="${D}" install
+	fi
+}
+
 multilib_src_install_all() {
-	find "${D}" -name '*.la' -delete || die
+	find "${ED}" -name '*.la' -delete || die
 
 	insinto /lib/udev/rules.d
 	doins "${FILESDIR}"/40-gentoo.rules
@@ -182,12 +185,12 @@ multilib_src_install_all() {
 }
 
 pkg_postinst() {
-	mkdir -p "${EROOT}"run
+	mkdir -p "${EROOT}"/run
 
 	# "losetup -f" is confused if there is an empty /dev/loop/, Bug #338766
 	# So try to remove it here (will only work if empty).
-	rmdir "${EROOT}"dev/loop 2>/dev/null
-	if [[ -d ${EROOT}dev/loop ]]; then
+	rmdir "${EROOT}"/dev/loop 2>/dev/null
+	if [[ -d ${EROOT}/dev/loop ]]; then
 		ewarn "Please make sure your remove /dev/loop,"
 		ewarn "else losetup may be confused when looking for unused devices."
 	fi
@@ -207,15 +210,16 @@ pkg_postinst() {
 	done
 
 	if has_version 'sys-apps/hwids[udev]'; then
-		udevadm hwdb --update --root="${ROOT%/}"
+		udevadm hwdb --update --root="${ROOT}"
 
 		# https://cgit.freedesktop.org/systemd/systemd/commit/?id=1fab57c209035f7e66198343074e9cee06718bda
 		# reload database after it has be rebuilt, but only if we are not upgrading
 		# also pass if we are -9999 since who knows what hwdb related changes there might be
-		if [[ ${rvres} == doit* ]] && [[ ${ROOT%/} == "" ]] && [[ ${PV} != "9999" ]]; then
+		if [[ ${rvres} == doit* ]] && [[ -z ${ROOT} ]] && [[ ${PV} != "9999" ]]; then
 			udevadm control --reload
 		fi
 	fi
+
 	if [[ ${rvres} != doitnew ]]; then
 		ewarn
 		ewarn "You need to restart eudev as soon as possible to make the"