Message ID | 20240213125218.5480-1-jbezem.extern@arri.de |
---|---|
State | New |
Headers | show |
Series | variables.rst: SDKPATHINSTALL correct installation default path | expand |
Hi Johan Thanks for pointing this out! On 2/13/24 at 13:52, Bezem, Johan (extern) wrote: > From: "Bezem, Johan" <jbezem.extern@arri.de> > > The SDKPATH variable seems mistakenly identified as the > default path where the SDK will be installed by the generated > installation script, unless option '-d' or a manual input > overrides this default. > > From what I understood from the various sources, the intended > variable is SDKPATHINSTALL. > > I have tried overriding both SDKPATH and SDKPATHINSTALL > (separately), and only overriding the latter had the intended > effect and was successful. > > Signed-off-by: Johan Bezem <jbezem.extern@arri.de> > --- > documentation/ref-manual/variables.rst | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/documentation/ref-manual/variables.rst b/documentation/ref-manual/variables.rst > index 6f7d6ff01..f292b8dad 100644 > --- a/documentation/ref-manual/variables.rst > +++ b/documentation/ref-manual/variables.rst > @@ -7611,7 +7611,7 @@ system and gives an overview of their function and contents. > variable in your distribution configuration file. If you do, the > configuration will not take effect. > > - :term:`SDKPATH` > + :term:`SDKPATHINSTALL` > Defines the path offered to the user for installation of the SDK that > is generated by the OpenEmbedded build system. The path appears as > the default location for installing the SDK when you run the SDK's I believe we should also keep SDKPATH as it's still used in some recipes in Poky. Could you describe it's current usage too? Another thing is that you should also update documentation/overview-manual/concepts.rst which refers to SDKPATH, and the description of SDKTARGETSYSROOT too. You could have caught this by running "make html": .../yocto-docs/documentation/overview-manual/concepts.rst:1378: WARNING: term not in glossary: 'SDKPATH' .../yocto-docs/documentation/ref-manual/variables.rst:7617: WARNING: term not in glossary: 'SDKPATH' Could you send an update? Thanks again Cheers Michael.
Hi Yohan On 2/14/24 at 10:31, Michael Opdenacker wrote: > Hi Johan > > Thanks for pointing this out! > > On 2/13/24 at 13:52, Bezem, Johan (extern) wrote: >> From: "Bezem, Johan" <jbezem.extern@arri.de> >> >> The SDKPATH variable seems mistakenly identified as the >> default path where the SDK will be installed by the generated >> installation script, unless option '-d' or a manual input >> overrides this default. >> >> From what I understood from the various sources, the intended >> variable is SDKPATHINSTALL. >> >> I have tried overriding both SDKPATH and SDKPATHINSTALL >> (separately), and only overriding the latter had the intended >> effect and was successful. >> >> Signed-off-by: Johan Bezem <jbezem.extern@arri.de> >> --- >> documentation/ref-manual/variables.rst | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/documentation/ref-manual/variables.rst >> b/documentation/ref-manual/variables.rst >> index 6f7d6ff01..f292b8dad 100644 >> --- a/documentation/ref-manual/variables.rst >> +++ b/documentation/ref-manual/variables.rst >> @@ -7611,7 +7611,7 @@ system and gives an overview of their function >> and contents. >> variable in your distribution configuration file. If you >> do, the >> configuration will not take effect. >> - :term:`SDKPATH` >> + :term:`SDKPATHINSTALL` >> Defines the path offered to the user for installation of the >> SDK that >> is generated by the OpenEmbedded build system. The path >> appears as >> the default location for installing the SDK when you run the >> SDK's > > > I believe we should also keep SDKPATH as it's still used in some > recipes in Poky. Could you describe it's current usage too? > > Another thing is that you should also update > documentation/overview-manual/concepts.rst which refers to SDKPATH, > and the description of SDKTARGETSYSROOT too. > > You could have caught this by running "make html": > .../yocto-docs/documentation/overview-manual/concepts.rst:1378: > WARNING: term not in glossary: 'SDKPATH' > .../yocto-docs/documentation/ref-manual/variables.rst:7617: WARNING: > term not in glossary: 'SDKPATH' > > Could you send an update? Do you need help doing this? I'll be happy to help if you don't have time right now. Cheers Michael.
diff --git a/documentation/ref-manual/variables.rst b/documentation/ref-manual/variables.rst index 6f7d6ff01..f292b8dad 100644 --- a/documentation/ref-manual/variables.rst +++ b/documentation/ref-manual/variables.rst @@ -7611,7 +7611,7 @@ system and gives an overview of their function and contents. variable in your distribution configuration file. If you do, the configuration will not take effect. - :term:`SDKPATH` + :term:`SDKPATHINSTALL` Defines the path offered to the user for installation of the SDK that is generated by the OpenEmbedded build system. The path appears as the default location for installing the SDK when you run the SDK's