ref-manual: extend/add HOSTTOOLS, HOSTTOOLS_DIR defns

Submitted by Robert P. J. Day on March 24, 2021, 4:43 p.m. | Patch ID: 179428

Details

Message ID 1191825-a9e8-ea67-69e8-468edc9e6095@crashcourse.ca
State New
Headers show

Commit Message

Robert P. J. Day March 24, 2021, 4:43 p.m.
Tighten up the definition for HOSTTOOLS, and add an entry for
HOSTTOOLS_DIR.

Signed-off-by: Robert P. J. Day <rpjday@crashcourse.ca>

---

Patch hide | download patch | download mbox

diff --git a/documentation/ref-manual/variables.rst b/documentation/ref-manual/variables.rst
index 0310429bd..80b73f70c 100644
--- a/documentation/ref-manual/variables.rst
+++ b/documentation/ref-manual/variables.rst
@@ -2793,16 +2793,22 @@  system and gives an overview of their function and contents.
          Linux, the value might be "mipsel-linux".

    :term:`HOSTTOOLS`
-      A space-separated list (filter) of tools on the build host that
-      should be allowed to be called from within build tasks. Using this
-      filter helps reduce the possibility of host contamination. If a tool
-      specified in the value of ``HOSTTOOLS`` is not found on the build
-      host, the OpenEmbedded build system produces an error and the build
-      is not started.
+      A space-separated list (filter) of tools that must exist on the
+      build host and which will be called from within build tasks.
+      If a tool specified in the value of ``HOSTTOOLS`` is not found
+      on the build host, the OpenEmbedded build system produces an error
+      and the build is not started.

-      For additional information, see
+      For additional information, see the related
       :term:`HOSTTOOLS_NONFATAL`.

+   :term:`HOSTTOOLS_DIR`
+      The directory which contains symlinks to all of the host tools
+      specified by ``HOSTTOOLS``, as well as any of the available
+      host tools specified by ``HOSTTOOLS_NONFATAL``.
+      The default value for this directory is typically
+      ``${TMPDIR}/hosttools``.
+
    :term:`HOSTTOOLS_NONFATAL`
       A space-separated list (filter) of tools on the build host that
       should be allowed to be called from within build tasks. Using this

Comments

Quentin Schulz March 24, 2021, 5:09 p.m.
Hi Robert,

Nitpicking, I'll leave the review of the essence of the patch to people
who know more about it.

On Wed, Mar 24, 2021 at 12:43:52PM -0400, Robert P. J. Day wrote:
> Tighten up the definition for HOSTTOOLS, and add an entry for
> HOSTTOOLS_DIR.
> 
> Signed-off-by: Robert P. J. Day <rpjday@crashcourse.ca>
> 
> ---
> 
> diff --git a/documentation/ref-manual/variables.rst b/documentation/ref-manual/variables.rst
> index 0310429bd..80b73f70c 100644
> --- a/documentation/ref-manual/variables.rst
> +++ b/documentation/ref-manual/variables.rst
> @@ -2793,16 +2793,22 @@ system and gives an overview of their function and contents.
>           Linux, the value might be "mipsel-linux".
> 
>     :term:`HOSTTOOLS`
> -      A space-separated list (filter) of tools on the build host that
> -      should be allowed to be called from within build tasks. Using this
> -      filter helps reduce the possibility of host contamination. If a tool
> -      specified in the value of ``HOSTTOOLS`` is not found on the build
> -      host, the OpenEmbedded build system produces an error and the build
> -      is not started.
> +      A space-separated list (filter) of tools that must exist on the
> +      build host and which will be called from within build tasks.
> +      If a tool specified in the value of ``HOSTTOOLS`` is not found
> +      on the build host, the OpenEmbedded build system produces an error
> +      and the build is not started.
> 
> -      For additional information, see
> +      For additional information, see the related
>        :term:`HOSTTOOLS_NONFATAL`.
> 
> +   :term:`HOSTTOOLS_DIR`
> +      The directory which contains symlinks to all of the host tools
> +      specified by ``HOSTTOOLS``, as well as any of the available

