[PATCH] variables.rst: SDKPATHINSTALL correct installation default path

[PATCH] variables.rst: SDKPATHINSTALL correct installation default path

[Johan Bezem]

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
-- 
2.30.2

Re: [docs] [PATCH] variables.rst: SDKPATHINSTALL correct installation default path

[Michael Opdenacker]

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.

-- 
Michael Opdenacker, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

Re: [docs] [PATCH] variables.rst: SDKPATHINSTALL correct installation default path

[Michael Opdenacker]

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.

-- 
Michael Opdenacker, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

Re: [docs] [PATCH] variables.rst: SDKPATHINSTALL correct installation default path

[Michael Opdenacker]

Hi Johan

On 2/26/24 at 17:15, 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.
>
> The intended variable is SDKPATHINSTALL. SDKPATH indicates
> where the SDK is being composed and built.
>
> The definitions have been added/updated.
>
> Signed-off-by: Johan Bezem <jbezem.extern@arri.de>
> ---
>   documentation/overview-manual/concepts.rst | 2 +-
>   documentation/ref-manual/variables.rst     | 6 +++++-
>   2 files changed, 6 insertions(+), 2 deletions(-)


Thanks for the new version!
Reviewed-by: Michael Opdenacker <michael.opdenacker@bootlin.com>
Merged into master-next.
Cheers
Michael.

-- 

Michael Opdenacker, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

Re: [docs] [PATCH] variables.rst: SDKPATHINSTALL correct installation default path

[Bezem, Johan (extern)]

Hi Michael,

I sent an updated patch where I corrected and tested according to your suggestions.
The mail system does not seem to link the two, but here is the link to the second patch, the first one can be ignored:
https://patchwork.yoctoproject.org/project/docs/patch/20240226161506.48020-1-jbezem.extern@arri.de/

Regards,

Johan

-----Ursprüngliche Nachricht-----
Von: docs@lists.yoctoproject.org <docs@lists.yoctoproject.org> Im Auftrag von Michael Opdenacker via lists.yoctoproject.org
Gesendet: Freitag, 23. Februar 2024 11:20
An: Bezem, Johan (extern) <JBezem.extern@arri.de>
Cc: docs@lists.yoctoproject.org
Betreff: Re: [docs] [PATCH] variables.rst: SDKPATHINSTALL correct installation default path

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.

--
Michael Opdenacker, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

Re: [docs] [PATCH] variables.rst: SDKPATHINSTALL correct installation default path

[Bezem, Johan (extern)]

Hi Michael,

As I'm not all that familiar (yet) with Yocto and BitBake, I need to carefully
research the variable SDKPATH, especially since the documentation is apparently incorrect.
And it's not my highest priority. So if you have the time, any help is welcome.

Rgds,

Johan

-----Ursprüngliche Nachricht-----
Von: docs@lists.yoctoproject.org <docs@lists.yoctoproject.org> Im Auftrag von Michael Opdenacker via lists.yoctoproject.org
Gesendet: Freitag, 23. Februar 2024 11:20
An: Bezem, Johan (extern) <JBezem.extern@arri.de>
Cc: docs@lists.yoctoproject.org
Betreff: Re: [docs] [PATCH] variables.rst: SDKPATHINSTALL correct installation default path

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.

--
Michael Opdenacker, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com