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.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by finch.gentoo.org (Postfix) with ESMTPS id C9AAF138334 for ; Sat, 1 Sep 2018 01:32:24 +0000 (UTC) Received: from pigeon.gentoo.org (localhost [127.0.0.1]) by pigeon.gentoo.org (Postfix) with SMTP id 7E75DE07FE; Sat, 1 Sep 2018 01:32:23 +0000 (UTC) Received: from smtp.gentoo.org (dev.gentoo.org [IPv6:2001:470:ea4a:1:5054:ff:fec7:86e4]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by pigeon.gentoo.org (Postfix) with ESMTPS id 41BB0E07FE for ; Sat, 1 Sep 2018 01:32:22 +0000 (UTC) Received: from oystercatcher.gentoo.org (oystercatcher.gentoo.org [148.251.78.52]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.gentoo.org (Postfix) with ESMTPS id CA66A335CA0 for ; Sat, 1 Sep 2018 01:32:20 +0000 (UTC) Received: from localhost.localdomain (localhost [IPv6:::1]) by oystercatcher.gentoo.org (Postfix) with ESMTP id C8F963A7 for ; Sat, 1 Sep 2018 01:32:18 +0000 (UTC) From: "Thomas Deutschmann" To: gentoo-commits@lists.gentoo.org Content-Transfer-Encoding: 8bit Content-type: text/plain; charset=UTF-8 Reply-To: gentoo-dev@lists.gentoo.org, "Thomas Deutschmann" Message-ID: <1535765526.3c3ef606b855d35fb0e0fc9dcc0c005b28f5e6d3.whissi@gentoo> Subject: [gentoo-commits] repo/gentoo:master commit in: sys-apps/keyutils/ X-VCS-Repository: repo/gentoo X-VCS-Files: sys-apps/keyutils/keyutils-1.5.10-r1.ebuild sys-apps/keyutils/keyutils-1.5.11-r1.ebuild X-VCS-Directories: sys-apps/keyutils/ X-VCS-Committer: whissi X-VCS-Committer-Name: Thomas Deutschmann X-VCS-Revision: 3c3ef606b855d35fb0e0fc9dcc0c005b28f5e6d3 X-VCS-Branch: master Date: Sat, 1 Sep 2018 01:32:18 +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-Archives-Salt: 449a48ac-32df-410c-b85a-d025e0594ed5 X-Archives-Hash: 9d55c0bec1d19e74bd71f3b2f0e553be commit: 3c3ef606b855d35fb0e0fc9dcc0c005b28f5e6d3 Author: Thomas Deutschmann gentoo org> AuthorDate: Sat Sep 1 01:20:01 2018 +0000 Commit: Thomas Deutschmann gentoo org> CommitDate: Sat Sep 1 01:32:06 2018 +0000 URL: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=3c3ef606 sys-apps/keyutils: kernel config checks rewritten - CONFIG_KEY_DH_OPERATIONS is only available in kernel >=4.7 - Package will now fail in setup phase when FEATURES=test is set and not all required kernel options are set to prevent false positive bug reports due to failures of test phase caused by missing options. Closes: https://bugs.gentoo.org/624380 Package-Manager: Portage-2.3.48, Repoman-2.3.10 sys-apps/keyutils/keyutils-1.5.10-r1.ebuild | 30 ++++++++++++++++++++++++----- sys-apps/keyutils/keyutils-1.5.11-r1.ebuild | 30 ++++++++++++++++++++++++----- 2 files changed, 50 insertions(+), 10 deletions(-) diff --git a/sys-apps/keyutils/keyutils-1.5.10-r1.ebuild b/sys-apps/keyutils/keyutils-1.5.10-r1.ebuild index 5a4a876d41c..e76b88beb5d 100644 --- a/sys-apps/keyutils/keyutils-1.5.10-r1.ebuild +++ b/sys-apps/keyutils/keyutils-1.5.10-r1.ebuild @@ -26,13 +26,33 @@ PATCHES=( ) pkg_setup() { - CONFIG_CHECK="~KEYS" - ERROR_KEYS="You must have CONFIG_KEYS to use this package!" + # To prevent a failure in test phase and false positive bug reports + # we are enforcing the following options because testsuite expects + # that these options are available. I.e. testsuite only decides based + # on kernel version which tests will be called, no feature checking. + if use test ; then + CONFIG_CHECK="KEYS" + ERROR_KEYS="You must have CONFIG_KEYS to run the package testsuite!" + + if kernel_is -ge 2 6 10 && kernel_is -lt 4 0 0 ; then + CONFIG_CHECK="${CONFIG_CHECK} KEYS_DEBUG_PROC_KEYS" + ERROR_KEYS_DEBUG_PROC_KEYS="You must have CONFIG_KEYS_DEBUG_PROC_KEYS to run the package testsuite!" + fi + + if kernel_is -ge 4 7 ; then + CONFIG_CHECK="${CONFIG_CHECK} KEY_DH_OPERATIONS" + ERROR_KEY_DH_OPERATIONS="You must have CONFIG_KEY_DH_OPERATIONS to run the package testsuite!" + fi + else + CONFIG_CHECK="~KEYS" + ERROR_KEYS="You will be unable to use this package on this system because CONFIG_KEYS is not set!" - if use test && kernel_is lt 4 0 0; then - CONFIG_CHECK="${CONFIG_CHECK} ~KEYS_DEBUG_PROC_KEYS ~KEY_DH_OPERATIONS" - ERROR_KEYS_DEBUG_PROC_KEYS="You must have CONFIG_KEYS_DEBUG_PROC_KEYS to run the package testsuite!" + if kernel_is -ge 4 7 ; then + CONFIG_CHECK="${CONFIG_CHECK} ~KEY_DH_OPERATIONS" + ERROR_KEY_DH_OPERATIONS="You will be unable to use Diffie-Hellman on this system because CONFIG_KEY_DH_OPERATIONS is not set!" + fi fi + linux-info_pkg_setup } diff --git a/sys-apps/keyutils/keyutils-1.5.11-r1.ebuild b/sys-apps/keyutils/keyutils-1.5.11-r1.ebuild index b47e4ff1d2d..23be07210ba 100644 --- a/sys-apps/keyutils/keyutils-1.5.11-r1.ebuild +++ b/sys-apps/keyutils/keyutils-1.5.11-r1.ebuild @@ -27,13 +27,33 @@ PATCHES=( ) pkg_setup() { - CONFIG_CHECK="~KEYS" - ERROR_KEYS="You must have CONFIG_KEYS to use this package!" + # To prevent a failure in test phase and false positive bug reports + # we are enforcing the following options because testsuite expects + # that these options are available. I.e. testsuite only decides based + # on kernel version which tests will be called, no feature checking. + if use test ; then + CONFIG_CHECK="KEYS" + ERROR_KEYS="You must have CONFIG_KEYS to run the package testsuite!" + + if kernel_is -ge 2 6 10 && kernel_is -lt 4 0 0 ; then + CONFIG_CHECK="${CONFIG_CHECK} KEYS_DEBUG_PROC_KEYS" + ERROR_KEYS_DEBUG_PROC_KEYS="You must have CONFIG_KEYS_DEBUG_PROC_KEYS to run the package testsuite!" + fi + + if kernel_is -ge 4 7 ; then + CONFIG_CHECK="${CONFIG_CHECK} KEY_DH_OPERATIONS" + ERROR_KEY_DH_OPERATIONS="You must have CONFIG_KEY_DH_OPERATIONS to run the package testsuite!" + fi + else + CONFIG_CHECK="~KEYS" + ERROR_KEYS="You will be unable to use this package on this system because CONFIG_KEYS is not set!" - if use test && kernel_is lt 4 0 0; then - CONFIG_CHECK="${CONFIG_CHECK} ~KEYS_DEBUG_PROC_KEYS ~KEY_DH_OPERATIONS" - ERROR_KEYS_DEBUG_PROC_KEYS="You must have CONFIG_KEYS_DEBUG_PROC_KEYS to run the package testsuite!" + if kernel_is -ge 4 7 ; then + CONFIG_CHECK="${CONFIG_CHECK} ~KEY_DH_OPERATIONS" + ERROR_KEY_DH_OPERATIONS="You will be unable to use Diffie-Hellman on this system because CONFIG_KEY_DH_OPERATIONS is not set!" + fi fi + linux-info_pkg_setup }