Re: [gentoo-pms] [PMS] [PATCH] Correct the definition of ESYSROOT as EPREFIX isn't always applicable

[Ulrich Mueller]

>>>>> On Sun, 28 Jul 2019, James Le Cuirot wrote:

> --- a/dependencies.tex
> +++ b/dependencies.tex
> @@ -32,13 +32,25 @@
> [...]
> +      If \t{SYSROOT} is:     & \t{\$\{ROOT\}}    & \t{/}           & Other             \\

I wonder if the third column should't say "Empty" or "Blank"?
Path variables in EAPI 7 don't have a trailing slash.

> --- a/ebuild-env-vars.tex
> +++ b/ebuild-env-vars.tex
> @@ -141,9 +141,10 @@ variable.
>  \t{ESYSROOT} &
>      Ditto &
>      No &
> -    Contains the concatenation of the paths in the \t{SYSROOT} and \t{EPREFIX} variables,
> -    for convenience. See also the \t{EPREFIX} variable. Only for EAPIs listed
> -    in table~\ref{tab:offset-env-vars-table} as supporting \t{ESYSROOT}. \\
> +    Contains the concatenation of the \t{SYSROOT} path and applicable prefix value. The prefix value
> +    is \t{EPREFIX}, \t{BROOT}, or blank if \t{SYSROOT} is equal to \t{ROOT}, \t{/}, or something
> +    else respectively. Only for EAPIs listed in table~\ref{tab:offset-env-vars-table} as supporting
> +    \t{ESYSROOT}. \\

I'd says something like this, in order to avoid defining the same thing
in two places:

    Contains the concatenation of the \t{SYSROOT} path and applicable prefix value, as determined
    by table~\ref{tab:depend-prefix}. Only for EAPIs listed in table~\ref{tab:offset-env-vars-table}
    as supporting \t{ESYSROOT}. \\

Ulrich