Re: [gentoo-dev] [PATCH eclass-manpages] Allow @INCLUDES_EPREFIX for path functions&variable

[Benda Xu]

Michał Górny <mgorny@g.o> writes:



> Add a new @INCLUDES_EPREFIX tag that indicates that a particular
> function (= getter) or variable is a path that includes ${EPREFIX}.
> This will be used in pkgcheck to detect accidental variable and getter
> combinations that result in double prefix.
>
> Signed-off-by: Michał Górny <mgorny@g.o>
> ---
>  eclass-to-manpage.awk | 9 +++++++++
>  1 file changed, 9 insertions(+)
>
> diff --git a/eclass-to-manpage.awk b/eclass-to-manpage.awk
> index 44708d4..f8620c8 100755
> --- a/eclass-to-manpage.awk
> +++ b/eclass-to-manpage.awk
> @@ -33,6 +33,7 @@
>  # @MAINTAINER:
>  # <optional; list of contacts, one per line>
>  # [@INTERNAL]
> +# [@INCLUDES_EPREFIX] (the function outputs path that includes ${EPREFIX})
>  # @DESCRIPTION:
>  # <required if no @RETURN; blurb about this function>
>
> @@ -42,6 +43,7 @@
>  # [@INTERNAL] (internal eclass use variable)
>  # [@DEFAULT_UNSET]
>  # [@REQUIRED]
> +# [@INCLUDES_EPREFIX] (the variable is a path that includes ${EPREFIX})
>  # @DESCRIPTION:
>  # <required; blurb about this variable>
>  # foo="<default value>"
> @@ -54,6 +56,7 @@
>  # [@INTERNAL] (internal eclass use variable)
>  # [@DEFAULT_UNSET]
>  # [@REQUIRED]
> +# [@INCLUDES_EPREFIX] (the variable is a path that includes ${EPREFIX})
>  # @DESCRIPTION:
>  # <required; blurb about this variable>
>  # foo="<default value>"
> @@ -252,6 +255,10 @@ function handle_function() {
>  		internal = 1
>  		getline
>  	}
> +	if ($2 == "@INCLUDES_EPREFIX") {
> +		includes_eprefix = 1
> +		getline
> +	}
>  	if ($2 == "@DESCRIPTION:")
>  		desc = eat_paragraph()
>
> @@ -314,6 +321,8 @@ function _handle_variable() {
>  			user_variable = 1
>  		else if ($2 == "@OUTPUT_VARIABLE")
>  			output_variable = 1
> +		else if ($2 == "@INCLUDES_EPREFIX")
> +			includes_eprefix = 1
>  		else
>  			opts = 0
>  	}


That looks like a good idea.



Yours,

Benda