:term:`HOSTTOOLS`

> +      host tools specified by ``HOSTTOOLS_NONFATAL``.

:term:`HOSTTOOLS_NONFATAL`

> +      The default value for this directory is typically
> +      ``${TMPDIR}/hosttools``.

I don't remember if :term: can be surrounded in ``, but if it can :term:
would make sense too here (it probably can't but better know it for sure
:) )

Quentin
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#956): https://lists.yoctoproject.org/g/docs/message/956
Mute This Topic: https://lists.yoctoproject.org/mt/81581518/3617530
Group Owner: docs+owner@lists.yoctoproject.org
Unsubscribe: https://lists.yoctoproject.org/g/docs/unsub [oe-patchwork@oe-patch.openembedded.org]
-=-=-=-=-=-=-=-=-=-=-=-
Robert P. J. Day March 24, 2021, 5:16 p.m.
On Wed, 24 Mar 2021, Quentin Schulz wrote:

> Hi Robert,
>
> Nitpicking, I'll leave the review of the essence of the patch to people
> who know more about it.
>
> On Wed, Mar 24, 2021 at 12:43:52PM -0400, Robert P. J. Day wrote:
> > Tighten up the definition for HOSTTOOLS, and add an entry for
> > HOSTTOOLS_DIR.
> >
> > Signed-off-by: Robert P. J. Day <rpjday@crashcourse.ca>
> >
> > ---
> >
> > diff --git a/documentation/ref-manual/variables.rst b/documentation/ref-manual/variables.rst
> > index 0310429bd..80b73f70c 100644
> > --- a/documentation/ref-manual/variables.rst
> > +++ b/documentation/ref-manual/variables.rst
> > @@ -2793,16 +2793,22 @@ system and gives an overview of their function and contents.
> >           Linux, the value might be "mipsel-linux".
> >
> >     :term:`HOSTTOOLS`
> > -      A space-separated list (filter) of tools on the build host that
> > -      should be allowed to be called from within build tasks. Using this
> > -      filter helps reduce the possibility of host contamination. If a tool
> > -      specified in the value of ``HOSTTOOLS`` is not found on the build
> > -      host, the OpenEmbedded build system produces an error and the build
> > -      is not started.
> > +      A space-separated list (filter) of tools that must exist on the
> > +      build host and which will be called from within build tasks.
> > +      If a tool specified in the value of ``HOSTTOOLS`` is not found
> > +      on the build host, the OpenEmbedded build system produces an error
> > +      and the build is not started.
> >
> > -      For additional information, see
> > +      For additional information, see the related
> >        :term:`HOSTTOOLS_NONFATAL`.
> >
> > +   :term:`HOSTTOOLS_DIR`
> > +      The directory which contains symlinks to all of the host tools
> > +      specified by ``HOSTTOOLS``, as well as any of the available
>
> :term:`HOSTTOOLS`
>
> > +      host tools specified by ``HOSTTOOLS_NONFATAL``.
>
> :term:`HOSTTOOLS_NONFATAL`
>
> > +      The default value for this directory is typically
> > +      ``${TMPDIR}/hosttools``.
>
> I don't remember if :term: can be surrounded in ``, but if it can :term:
> would make sense too here (it probably can't but better know it for sure
> :) )

  i was actually grappling with that, i think it was only because they
were all next to each other that i got lazy.

rday
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#957): https://lists.yoctoproject.org/g/docs/message/957
Mute This Topic: https://lists.yoctoproject.org/mt/81581518/3617530
Group Owner: docs+owner@lists.yoctoproject.org
Unsubscribe: https://lists.yoctoproject.org/g/docs/unsub [oe-patchwork@oe-patch.openembedded.org]
-=-=-=-=-=-=-=-=-=-=-=-