All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] poky: Use SDKPATHINSTALL instead of SDKPATH
@ 2021-07-30 15:50 Richard Purdie
  2021-08-03 12:32 ` [poky] " Michael Opdenacker
  2021-09-15 17:08 ` Denys Dmytriyenko
  0 siblings, 2 replies; 4+ messages in thread
From: Richard Purdie @ 2021-07-30 15:50 UTC (permalink / raw)
  To: poky

SDKPATHINSTALL is the default installation path used for the SDK but is not
the path encoded into every SDK binary as the default path. This change
allows it to contain things like dates without requiring every nativesdk
recipe to rebuild.

Partially fixes [YOCTO #14100]

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---
 meta-poky/conf/distro/poky.conf | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta-poky/conf/distro/poky.conf b/meta-poky/conf/distro/poky.conf
index 8ffd805945f..c7c46cd226f 100644
--- a/meta-poky/conf/distro/poky.conf
+++ b/meta-poky/conf/distro/poky.conf
@@ -23,7 +23,7 @@ PREFERRED_VERSION_linux-yocto ?= "5.10%"
 PREFERRED_VERSION_linux-yocto-rt ?= "5.10%"
 
 SDK_NAME = "${DISTRO}-${TCLIBC}-${SDKMACHINE}-${IMAGE_BASENAME}-${TUNE_PKGARCH}-${MACHINE}"
-SDKPATH = "/opt/${DISTRO}/${SDK_VERSION}"
+SDKPATHINSTALL = "/opt/${DISTRO}/${SDK_VERSION}"
 
 DISTRO_EXTRA_RDEPENDS += "${POKY_DEFAULT_EXTRA_RDEPENDS}"
 DISTRO_EXTRA_RRECOMMENDS += "${POKY_DEFAULT_EXTRA_RRECOMMENDS}"
-- 
2.30.2


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [poky] [PATCH] poky: Use SDKPATHINSTALL instead of SDKPATH
  2021-07-30 15:50 [PATCH] poky: Use SDKPATHINSTALL instead of SDKPATH Richard Purdie
@ 2021-08-03 12:32 ` Michael Opdenacker
  2021-08-04 15:36   ` Richard Purdie
  2021-09-15 17:08 ` Denys Dmytriyenko
  1 sibling, 1 reply; 4+ messages in thread
From: Michael Opdenacker @ 2021-08-03 12:32 UTC (permalink / raw)
  To: Richard Purdie, poky

Hi Richard,

On 7/30/21 5:50 PM, Richard Purdie wrote:
> SDKPATHINSTALL is the default installation path used for the SDK but is not
> the path encoded into every SDK binary as the default path. This change
> allows it to contain things like dates without requiring every nativesdk
> recipe to rebuild.


I realize that SDKPATHINSTALL is not documented while SDKPATH is.
Is it worth adding to the documentation?

Cheers,
Michael.

-- 

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


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [poky] [PATCH] poky: Use SDKPATHINSTALL instead of SDKPATH
  2021-08-03 12:32 ` [poky] " Michael Opdenacker
@ 2021-08-04 15:36   ` Richard Purdie
  0 siblings, 0 replies; 4+ messages in thread
From: Richard Purdie @ 2021-08-04 15:36 UTC (permalink / raw)
  To: Michael Opdenacker, poky

On Tue, 2021-08-03 at 14:32 +0200, Michael Opdenacker wrote:
> Hi Richard,
> 
> On 7/30/21 5:50 PM, Richard Purdie wrote:
> > SDKPATHINSTALL is the default installation path used for the SDK but is not
> > the path encoded into every SDK binary as the default path. This change
> > allows it to contain things like dates without requiring every nativesdk
> > recipe to rebuild.
> 
> 
> I realize that SDKPATHINSTALL is not documented while SDKPATH is.
> Is it worth adding to the documentation?

Yes, and this one needs a note in the migration guide too as we should
recommend distros switch to it like I did with Poky.

Cheers,

Richard


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [poky] [PATCH] poky: Use SDKPATHINSTALL instead of SDKPATH
  2021-07-30 15:50 [PATCH] poky: Use SDKPATHINSTALL instead of SDKPATH Richard Purdie
  2021-08-03 12:32 ` [poky] " Michael Opdenacker
@ 2021-09-15 17:08 ` Denys Dmytriyenko
  1 sibling, 0 replies; 4+ messages in thread
From: Denys Dmytriyenko @ 2021-09-15 17:08 UTC (permalink / raw)
  To: Richard Purdie; +Cc: poky

Can we get this change backported to Dunfell? Since corresponding OE-Core 
change got picked up for Dunfell separately and now the default SDK path is 
different from couple weeks ago...
https://git.openembedded.org/openembedded-core/commit/?h=dunfell&id=33a5714592bf41a26fcd7e8ff595b45538082431


On Fri, Jul 30, 2021 at 04:50:49PM +0100, Richard Purdie wrote:
> SDKPATHINSTALL is the default installation path used for the SDK but is not
> the path encoded into every SDK binary as the default path. This change
> allows it to contain things like dates without requiring every nativesdk
> recipe to rebuild.
> 
> Partially fixes [YOCTO #14100]
> 
> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> ---
>  meta-poky/conf/distro/poky.conf | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/meta-poky/conf/distro/poky.conf b/meta-poky/conf/distro/poky.conf
> index 8ffd805945f..c7c46cd226f 100644
> --- a/meta-poky/conf/distro/poky.conf
> +++ b/meta-poky/conf/distro/poky.conf
> @@ -23,7 +23,7 @@ PREFERRED_VERSION_linux-yocto ?= "5.10%"
>  PREFERRED_VERSION_linux-yocto-rt ?= "5.10%"
>  
>  SDK_NAME = "${DISTRO}-${TCLIBC}-${SDKMACHINE}-${IMAGE_BASENAME}-${TUNE_PKGARCH}-${MACHINE}"
> -SDKPATH = "/opt/${DISTRO}/${SDK_VERSION}"
> +SDKPATHINSTALL = "/opt/${DISTRO}/${SDK_VERSION}"
>  
>  DISTRO_EXTRA_RDEPENDS += "${POKY_DEFAULT_EXTRA_RDEPENDS}"
>  DISTRO_EXTRA_RRECOMMENDS += "${POKY_DEFAULT_EXTRA_RRECOMMENDS}"
> -- 
> 2.30.2
> 

-- 
Regards,
Denys Dmytriyenko <denis@denix.org>
PGP: 0x420902729A92C964 - https://denix.org/0x420902729A92C964
Fingerprint: 25FC E4A5 8A72 2F69 1186  6D76 4209 0272 9A92 C964

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2021-09-15 17:08 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-07-30 15:50 [PATCH] poky: Use SDKPATHINSTALL instead of SDKPATH Richard Purdie
2021-08-03 12:32 ` [poky] " Michael Opdenacker
2021-08-04 15:36   ` Richard Purdie
2021-09-15 17:08 ` Denys Dmytriyenko

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.