Re: [gentoo-dev] [PATCH v2] linux-info.eclass: Documentation updates

[Mike Pagano]

Updates based on Ulm's review. Thanks for the review.

Signed-off-by: Mike Pagano <mpag...@gentoo.org>
---
 eclass/linux-info.eclass | 35 ++++++++++++++++++++++++++++++-----
 1 file changed, 30 insertions(+), 5 deletions(-)

diff --git a/eclass/linux-info.eclass b/eclass/linux-info.eclass
index d2003874d..7e130062a 100644
--- a/eclass/linux-info.eclass
+++ b/eclass/linux-info.eclass
@@ -169,10 +169,9 @@ set_arch_to_portage() {
 # Set the env ARCH to match what the package manager expects.
 set_arch_to_pkgmgr() { export ARCH=$(tc-arch); }

-# -------------------
-# qeinfo is a quiet einfo call when EBUILD_PHASE
-# should not have visible output.
+# @FUNCTION: qout
+# @DESCRIPTION:
+# qout <einfo | ewarn | eerror>  is a quiet call when EBUILD_PHASE should not 
have visible output.
 qout() {
        local outputmsg type
        type=${1}
@@ -186,8 +185,21 @@ qout() {
        [ -n "${outputmsg}" ] && ${type} "${outputmsg}"
 }

+# @DESCRIPTION:
+# qeinfo is a quiet einfo call when EBUILD_PHASE should not have visible 
output.
 qeinfo() { qout einfo "${@}" ; }
+
+# @FUNCTION: qewarn
+# @DESCRIPTION:
+# qewarn is a quiet ewarn call when EBUILD_PHASE
+# should not have visible output.
 qewarn() { qout ewarn "${@}" ; }
+
+# @FUNCTION: qeerror
+# @DESCRIPTION:
+# qeerror is a quiet error call when EBUILD_PHASE
+# should not have visible output.
 qeerror() { qout eerror "${@}" ; }

@@ -269,6 +281,10 @@ getfilevar_noexec() {
 # config is available at all.
 _LINUX_CONFIG_EXISTS_DONE=

+# @INTERNAL
+# @DESCRIPTION:
+# Helper funciton which returns an error before the function argument is run 
if no config exists
 linux_config_qa_check() {
        local f="$1"
        if [ -z "${_LINUX_CONFIG_EXISTS_DONE}" ]; then
@@ -438,6 +454,9 @@ kernel_is() {
                "${1:-${KV_MAJOR:-0}}.${2:-${KV_MINOR:-0}}.${3:-${KV_PATCH:-0}}"
 }

+# @INTERNAL
+# @DESCRIPTION:
 # Check if the Makefile is valid for direct parsing.
 # Check status results:
 # - PASS, use 'getfilevar' to extract values
@@ -453,7 +472,10 @@ get_makefile_extract_function() {
        echo "${mkfunc}"
 }

+# @ECLASS_VARIABLE: get_version_warning_done
+# @INTERNAL
+# @DESCRIPTION:
+# Internal variable, so we know to only print the warning once.
 get_version_warning_done=

@@ -870,6 +892,9 @@ check_extra_config() {
        export LINUX_CONFIG_EXISTS_DONE="${old_LINUX_CONFIG_EXISTS_DONE}"
 }

+# @DESCRIPTION:
+# Function to make sure a ZLIB_INFLATE configuration has the required symbols.
 check_zlibinflate() {
        if ! use kernel_linux; then
                die "${FUNCNAME}() called on non-Linux system, please fix the 
ebuild"
--
2.35.1

-# qeinfo "Message" +# @FUNCTION: qeinfo # File Functions +# @FUNCTION: linux_config_qa_check +# @FUNCTION: get_makefile_extract_function -# internal variable, so we know to only print the warning once # @FUNCTION: get_version +# @FUNCTION: check_zlibinflate

OpenPGP_0x92A6DBEC81F2B137.asc
Description: OpenPGP public key

OpenPGP_signature
Description: OpenPGP digital signature