Re: [gentoo-dev] [PATCH 6/7] linux-info.eclass: respect eselect kernel setting in binpkgs

2024-05-15 Thread Mike Gilbert
On Tue, May 14, 2024 at 7:59 AM Andrew Ammerlaan
 wrote:
> Commit f51cd5b64c14ddfb83488a12d538c66a4a309376 resets kernel environment
> variables when binpkgs are merged, this makes sense since we care about the
> system that the binpkg will be installed on, not the system the binpkg was
> built on. This introduced Bug 931213 as a side-effect, which will be fixed
> in a separate commit to linux-mod-r1.eclass. In preparation of that, we 
> document
> and rename the LINUX_INFO_BINARY_RESET to SKIP_KERNEL_BINPKG_ENV_RESET.
>
> However, commit f51cd5b64c14ddfb83488a12d538c66a4a309376 also makes binpkgs
> always use the running kernel version. This behaviour is surprising and
> confusing. In principle the 'eselect kernel' setting should still be respected
> when binpkgs are used.

Seems reasonable to me as the author of
f51cd5b64c14ddfb83488a12d538c66a4a309376.



[gentoo-dev] [PATCH 6/7] linux-info.eclass: respect eselect kernel setting in binpkgs

2024-05-14 Thread Andrew Ammerlaan
Commit f51cd5b64c14ddfb83488a12d538c66a4a309376 resets kernel environment
variables when binpkgs are merged, this makes sense since we care about the
system that the binpkg will be installed on, not the system the binpkg was
built on. This introduced Bug 931213 as a side-effect, which will be fixed
in a separate commit to linux-mod-r1.eclass. In preparation of that, we document
and rename the LINUX_INFO_BINARY_RESET to SKIP_KERNEL_BINPKG_ENV_RESET.

However, commit f51cd5b64c14ddfb83488a12d538c66a4a309376 also makes binpkgs
always use the running kernel version. This behaviour is surprising and
confusing. In principle the 'eselect kernel' setting should still be respected
when binpkgs are used.

Bug: https://bugs.gentoo.org/926063
Bug: https://bugs.gentoo.org/931213
Signed-off-by: Andrew Ammerlaan 
---
 eclass/linux-info.eclass | 14 +++---
 1 file changed, 11 insertions(+), 3 deletions(-)

diff --git a/eclass/linux-info.eclass b/eclass/linux-info.eclass
index 864594f607ca..9449a6359d2a 100644
--- a/eclass/linux-info.eclass
+++ b/eclass/linux-info.eclass
@@ -158,6 +158,14 @@ KERNEL_DIR="${KERNEL_DIR:-${ROOT%/}/usr/src/linux}"
 # This is a user flag and should under _no circumstances_ be set in the ebuild.
 : "${SKIP_KERNEL_CHECK:=""}"
 
+# @ECLASS_VARIABLE: SKIP_KERNEL_BINPKG_ENV_RESET
+# @DEFAULT_UNSET
+# @DESCRIPTION:
+# If set, do not reset the kernel environment variables when merging a package
+# as a binpkg.
+# Main use-case is for kernel modules, i.e. linux-mod-r1.eclass.
+# This should be set before running linux-info_pkg_setup
+
 # And to ensure all the weirdness with crosscompile
 inherit toolchain-funcs
 [[ ${EAPI} == 6 ]] && inherit eapi7-ver
@@ -696,12 +704,12 @@ linux-info_get_any_version() {
die "${FUNCNAME}() called on non-Linux system, please fix the 
ebuild"
fi
 
-   if [[ ${MERGE_TYPE} == binary && -z ${LINUX_INFO_BINARY_RESET} ]]; then
+   if [[ ${MERGE_TYPE} == binary && -z ${SKIP_KERNEL_BINPKG_ENV_RESET} ]]; 
then
unset KV_FULL _LINUX_CONFIG_EXISTS_DONE KV_OUT_DIR
-   LINUX_INFO_BINARY_RESET=1
+   SKIP_KERNEL_BINPKG_ENV_RESET=1
fi
 
-   if [[ ${MERGE_TYPE} != binary ]] && ! get_version; then
+   if ! get_version; then
ewarn "Unable to calculate Linux Kernel version for build, 
attempting to use running version"
fi
 
-- 
2.45.0