• [gentoo-dev] [PATCH 6/6] linux-info.eclass: Rework get_version and fix

    From James Le Cuirot@21:1/5 to All on Fri May 23 19:40:01 2025
    get_version used to detect the output directory under /lib/modules if KBUILD_OUTPUT wasn't explicitly set, but this broke when bug #662772 was
    fixed. The code is still there but effectively does nothing as
    OUTPUT_DIR is already used to set KV_OUT_DIR above.

    Bug: https://bugs.gentoo.org/662772
    Signed-off-by: James Le Cuirot <chewi@gentoo.org>
    ---
    eclass/linux-info.eclass | 59 ++++++++++++++++++++--------------------
    1 file changed, 29 insertions(+), 30 deletions(-)

    diff --git a/eclass/linux-info.eclass b/eclass/linux-info.eclass
    index 067c94ccdc1f8..a2b38d604b214 100644
    --- a/eclass/linux-info.eclass
    +++ b/eclass/linux-info.eclass
    @@ -485,8 +485,6 @@ get_version() {
    die "${FUNCNAME}() called on non-Linux system, please fix the ebuild"
    fi

    - local tmplocal
    -
    [[ -n ${SKIP_KERNEL_CHECK} ]] && return

    # No need to execute this twice assuming KV_FULL is populated.
    @@ -541,13 +539,6 @@ get_version() {
    return 1
    fi

    - # OK so now we know our sources directory, but they might be using
    - # KBUILD_OUTPUT, and we need this for .config and localversions-*
    - # so we better find it, eh?
    - #
    - # Do we pass KBUILD_OUTPUT on the CLI?
    - local OUTPUT_DIR=${KBUILD_OUTPUT}
    -
    # And contrary to existing functions, I feel we shouldn't trust the
    # directory name to find version information as this seems insane.
    # So we p