[meta-arago][oe-layersetup PATCH 1/6] Remove unused sample-file bblayers-zeus-browser.conf.sample

[meta-arago][oe-layersetup PATCH 1/6] Remove unused sample-file bblayers-zeus-browser.conf.sample

[Andrew Davis]

Signed-off-by: Andrew Davis <afd@ti.com>
---
 sample-files/bblayers-zeus-browser.conf.sample | 12 ------------
 1 file changed, 12 deletions(-)
 delete mode 100644 sample-files/bblayers-zeus-browser.conf.sample

diff --git a/sample-files/bblayers-zeus-browser.conf.sample b/sample-files/bblayers-zeus-browser.conf.sample
deleted file mode 100644
index bb83fac..0000000
--- a/sample-files/bblayers-zeus-browser.conf.sample
+++ /dev/null
@@ -1,12 +0,0 @@
-# This template file was created by taking the oe-core/meta/conf/bblayers.conf
-# file and removing the BBLAYERS section at the end.
-
-# LAYER_CONF_VERSION is increased each time build/conf/bblayers.conf
-# changes incompatibly
-LCONF_VERSION = "6"
-
-BBPATH = "${TOPDIR}"
-BBFILES ?= ""
-
-OVERRIDES = "forcevariable"
-LAYERSERIES_COMPAT_browser-layer_forcevariable = "zeus"
-- 
2.17.1

[meta-arago][oe-layersetup PATCH 2/6] Add layers for split meta-ti master branch

[Andrew Davis]

The latest master meta-ti split into two layers, add new layers here.

Signed-off-by: Andrew Davis <afd@ti.com>
---
 configs/arago-master-config.txt      | 2 +-
 configs/distroless-master-config.txt | 2 +-
 configs/poky-master-config.txt       | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/configs/arago-master-config.txt b/configs/arago-master-config.txt
index bd0ccec..532a145 100644
--- a/configs/arago-master-config.txt
+++ b/configs/arago-master-config.txt
@@ -7,7 +7,7 @@ meta-arago,git://git.yoctoproject.org/meta-arago,master,HEAD,layers=meta-arago-d
 meta-qt5,https://github.com/meta-qt5/meta-qt5.git,master,HEAD,layers=
 meta-virtualization,git://git.yoctoproject.org/meta-virtualization,master,HEAD,layers=
 meta-openembedded,git://git.openembedded.org/meta-openembedded,master,HEAD,layers=meta-networking:meta-python:meta-oe:meta-gnome:meta-filesystems
-meta-ti,git://git.yoctoproject.org/meta-ti,master,HEAD,layers=
+meta-ti,git://git.yoctoproject.org/meta-ti,master,HEAD,layers=meta-ti-extras:meta-ti-bsp
 meta-arm,git://git.yoctoproject.org/meta-arm,master,HEAD,layers=meta-arm:meta-arm-toolchain
 oe-core,git://git.openembedded.org/openembedded-core,master,HEAD,layers=meta
 OECORELAYERCONF=./sample-files/bblayers.conf.sample
diff --git a/configs/distroless-master-config.txt b/configs/distroless-master-config.txt
index 8211bd5..5c55459 100644
--- a/configs/distroless-master-config.txt
+++ b/configs/distroless-master-config.txt
@@ -1,7 +1,7 @@
 # This file takes repo entries in the format
 # repo name,repo uri,repo branch,repo commit[,layers=layer1:layer2...:layern]
 bitbake,git://git.openembedded.org/bitbake,master,HEAD
-meta-ti,git://git.yoctoproject.org/meta-ti,master,HEAD,layers=
+meta-ti,git://git.yoctoproject.org/meta-ti,master,HEAD,layers=meta-ti-extras:meta-ti-bsp
 meta-arm,git://git.yoctoproject.org/meta-arm,master,HEAD,layers=meta-arm:meta-arm-toolchain
 oe-core,git://git.openembedded.org/openembedded-core,master,HEAD,layers=meta
 OECORELAYERCONF=./sample-files/bblayers.conf.sample
diff --git a/configs/poky-master-config.txt b/configs/poky-master-config.txt
index f213411..b395ff8 100644
--- a/configs/poky-master-config.txt
+++ b/configs/poky-master-config.txt
@@ -2,7 +2,7 @@
 # repo name,repo uri,repo branch,repo commit[,layers=layer1:layer2...:layern]
 bitbake,git://git.openembedded.org/bitbake,master,HEAD
 poky,git://git.yoctoproject.org/poky,master,HEAD,layers=meta:meta-poky:meta-yocto-bsp
-meta-ti,git://git.yoctoproject.org/meta-ti,master,HEAD,layers=
+meta-ti,git://git.yoctoproject.org/meta-ti,master,HEAD,layers=meta-ti-extras:meta-ti-bsp
 meta-arm,git://git.yoctoproject.org/meta-arm,master,HEAD,layers=meta-arm:meta-arm-toolchain
 OECORELAYERCONF=./sample-files/bblayers.conf.sample
 OECORELOCALCONF=./sample-files/local-poky-v2.conf.sample
-- 
2.17.1

[meta-arago][oe-layersetup PATCH 3/6] Remove image-prelink from sample USER_CLASSES

[Andrew Davis]

Prelinking hasn't worked on ARM in a while so this class was
removed in the latests core master. Remove it here.

Signed-off-by: Andrew Davis <afd@ti.com>
---
 sample-files/local-arago64-v2.conf.sample | 4 +---
 sample-files/local-poky-v2.conf.sample    | 4 +---
 2 files changed, 2 insertions(+), 6 deletions(-)

diff --git a/sample-files/local-arago64-v2.conf.sample b/sample-files/local-arago64-v2.conf.sample
index 9549e8a..13585cb 100644
--- a/sample-files/local-arago64-v2.conf.sample
+++ b/sample-files/local-arago64-v2.conf.sample
@@ -112,11 +112,9 @@ EXTRA_IMAGE_FEATURES = "debug-tweaks"
 # enable extra features. Some available options which can be included in this variable 
 # are:
 #   - 'buildstats' collect build statistics
-#   - 'image-prelink' in order to prelink the filesystem image
 #   - 'image-swab' to perform host system intrusion detection
-# NOTE: if listing mklibs & prelink both, then make sure mklibs is before prelink
 # NOTE: mklibs also needs to be explicitly enabled for a given image, see local.conf.extended
-USER_CLASSES ?= "buildstats image-prelink"
+USER_CLASSES ?= "buildstats"
 
 # By default disable interactive patch resolution (tasks will just fail instead):
 PATCHRESOLVE = "noop"
diff --git a/sample-files/local-poky-v2.conf.sample b/sample-files/local-poky-v2.conf.sample
index 5cbe6ef..a958055 100644
--- a/sample-files/local-poky-v2.conf.sample
+++ b/sample-files/local-poky-v2.conf.sample
@@ -109,11 +109,9 @@ EXTRA_IMAGE_FEATURES = "debug-tweaks"
 # enable extra features. Some available options which can be included in this variable 
 # are:
 #   - 'buildstats' collect build statistics
-#   - 'image-prelink' in order to prelink the filesystem image
 #   - 'image-swab' to perform host system intrusion detection
-# NOTE: if listing mklibs & prelink both, then make sure mklibs is before prelink
 # NOTE: mklibs also needs to be explicitly enabled for a given image, see local.conf.extended
-USER_CLASSES ?= "buildstats image-prelink"
+USER_CLASSES ?= "buildstats"
 
 # By default disable interactive patch resolution (tasks will just fail instead):
 PATCHRESOLVE = "noop"
-- 
2.17.1

[meta-arago][oe-layersetup PATCH 4/6] Rename BB_ENV_EXTRAWHITE to BB_ENV_PASSTHROUGH_ADDITIONS

[Andrew Davis]

This variable was renamed upstream, rename it in our setenv file.

Signed-off-by: Andrew Davis <afd@ti.com>
---
 oe-layertool-setup.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/oe-layertool-setup.sh b/oe-layertool-setup.sh
index f171f94..72416b0 100755
--- a/oe-layertool-setup.sh
+++ b/oe-layertool-setup.sh
@@ -853,7 +853,7 @@ unset BITBAKEDIR
 unset SCRIPTS
 export PATH
 export BUILDDIR=${builddir}
-export BB_ENV_EXTRAWHITE="MACHINE DISTRO TCMODE TCLIBC http_proxy ftp_proxy https_proxy all_proxy ALL_PROXY no_proxy SSH_AGENT_PID SSH_AUTH_SOCK BB_SRCREV_POLICY SDKMACHINE BB_NUMBER_THREADS PARALLEL_MAKE GIT_PROXY_COMMAND GIT_PROXY_IGNORE SOCKS5_PASSWD SOCKS5_USER OEBASE META_SDK_PATH TOOLCHAIN_TYPE TOOLCHAIN_BRAND TOOLCHAIN_BASE TOOLCHAIN_PATH TOOLCHAIN_PATH_ARMV5 TOOLCHAIN_PATH_ARMV7 TOOLCHAIN_PATH_ARMV8 EXTRA_TISDK_FILES TISDK_VERSION ARAGO_BRAND ARAGO_RT_ENABLE ARAGO_SYSTEST_ENABLE ARAGO_KERNEL_SUFFIX TI_SECURE_DEV_PKG_CAT TI_SECURE_DEV_PKG_AUTO TI_SECURE_DEV_PKG_K3 ARAGO_SYSVINIT SYSFW_FILE"
+export BB_ENV_PASSTHROUGH_ADDITIONS="MACHINE DISTRO TCMODE TCLIBC http_proxy ftp_proxy https_proxy all_proxy ALL_PROXY no_proxy SSH_AGENT_PID SSH_AUTH_SOCK BB_SRCREV_POLICY SDKMACHINE BB_NUMBER_THREADS PARALLEL_MAKE GIT_PROXY_COMMAND GIT_PROXY_IGNORE SOCKS5_PASSWD SOCKS5_USER OEBASE META_SDK_PATH TOOLCHAIN_TYPE TOOLCHAIN_BRAND TOOLCHAIN_BASE TOOLCHAIN_PATH TOOLCHAIN_PATH_ARMV5 TOOLCHAIN_PATH_ARMV7 TOOLCHAIN_PATH_ARMV8 EXTRA_TISDK_FILES TISDK_VERSION ARAGO_BRAND ARAGO_RT_ENABLE ARAGO_SYSTEST_ENABLE ARAGO_KERNEL_SUFFIX TI_SECURE_DEV_PKG_CAT TI_SECURE_DEV_PKG_AUTO TI_SECURE_DEV_PKG_K3 ARAGO_SYSVINIT SYSFW_FILE"
 EOM
 }
 
-- 
2.17.1

[meta-arago][oe-layersetup PATCH 5/6] Build Poky in a directory named poky-tmp vs arago-tmp

[Andrew Davis]

This also allows us to keep version of both should we need.

Signed-off-by: Andrew Davis <afd@ti.com>
---
 sample-files/local-poky-v2.conf.sample | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sample-files/local-poky-v2.conf.sample b/sample-files/local-poky-v2.conf.sample
index a958055..4bd0563 100644
--- a/sample-files/local-poky-v2.conf.sample
+++ b/sample-files/local-poky-v2.conf.sample
@@ -48,7 +48,7 @@ SSTATE_DIR = "${OEBASE}/build/sstate-cache"
 # where BitBake should place its temporary files and output. Keep in mind that
 # this includes the extraction and compilation of many applications and the toolchain
 # which can use Gigabytes of hard disk space.
-TMPDIR = "${OEBASE}/build/arago-tmp"
+TMPDIR = "${OEBASE}/build/poky-tmp"
 
 #
 # Machine Selection
-- 
2.17.1

[meta-arago][oe-layersetup PATCH 6/6] Build Poky using meta-yocto instead of combined poky repo

[Andrew Davis]

The Poky repo contains meta-yocto plus extra bits needed for an
all-in-one build, but since we already pull in tools with this
script like bitbake and oe-core, lets use those and save some
space/bandwidth.

Signed-off-by: Andrew Davis <afd@ti.com>
---
 configs/poky-master-config.txt | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/configs/poky-master-config.txt b/configs/poky-master-config.txt
index b395ff8..a541501 100644
--- a/configs/poky-master-config.txt
+++ b/configs/poky-master-config.txt
@@ -1,8 +1,10 @@
 # This file takes repo entries in the format
 # repo name,repo uri,repo branch,repo commit[,layers=layer1:layer2...:layern]
+
 bitbake,git://git.openembedded.org/bitbake,master,HEAD
-poky,git://git.yoctoproject.org/poky,master,HEAD,layers=meta:meta-poky:meta-yocto-bsp
+meta-yocto,git://git.yoctoproject.org/meta-yocto,master,HEAD,layers=meta-poky:meta-yocto-bsp
 meta-ti,git://git.yoctoproject.org/meta-ti,master,HEAD,layers=meta-ti-extras:meta-ti-bsp
 meta-arm,git://git.yoctoproject.org/meta-arm,master,HEAD,layers=meta-arm:meta-arm-toolchain
+oe-core,git://git.openembedded.org/openembedded-core,master,HEAD,layers=meta
 OECORELAYERCONF=./sample-files/bblayers.conf.sample
 OECORELOCALCONF=./sample-files/local-poky-v2.conf.sample
-- 
2.17.1

Re: [meta-arago][oe-layersetup PATCH 4/6] Rename BB_ENV_EXTRAWHITE to BB_ENV_PASSTHROUGH_ADDITIONS

[Denys Dmytriyenko]

On Tue, Apr 26, 2022 at 03:02:49PM -0500, Andrew F. Davis via lists.yoctoproject.org wrote:
> This variable was renamed upstream, rename it in our setenv file.

This will break dunfell...


> Signed-off-by: Andrew Davis <afd@ti.com>
> ---
>  oe-layertool-setup.sh | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/oe-layertool-setup.sh b/oe-layertool-setup.sh
> index f171f94..72416b0 100755
> --- a/oe-layertool-setup.sh
> +++ b/oe-layertool-setup.sh
> @@ -853,7 +853,7 @@ unset BITBAKEDIR
>  unset SCRIPTS
>  export PATH
>  export BUILDDIR=${builddir}
> -export BB_ENV_EXTRAWHITE="MACHINE DISTRO TCMODE TCLIBC http_proxy ftp_proxy https_proxy all_proxy ALL_PROXY no_proxy SSH_AGENT_PID SSH_AUTH_SOCK BB_SRCREV_POLICY SDKMACHINE BB_NUMBER_THREADS PARALLEL_MAKE GIT_PROXY_COMMAND GIT_PROXY_IGNORE SOCKS5_PASSWD SOCKS5_USER OEBASE META_SDK_PATH TOOLCHAIN_TYPE TOOLCHAIN_BRAND TOOLCHAIN_BASE TOOLCHAIN_PATH TOOLCHAIN_PATH_ARMV5 TOOLCHAIN_PATH_ARMV7 TOOLCHAIN_PATH_ARMV8 EXTRA_TISDK_FILES TISDK_VERSION ARAGO_BRAND ARAGO_RT_ENABLE ARAGO_SYSTEST_ENABLE ARAGO_KERNEL_SUFFIX TI_SECURE_DEV_PKG_CAT TI_SECURE_DEV_PKG_AUTO TI_SECURE_DEV_PKG_K3 ARAGO_SYSVINIT SYSFW_FILE"
> +export BB_ENV_PASSTHROUGH_ADDITIONS="MACHINE DISTRO TCMODE TCLIBC http_proxy ftp_proxy https_proxy all_proxy ALL_PROXY no_proxy SSH_AGENT_PID SSH_AUTH_SOCK BB_SRCREV_POLICY SDKMACHINE BB_NUMBER_THREADS PARALLEL_MAKE GIT_PROXY_COMMAND GIT_PROXY_IGNORE SOCKS5_PASSWD SOCKS5_USER OEBASE META_SDK_PATH TOOLCHAIN_TYPE TOOLCHAIN_BRAND TOOLCHAIN_BASE TOOLCHAIN_PATH TOOLCHAIN_PATH_ARMV5 TOOLCHAIN_PATH_ARMV7 TOOLCHAIN_PATH_ARMV8 EXTRA_TISDK_FILES TISDK_VERSION ARAGO_BRAND ARAGO_RT_ENABLE ARAGO_SYSTEST_ENABLE ARAGO_KERNEL_SUFFIX TI_SECURE_DEV_PKG_CAT TI_SECURE_DEV_PKG_AUTO TI_SECURE_DEV_PKG_K3 ARAGO_SYSVINIT SYSFW_FILE"
>  EOM
>  }

Re: [meta-arago][oe-layersetup PATCH 2/6] Add layers for split meta-ti master branch

[Denys Dmytriyenko]

On Tue, Apr 26, 2022 at 03:02:47PM -0500, Andrew F. Davis via lists.yoctoproject.org wrote:
> The latest master meta-ti split into two layers, add new layers here.

Can you swap the order of layers in meta-ti?
Other repos usually have the main repo listed first.


> Signed-off-by: Andrew Davis <afd@ti.com>
> ---
>  configs/arago-master-config.txt      | 2 +-
>  configs/distroless-master-config.txt | 2 +-
>  configs/poky-master-config.txt       | 2 +-
>  3 files changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/configs/arago-master-config.txt b/configs/arago-master-config.txt
> index bd0ccec..532a145 100644
> --- a/configs/arago-master-config.txt
> +++ b/configs/arago-master-config.txt
> @@ -7,7 +7,7 @@ meta-arago,git://git.yoctoproject.org/meta-arago,master,HEAD,layers=meta-arago-d
>  meta-qt5,https://github.com/meta-qt5/meta-qt5.git,master,HEAD,layers=
>  meta-virtualization,git://git.yoctoproject.org/meta-virtualization,master,HEAD,layers=
>  meta-openembedded,git://git.openembedded.org/meta-openembedded,master,HEAD,layers=meta-networking:meta-python:meta-oe:meta-gnome:meta-filesystems
> -meta-ti,git://git.yoctoproject.org/meta-ti,master,HEAD,layers=
> +meta-ti,git://git.yoctoproject.org/meta-ti,master,HEAD,layers=meta-ti-extras:meta-ti-bsp
>  meta-arm,git://git.yoctoproject.org/meta-arm,master,HEAD,layers=meta-arm:meta-arm-toolchain
>  oe-core,git://git.openembedded.org/openembedded-core,master,HEAD,layers=meta
>  OECORELAYERCONF=./sample-files/bblayers.conf.sample
> diff --git a/configs/distroless-master-config.txt b/configs/distroless-master-config.txt
> index 8211bd5..5c55459 100644
> --- a/configs/distroless-master-config.txt
> +++ b/configs/distroless-master-config.txt
> @@ -1,7 +1,7 @@
>  # This file takes repo entries in the format
>  # repo name,repo uri,repo branch,repo commit[,layers=layer1:layer2...:layern]
>  bitbake,git://git.openembedded.org/bitbake,master,HEAD
> -meta-ti,git://git.yoctoproject.org/meta-ti,master,HEAD,layers=
> +meta-ti,git://git.yoctoproject.org/meta-ti,master,HEAD,layers=meta-ti-extras:meta-ti-bsp
>  meta-arm,git://git.yoctoproject.org/meta-arm,master,HEAD,layers=meta-arm:meta-arm-toolchain
>  oe-core,git://git.openembedded.org/openembedded-core,master,HEAD,layers=meta
>  OECORELAYERCONF=./sample-files/bblayers.conf.sample
> diff --git a/configs/poky-master-config.txt b/configs/poky-master-config.txt
> index f213411..b395ff8 100644
> --- a/configs/poky-master-config.txt
> +++ b/configs/poky-master-config.txt
> @@ -2,7 +2,7 @@
>  # repo name,repo uri,repo branch,repo commit[,layers=layer1:layer2...:layern]
>  bitbake,git://git.openembedded.org/bitbake,master,HEAD
>  poky,git://git.yoctoproject.org/poky,master,HEAD,layers=meta:meta-poky:meta-yocto-bsp
> -meta-ti,git://git.yoctoproject.org/meta-ti,master,HEAD,layers=
> +meta-ti,git://git.yoctoproject.org/meta-ti,master,HEAD,layers=meta-ti-extras:meta-ti-bsp
>  meta-arm,git://git.yoctoproject.org/meta-arm,master,HEAD,layers=meta-arm:meta-arm-toolchain
>  OECORELAYERCONF=./sample-files/bblayers.conf.sample
>  OECORELOCALCONF=./sample-files/local-poky-v2.conf.sample
> -- 
> 2.17.1

Re: [meta-arago][oe-layersetup PATCH 3/6] Remove image-prelink from sample USER_CLASSES

[Denys Dmytriyenko]

On Tue, Apr 26, 2022 at 03:02:48PM -0500, Andrew F. Davis via lists.yoctoproject.org wrote:
> Prelinking hasn't worked on ARM in a while so this class was
> removed in the latests core master. Remove it here.

This file is used for both dunfell and master. Maybe this change calls for v3 
of the corresponding sample files?


> Signed-off-by: Andrew Davis <afd@ti.com>
> ---
>  sample-files/local-arago64-v2.conf.sample | 4 +---
>  sample-files/local-poky-v2.conf.sample    | 4 +---
>  2 files changed, 2 insertions(+), 6 deletions(-)
> 
> diff --git a/sample-files/local-arago64-v2.conf.sample b/sample-files/local-arago64-v2.conf.sample
> index 9549e8a..13585cb 100644
> --- a/sample-files/local-arago64-v2.conf.sample
> +++ b/sample-files/local-arago64-v2.conf.sample
> @@ -112,11 +112,9 @@ EXTRA_IMAGE_FEATURES = "debug-tweaks"
>  # enable extra features. Some available options which can be included in this variable 
>  # are:
>  #   - 'buildstats' collect build statistics
> -#   - 'image-prelink' in order to prelink the filesystem image
>  #   - 'image-swab' to perform host system intrusion detection
> -# NOTE: if listing mklibs & prelink both, then make sure mklibs is before prelink
>  # NOTE: mklibs also needs to be explicitly enabled for a given image, see local.conf.extended
> -USER_CLASSES ?= "buildstats image-prelink"
> +USER_CLASSES ?= "buildstats"
>  
>  # By default disable interactive patch resolution (tasks will just fail instead):
>  PATCHRESOLVE = "noop"
> diff --git a/sample-files/local-poky-v2.conf.sample b/sample-files/local-poky-v2.conf.sample
> index 5cbe6ef..a958055 100644
> --- a/sample-files/local-poky-v2.conf.sample
> +++ b/sample-files/local-poky-v2.conf.sample
> @@ -109,11 +109,9 @@ EXTRA_IMAGE_FEATURES = "debug-tweaks"
>  # enable extra features. Some available options which can be included in this variable 
>  # are:
>  #   - 'buildstats' collect build statistics
> -#   - 'image-prelink' in order to prelink the filesystem image
>  #   - 'image-swab' to perform host system intrusion detection
> -# NOTE: if listing mklibs & prelink both, then make sure mklibs is before prelink
>  # NOTE: mklibs also needs to be explicitly enabled for a given image, see local.conf.extended
> -USER_CLASSES ?= "buildstats image-prelink"
> +USER_CLASSES ?= "buildstats"
>  
>  # By default disable interactive patch resolution (tasks will just fail instead):
>  PATCHRESOLVE = "noop"
> -- 
> 2.17.1

Re: [meta-arago][oe-layersetup PATCH 5/6] Build Poky in a directory named poky-tmp vs arago-tmp

[Denys Dmytriyenko]

On Tue, Apr 26, 2022 at 03:02:50PM -0500, Andrew F. Davis via lists.yoctoproject.org wrote:
> This also allows us to keep version of both should we need.

I'm fine with the change, although it's really a NOOP, as TMPDIR gets replaced 
by tisdk-build-scripts, Jenkins jobs and even a distro config... But for 
consistency, it's fine.


> Signed-off-by: Andrew Davis <afd@ti.com>
> ---
>  sample-files/local-poky-v2.conf.sample | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/sample-files/local-poky-v2.conf.sample b/sample-files/local-poky-v2.conf.sample
> index a958055..4bd0563 100644
> --- a/sample-files/local-poky-v2.conf.sample
> +++ b/sample-files/local-poky-v2.conf.sample
> @@ -48,7 +48,7 @@ SSTATE_DIR = "${OEBASE}/build/sstate-cache"
>  # where BitBake should place its temporary files and output. Keep in mind that
>  # this includes the extraction and compilation of many applications and the toolchain
>  # which can use Gigabytes of hard disk space.
> -TMPDIR = "${OEBASE}/build/arago-tmp"
> +TMPDIR = "${OEBASE}/build/poky-tmp"
>  
>  #
>  # Machine Selection
> -- 
> 2.17.1

Re: [meta-arago][oe-layersetup PATCH 2/6] Add layers for split meta-ti master branch

[Andrew Davis]

On 4/26/22 4:09 PM, Denys Dmytriyenko wrote:
> On Tue, Apr 26, 2022 at 03:02:47PM -0500, Andrew F. Davis via lists.yoctoproject.org wrote:
>> The latest master meta-ti split into two layers, add new layers here.
> 
> Can you swap the order of layers in meta-ti?
> Other repos usually have the main repo listed first.
> 


That changes the order they go into to bblayers, don't we want
meta-ti-bsp to be the bottom layer?

Andrew


> 
>> Signed-off-by: Andrew Davis <afd@ti.com>
>> ---
>>   configs/arago-master-config.txt      | 2 +-
>>   configs/distroless-master-config.txt | 2 +-
>>   configs/poky-master-config.txt       | 2 +-
>>   3 files changed, 3 insertions(+), 3 deletions(-)
>>
>> diff --git a/configs/arago-master-config.txt b/configs/arago-master-config.txt
>> index bd0ccec..532a145 100644
>> --- a/configs/arago-master-config.txt
>> +++ b/configs/arago-master-config.txt
>> @@ -7,7 +7,7 @@ meta-arago,git://git.yoctoproject.org/meta-arago,master,HEAD,layers=meta-arago-d
>>   meta-qt5,https://github.com/meta-qt5/meta-qt5.git,master,HEAD,layers=
>>   meta-virtualization,git://git.yoctoproject.org/meta-virtualization,master,HEAD,layers=
>>   meta-openembedded,git://git.openembedded.org/meta-openembedded,master,HEAD,layers=meta-networking:meta-python:meta-oe:meta-gnome:meta-filesystems
>> -meta-ti,git://git.yoctoproject.org/meta-ti,master,HEAD,layers=
>> +meta-ti,git://git.yoctoproject.org/meta-ti,master,HEAD,layers=meta-ti-extras:meta-ti-bsp
>>   meta-arm,git://git.yoctoproject.org/meta-arm,master,HEAD,layers=meta-arm:meta-arm-toolchain
>>   oe-core,git://git.openembedded.org/openembedded-core,master,HEAD,layers=meta
>>   OECORELAYERCONF=./sample-files/bblayers.conf.sample
>> diff --git a/configs/distroless-master-config.txt b/configs/distroless-master-config.txt
>> index 8211bd5..5c55459 100644
>> --- a/configs/distroless-master-config.txt
>> +++ b/configs/distroless-master-config.txt
>> @@ -1,7 +1,7 @@
>>   # This file takes repo entries in the format
>>   # repo name,repo uri,repo branch,repo commit[,layers=layer1:layer2...:layern]
>>   bitbake,git://git.openembedded.org/bitbake,master,HEAD
>> -meta-ti,git://git.yoctoproject.org/meta-ti,master,HEAD,layers=
>> +meta-ti,git://git.yoctoproject.org/meta-ti,master,HEAD,layers=meta-ti-extras:meta-ti-bsp
>>   meta-arm,git://git.yoctoproject.org/meta-arm,master,HEAD,layers=meta-arm:meta-arm-toolchain
>>   oe-core,git://git.openembedded.org/openembedded-core,master,HEAD,layers=meta
>>   OECORELAYERCONF=./sample-files/bblayers.conf.sample
>> diff --git a/configs/poky-master-config.txt b/configs/poky-master-config.txt
>> index f213411..b395ff8 100644
>> --- a/configs/poky-master-config.txt
>> +++ b/configs/poky-master-config.txt
>> @@ -2,7 +2,7 @@
>>   # repo name,repo uri,repo branch,repo commit[,layers=layer1:layer2...:layern]
>>   bitbake,git://git.openembedded.org/bitbake,master,HEAD
>>   poky,git://git.yoctoproject.org/poky,master,HEAD,layers=meta:meta-poky:meta-yocto-bsp
>> -meta-ti,git://git.yoctoproject.org/meta-ti,master,HEAD,layers=
>> +meta-ti,git://git.yoctoproject.org/meta-ti,master,HEAD,layers=meta-ti-extras:meta-ti-bsp
>>   meta-arm,git://git.yoctoproject.org/meta-arm,master,HEAD,layers=meta-arm:meta-arm-toolchain
>>   OECORELAYERCONF=./sample-files/bblayers.conf.sample
>>   OECORELOCALCONF=./sample-files/local-poky-v2.conf.sample
>> -- 
>> 2.17.1

Re: [meta-arago][oe-layersetup PATCH 6/6] Build Poky using meta-yocto instead of combined poky repo

[Denys Dmytriyenko]

On Tue, Apr 26, 2022 at 03:02:51PM -0500, Andrew F. Davis via lists.yoctoproject.org wrote:
> The Poky repo contains meta-yocto plus extra bits needed for an
> all-in-one build, but since we already pull in tools with this
> script like bitbake and oe-core, lets use those and save some
> space/bandwidth.

There are other nuances of "flattened" Poky repo (commits are different, 
plus some other minor bits and pieces). The only overhead is to clone Bitbake 
twice (not OE-Core), which is neglectable and allows building the true Poky, 
instead of OE-Core + meta-yocto/meta-poky... :)


> Signed-off-by: Andrew Davis <afd@ti.com>
> ---
>  configs/poky-master-config.txt | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/configs/poky-master-config.txt b/configs/poky-master-config.txt
> index b395ff8..a541501 100644
> --- a/configs/poky-master-config.txt
> +++ b/configs/poky-master-config.txt
> @@ -1,8 +1,10 @@
>  # This file takes repo entries in the format
>  # repo name,repo uri,repo branch,repo commit[,layers=layer1:layer2...:layern]
> +
>  bitbake,git://git.openembedded.org/bitbake,master,HEAD
> -poky,git://git.yoctoproject.org/poky,master,HEAD,layers=meta:meta-poky:meta-yocto-bsp
> +meta-yocto,git://git.yoctoproject.org/meta-yocto,master,HEAD,layers=meta-poky:meta-yocto-bsp
>  meta-ti,git://git.yoctoproject.org/meta-ti,master,HEAD,layers=meta-ti-extras:meta-ti-bsp
>  meta-arm,git://git.yoctoproject.org/meta-arm,master,HEAD,layers=meta-arm:meta-arm-toolchain
> +oe-core,git://git.openembedded.org/openembedded-core,master,HEAD,layers=meta
>  OECORELAYERCONF=./sample-files/bblayers.conf.sample
>  OECORELOCALCONF=./sample-files/local-poky-v2.conf.sample
> -- 
> 2.17.1

Re: [meta-arago][oe-layersetup PATCH 3/6] Remove image-prelink from sample USER_CLASSES

[Andrew Davis]

On 4/26/22 4:11 PM, Denys Dmytriyenko wrote:
> On Tue, Apr 26, 2022 at 03:02:48PM -0500, Andrew F. Davis via lists.yoctoproject.org wrote:
>> Prelinking hasn't worked on ARM in a while so this class was
>> removed in the latests core master. Remove it here.
> 
> This file is used for both dunfell and master. Maybe this change calls for v3
> of the corresponding sample files?
> 


I only see these v2 files referenced in master configs.

Andrew


> 
>> Signed-off-by: Andrew Davis <afd@ti.com>
>> ---
>>   sample-files/local-arago64-v2.conf.sample | 4 +---
>>   sample-files/local-poky-v2.conf.sample    | 4 +---
>>   2 files changed, 2 insertions(+), 6 deletions(-)
>>
>> diff --git a/sample-files/local-arago64-v2.conf.sample b/sample-files/local-arago64-v2.conf.sample
>> index 9549e8a..13585cb 100644
>> --- a/sample-files/local-arago64-v2.conf.sample
>> +++ b/sample-files/local-arago64-v2.conf.sample
>> @@ -112,11 +112,9 @@ EXTRA_IMAGE_FEATURES = "debug-tweaks"
>>   # enable extra features. Some available options which can be included in this variable
>>   # are:
>>   #   - 'buildstats' collect build statistics
>> -#   - 'image-prelink' in order to prelink the filesystem image
>>   #   - 'image-swab' to perform host system intrusion detection
>> -# NOTE: if listing mklibs & prelink both, then make sure mklibs is before prelink
>>   # NOTE: mklibs also needs to be explicitly enabled for a given image, see local.conf.extended
>> -USER_CLASSES ?= "buildstats image-prelink"
>> +USER_CLASSES ?= "buildstats"
>>   
>>   # By default disable interactive patch resolution (tasks will just fail instead):
>>   PATCHRESOLVE = "noop"
>> diff --git a/sample-files/local-poky-v2.conf.sample b/sample-files/local-poky-v2.conf.sample
>> index 5cbe6ef..a958055 100644
>> --- a/sample-files/local-poky-v2.conf.sample
>> +++ b/sample-files/local-poky-v2.conf.sample
>> @@ -109,11 +109,9 @@ EXTRA_IMAGE_FEATURES = "debug-tweaks"
>>   # enable extra features. Some available options which can be included in this variable
>>   # are:
>>   #   - 'buildstats' collect build statistics
>> -#   - 'image-prelink' in order to prelink the filesystem image
>>   #   - 'image-swab' to perform host system intrusion detection
>> -# NOTE: if listing mklibs & prelink both, then make sure mklibs is before prelink
>>   # NOTE: mklibs also needs to be explicitly enabled for a given image, see local.conf.extended
>> -USER_CLASSES ?= "buildstats image-prelink"
>> +USER_CLASSES ?= "buildstats"
>>   
>>   # By default disable interactive patch resolution (tasks will just fail instead):
>>   PATCHRESOLVE = "noop"
>> -- 
>> 2.17.1

Re: [meta-arago][oe-layersetup PATCH 4/6] Rename BB_ENV_EXTRAWHITE to BB_ENV_PASSTHROUGH_ADDITIONS

[Andrew Davis]

On 4/26/22 4:08 PM, Denys Dmytriyenko wrote:
> On Tue, Apr 26, 2022 at 03:02:49PM -0500, Andrew F. Davis via lists.yoctoproject.org wrote:
>> This variable was renamed upstream, rename it in our setenv file.
> 
> This will break dunfell...
> 


And not doing it breaks master...

There is a check for BB_ENV_EXTRAWHITE, having it set will halt
build (think the check is in bitbake but haven't looked).


> 
>> Signed-off-by: Andrew Davis <afd@ti.com>
>> ---
>>   oe-layertool-setup.sh | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/oe-layertool-setup.sh b/oe-layertool-setup.sh
>> index f171f94..72416b0 100755
>> --- a/oe-layertool-setup.sh
>> +++ b/oe-layertool-setup.sh
>> @@ -853,7 +853,7 @@ unset BITBAKEDIR
>>   unset SCRIPTS
>>   export PATH
>>   export BUILDDIR=${builddir}
>> -export BB_ENV_EXTRAWHITE="MACHINE DISTRO TCMODE TCLIBC http_proxy ftp_proxy https_proxy all_proxy ALL_PROXY no_proxy SSH_AGENT_PID SSH_AUTH_SOCK BB_SRCREV_POLICY SDKMACHINE BB_NUMBER_THREADS PARALLEL_MAKE GIT_PROXY_COMMAND GIT_PROXY_IGNORE SOCKS5_PASSWD SOCKS5_USER OEBASE META_SDK_PATH TOOLCHAIN_TYPE TOOLCHAIN_BRAND TOOLCHAIN_BASE TOOLCHAIN_PATH TOOLCHAIN_PATH_ARMV5 TOOLCHAIN_PATH_ARMV7 TOOLCHAIN_PATH_ARMV8 EXTRA_TISDK_FILES TISDK_VERSION ARAGO_BRAND ARAGO_RT_ENABLE ARAGO_SYSTEST_ENABLE ARAGO_KERNEL_SUFFIX TI_SECURE_DEV_PKG_CAT TI_SECURE_DEV_PKG_AUTO TI_SECURE_DEV_PKG_K3 ARAGO_SYSVINIT SYSFW_FILE"
>> +export BB_ENV_PASSTHROUGH_ADDITIONS="MACHINE DISTRO TCMODE TCLIBC http_proxy ftp_proxy https_proxy all_proxy ALL_PROXY no_proxy SSH_AGENT_PID SSH_AUTH_SOCK BB_SRCREV_POLICY SDKMACHINE BB_NUMBER_THREADS PARALLEL_MAKE GIT_PROXY_COMMAND GIT_PROXY_IGNORE SOCKS5_PASSWD SOCKS5_USER OEBASE META_SDK_PATH TOOLCHAIN_TYPE TOOLCHAIN_BRAND TOOLCHAIN_BASE TOOLCHAIN_PATH TOOLCHAIN_PATH_ARMV5 TOOLCHAIN_PATH_ARMV7 TOOLCHAIN_PATH_ARMV8 EXTRA_TISDK_FILES TISDK_VERSION ARAGO_BRAND ARAGO_RT_ENABLE ARAGO_SYSTEST_ENABLE ARAGO_KERNEL_SUFFIX TI_SECURE_DEV_PKG_CAT TI_SECURE_DEV_PKG_AUTO TI_SECURE_DEV_PKG_K3 ARAGO_SYSVINIT SYSFW_FILE"
>>   EOM
>>   }

Re: [meta-arago][oe-layersetup PATCH 5/6] Build Poky in a directory named poky-tmp vs arago-tmp

[Andrew Davis]

On 4/26/22 4:14 PM, Denys Dmytriyenko wrote:
> On Tue, Apr 26, 2022 at 03:02:50PM -0500, Andrew F. Davis via lists.yoctoproject.org wrote:
>> This also allows us to keep version of both should we need.
> 
> I'm fine with the change, although it's really a NOOP, as TMPDIR gets replaced
> by tisdk-build-scripts, Jenkins jobs and even a distro config... But for
> consistency, it's fine.
> 


I'm not a build script nor a Jenkins job :) so this *does* change the behavior
for me building on my local desktop. (Poky doesn't change this in distro config).

Andrew


> 
>> Signed-off-by: Andrew Davis <afd@ti.com>
>> ---
>>   sample-files/local-poky-v2.conf.sample | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/sample-files/local-poky-v2.conf.sample b/sample-files/local-poky-v2.conf.sample
>> index a958055..4bd0563 100644
>> --- a/sample-files/local-poky-v2.conf.sample
>> +++ b/sample-files/local-poky-v2.conf.sample
>> @@ -48,7 +48,7 @@ SSTATE_DIR = "${OEBASE}/build/sstate-cache"
>>   # where BitBake should place its temporary files and output. Keep in mind that
>>   # this includes the extraction and compilation of many applications and the toolchain
>>   # which can use Gigabytes of hard disk space.
>> -TMPDIR = "${OEBASE}/build/arago-tmp"
>> +TMPDIR = "${OEBASE}/build/poky-tmp"
>>   
>>   #
>>   # Machine Selection
>> -- 
>> 2.17.1

Re: [meta-arago][oe-layersetup PATCH 2/6] Add layers for split meta-ti master branch

[Denys Dmytriyenko]

On Tue, Apr 26, 2022 at 04:18:18PM -0500, Andrew Davis wrote:
> On 4/26/22 4:09 PM, Denys Dmytriyenko wrote:
> >On Tue, Apr 26, 2022 at 03:02:47PM -0500, Andrew F. Davis via lists.yoctoproject.org wrote:
> >>The latest master meta-ti split into two layers, add new layers here.
> >
> >Can you swap the order of layers in meta-ti?
> >Other repos usually have the main repo listed first.
> 
> 
> That changes the order they go into to bblayers, don't we want
> meta-ti-bsp to be the bottom layer?

Right, the order of entries in bblayers.conf affects the way "shadowed" config 
files are parsed/visible.

So, if you eventually want a same-named config file from meta-ti-extras to 
override the same config file from meta-ti-bsp, then this order is correct. 
But I'd be very cautious overriding each other config files from layers in the 
same repo...

Anyway, I retract my comment:

Reviewed-by: Denys Dmytriyenko <denys@konsulko.com>

-- 
Denys


> >
> >>Signed-off-by: Andrew Davis <afd@ti.com>
> >>---
> >>  configs/arago-master-config.txt      | 2 +-
> >>  configs/distroless-master-config.txt | 2 +-
> >>  configs/poky-master-config.txt       | 2 +-
> >>  3 files changed, 3 insertions(+), 3 deletions(-)
> >>
> >>diff --git a/configs/arago-master-config.txt b/configs/arago-master-config.txt
> >>index bd0ccec..532a145 100644
> >>--- a/configs/arago-master-config.txt
> >>+++ b/configs/arago-master-config.txt
> >>@@ -7,7 +7,7 @@ meta-arago,git://git.yoctoproject.org/meta-arago,master,HEAD,layers=meta-arago-d
> >>  meta-qt5,https://github.com/meta-qt5/meta-qt5.git,master,HEAD,layers=
> >>  meta-virtualization,git://git.yoctoproject.org/meta-virtualization,master,HEAD,layers=
> >>  meta-openembedded,git://git.openembedded.org/meta-openembedded,master,HEAD,layers=meta-networking:meta-python:meta-oe:meta-gnome:meta-filesystems
> >>-meta-ti,git://git.yoctoproject.org/meta-ti,master,HEAD,layers=
> >>+meta-ti,git://git.yoctoproject.org/meta-ti,master,HEAD,layers=meta-ti-extras:meta-ti-bsp
> >>  meta-arm,git://git.yoctoproject.org/meta-arm,master,HEAD,layers=meta-arm:meta-arm-toolchain
> >>  oe-core,git://git.openembedded.org/openembedded-core,master,HEAD,layers=meta
> >>  OECORELAYERCONF=./sample-files/bblayers.conf.sample
> >>diff --git a/configs/distroless-master-config.txt b/configs/distroless-master-config.txt
> >>index 8211bd5..5c55459 100644
> >>--- a/configs/distroless-master-config.txt
> >>+++ b/configs/distroless-master-config.txt
> >>@@ -1,7 +1,7 @@
> >>  # This file takes repo entries in the format
> >>  # repo name,repo uri,repo branch,repo commit[,layers=layer1:layer2...:layern]
> >>  bitbake,git://git.openembedded.org/bitbake,master,HEAD
> >>-meta-ti,git://git.yoctoproject.org/meta-ti,master,HEAD,layers=
> >>+meta-ti,git://git.yoctoproject.org/meta-ti,master,HEAD,layers=meta-ti-extras:meta-ti-bsp
> >>  meta-arm,git://git.yoctoproject.org/meta-arm,master,HEAD,layers=meta-arm:meta-arm-toolchain
> >>  oe-core,git://git.openembedded.org/openembedded-core,master,HEAD,layers=meta
> >>  OECORELAYERCONF=./sample-files/bblayers.conf.sample
> >>diff --git a/configs/poky-master-config.txt b/configs/poky-master-config.txt
> >>index f213411..b395ff8 100644
> >>--- a/configs/poky-master-config.txt
> >>+++ b/configs/poky-master-config.txt
> >>@@ -2,7 +2,7 @@
> >>  # repo name,repo uri,repo branch,repo commit[,layers=layer1:layer2...:layern]
> >>  bitbake,git://git.openembedded.org/bitbake,master,HEAD
> >>  poky,git://git.yoctoproject.org/poky,master,HEAD,layers=meta:meta-poky:meta-yocto-bsp
> >>-meta-ti,git://git.yoctoproject.org/meta-ti,master,HEAD,layers=
> >>+meta-ti,git://git.yoctoproject.org/meta-ti,master,HEAD,layers=meta-ti-extras:meta-ti-bsp
> >>  meta-arm,git://git.yoctoproject.org/meta-arm,master,HEAD,layers=meta-arm:meta-arm-toolchain
> >>  OECORELAYERCONF=./sample-files/bblayers.conf.sample
> >>  OECORELOCALCONF=./sample-files/local-poky-v2.conf.sample
> >>-- 
> >>2.17.1
> 

Re: [meta-arago][oe-layersetup PATCH 6/6] Build Poky using meta-yocto instead of combined poky repo

[Andrew Davis]

On 4/26/22 4:18 PM, Denys Dmytriyenko wrote:
> On Tue, Apr 26, 2022 at 03:02:51PM -0500, Andrew F. Davis via lists.yoctoproject.org wrote:
>> The Poky repo contains meta-yocto plus extra bits needed for an
>> all-in-one build, but since we already pull in tools with this
>> script like bitbake and oe-core, lets use those and save some
>> space/bandwidth.
> 
> There are other nuances of "flattened" Poky repo (commits are different,
> plus some other minor bits and pieces). The only overhead is to clone Bitbake
> twice (not OE-Core), which is neglectable and allows building the true Poky,
> instead of OE-Core + meta-yocto/meta-poky... :)
> 


Could you let me know what the other "minor bits and pieces" we are concerned
about losing here? It seems the "flattened" Poky repo is an automatic merge of
Bitbake + OE-Core + meta-yocto/meta-poky + some random scripts we don't care about
since we do the same thing here in oe-layersetup.

If we want "true" Poky, then we just clone flattened Poky and don't need
oe-layersetup at all here.

Andrew


> 
>> Signed-off-by: Andrew Davis <afd@ti.com>
>> ---
>>   configs/poky-master-config.txt | 4 +++-
>>   1 file changed, 3 insertions(+), 1 deletion(-)
>>
>> diff --git a/configs/poky-master-config.txt b/configs/poky-master-config.txt
>> index b395ff8..a541501 100644
>> --- a/configs/poky-master-config.txt
>> +++ b/configs/poky-master-config.txt
>> @@ -1,8 +1,10 @@
>>   # This file takes repo entries in the format
>>   # repo name,repo uri,repo branch,repo commit[,layers=layer1:layer2...:layern]
>> +
>>   bitbake,git://git.openembedded.org/bitbake,master,HEAD
>> -poky,git://git.yoctoproject.org/poky,master,HEAD,layers=meta:meta-poky:meta-yocto-bsp
>> +meta-yocto,git://git.yoctoproject.org/meta-yocto,master,HEAD,layers=meta-poky:meta-yocto-bsp
>>   meta-ti,git://git.yoctoproject.org/meta-ti,master,HEAD,layers=meta-ti-extras:meta-ti-bsp
>>   meta-arm,git://git.yoctoproject.org/meta-arm,master,HEAD,layers=meta-arm:meta-arm-toolchain
>> +oe-core,git://git.openembedded.org/openembedded-core,master,HEAD,layers=meta
>>   OECORELAYERCONF=./sample-files/bblayers.conf.sample
>>   OECORELOCALCONF=./sample-files/local-poky-v2.conf.sample
>> -- 
>> 2.17.1

Re: [meta-arago][oe-layersetup PATCH 3/6] Remove image-prelink from sample USER_CLASSES

[Denys Dmytriyenko]

On Tue, Apr 26, 2022 at 04:20:14PM -0500, Andrew Davis via lists.yoctoproject.org wrote:
> On 4/26/22 4:11 PM, Denys Dmytriyenko wrote:
> >On Tue, Apr 26, 2022 at 03:02:48PM -0500, Andrew F. Davis via lists.yoctoproject.org wrote:
> >>Prelinking hasn't worked on ARM in a while so this class was
> >>removed in the latests core master. Remove it here.
> >
> >This file is used for both dunfell and master. Maybe this change calls for v3
> >of the corresponding sample files?
> >
> 
> 
> I only see these v2 files referenced in master configs.

Ah, v2 wasn't that old and was still not used in dunfell (it felt like 
eternity for some reason):

https://lists.yoctoproject.org/g/meta-arago/message/13289

Ok, no need for v3:

Reviewed-by: Denys Dmytriyenko <denys@konsulko.com>


> >>Signed-off-by: Andrew Davis <afd@ti.com>
> >>---
> >>  sample-files/local-arago64-v2.conf.sample | 4 +---
> >>  sample-files/local-poky-v2.conf.sample    | 4 +---
> >>  2 files changed, 2 insertions(+), 6 deletions(-)
> >>
> >>diff --git a/sample-files/local-arago64-v2.conf.sample b/sample-files/local-arago64-v2.conf.sample
> >>index 9549e8a..13585cb 100644
> >>--- a/sample-files/local-arago64-v2.conf.sample
> >>+++ b/sample-files/local-arago64-v2.conf.sample
> >>@@ -112,11 +112,9 @@ EXTRA_IMAGE_FEATURES = "debug-tweaks"
> >>  # enable extra features. Some available options which can be included in this variable
> >>  # are:
> >>  #   - 'buildstats' collect build statistics
> >>-#   - 'image-prelink' in order to prelink the filesystem image
> >>  #   - 'image-swab' to perform host system intrusion detection
> >>-# NOTE: if listing mklibs & prelink both, then make sure mklibs is before prelink
> >>  # NOTE: mklibs also needs to be explicitly enabled for a given image, see local.conf.extended
> >>-USER_CLASSES ?= "buildstats image-prelink"
> >>+USER_CLASSES ?= "buildstats"
> >>  # By default disable interactive patch resolution (tasks will just fail instead):
> >>  PATCHRESOLVE = "noop"
> >>diff --git a/sample-files/local-poky-v2.conf.sample b/sample-files/local-poky-v2.conf.sample
> >>index 5cbe6ef..a958055 100644
> >>--- a/sample-files/local-poky-v2.conf.sample
> >>+++ b/sample-files/local-poky-v2.conf.sample
> >>@@ -109,11 +109,9 @@ EXTRA_IMAGE_FEATURES = "debug-tweaks"
> >>  # enable extra features. Some available options which can be included in this variable
> >>  # are:
> >>  #   - 'buildstats' collect build statistics
> >>-#   - 'image-prelink' in order to prelink the filesystem image
> >>  #   - 'image-swab' to perform host system intrusion detection
> >>-# NOTE: if listing mklibs & prelink both, then make sure mklibs is before prelink
> >>  # NOTE: mklibs also needs to be explicitly enabled for a given image, see local.conf.extended
> >>-USER_CLASSES ?= "buildstats image-prelink"
> >>+USER_CLASSES ?= "buildstats"
> >>  # By default disable interactive patch resolution (tasks will just fail instead):
> >>  PATCHRESOLVE = "noop"
> >>-- 
> >>2.17.1

Re: [meta-arago][oe-layersetup PATCH 1/6] Remove unused sample-file bblayers-zeus-browser.conf.sample

[Denys Dmytriyenko]

On Tue, Apr 26, 2022 at 03:02:46PM -0500, Andrew F. Davis via lists.yoctoproject.org wrote:
> Signed-off-by: Andrew Davis <afd@ti.com>

Yeah, all zeus configs were removed, no need for this snippet either:

Reviewed-by: Denys Dmytriyenko <denys@konsulko.com>


> ---
>  sample-files/bblayers-zeus-browser.conf.sample | 12 ------------
>  1 file changed, 12 deletions(-)
>  delete mode 100644 sample-files/bblayers-zeus-browser.conf.sample
> 
> diff --git a/sample-files/bblayers-zeus-browser.conf.sample b/sample-files/bblayers-zeus-browser.conf.sample
> deleted file mode 100644
> index bb83fac..0000000
> --- a/sample-files/bblayers-zeus-browser.conf.sample
> +++ /dev/null
> @@ -1,12 +0,0 @@
> -# This template file was created by taking the oe-core/meta/conf/bblayers.conf
> -# file and removing the BBLAYERS section at the end.
> -
> -# LAYER_CONF_VERSION is increased each time build/conf/bblayers.conf
> -# changes incompatibly
> -LCONF_VERSION = "6"
> -
> -BBPATH = "${TOPDIR}"
> -BBFILES ?= ""
> -
> -OVERRIDES = "forcevariable"
> -LAYERSERIES_COMPAT_browser-layer_forcevariable = "zeus"
> -- 
> 2.17.1

Re: [meta-arago][oe-layersetup PATCH 6/6] Build Poky using meta-yocto instead of combined poky repo

[Denys Dmytriyenko]

On Tue, Apr 26, 2022 at 04:29:53PM -0500, Andrew Davis wrote:
> On 4/26/22 4:18 PM, Denys Dmytriyenko wrote:
> >On Tue, Apr 26, 2022 at 03:02:51PM -0500, Andrew F. Davis via lists.yoctoproject.org wrote:
> >>The Poky repo contains meta-yocto plus extra bits needed for an
> >>all-in-one build, but since we already pull in tools with this
> >>script like bitbake and oe-core, lets use those and save some
> >>space/bandwidth.
> >
> >There are other nuances of "flattened" Poky repo (commits are different,
> >plus some other minor bits and pieces). The only overhead is to clone Bitbake
> >twice (not OE-Core), which is neglectable and allows building the true Poky,
> >instead of OE-Core + meta-yocto/meta-poky... :)
> >
> 
> 
> Could you let me know what the other "minor bits and pieces" we are concerned
> about losing here? It seems the "flattened" Poky repo is an automatic merge of
> Bitbake + OE-Core + meta-yocto/meta-poky + some random scripts we don't care about
> since we do the same thing here in oe-layersetup.
> 
> If we want "true" Poky, then we just clone flattened Poky and don't need
> oe-layersetup at all here.

Yeah, the biggest difference is all the different commit IDs between all 
those layers due to running combo-layer tool, which I don't believe runs 
automatically anyway. And, sure, combo-layer was one of the ways to do layer 
setup, vs. oe-layersetup tool. But since I wanted to do Poky builds with 
meta-ti on the same build farm with Arago, which uses tisdk-build-scripts 
and oe-layersetup, adding this config back then was the simplest approach. 
And if this is no longer used/needed, you can definitely change or remove 
it...


> >>Signed-off-by: Andrew Davis <afd@ti.com>
> >>---
> >>  configs/poky-master-config.txt | 4 +++-
> >>  1 file changed, 3 insertions(+), 1 deletion(-)
> >>
> >>diff --git a/configs/poky-master-config.txt b/configs/poky-master-config.txt
> >>index b395ff8..a541501 100644
> >>--- a/configs/poky-master-config.txt
> >>+++ b/configs/poky-master-config.txt
> >>@@ -1,8 +1,10 @@
> >>  # This file takes repo entries in the format
> >>  # repo name,repo uri,repo branch,repo commit[,layers=layer1:layer2...:layern]
> >>+
> >>  bitbake,git://git.openembedded.org/bitbake,master,HEAD
> >>-poky,git://git.yoctoproject.org/poky,master,HEAD,layers=meta:meta-poky:meta-yocto-bsp
> >>+meta-yocto,git://git.yoctoproject.org/meta-yocto,master,HEAD,layers=meta-poky:meta-yocto-bsp
> >>  meta-ti,git://git.yoctoproject.org/meta-ti,master,HEAD,layers=meta-ti-extras:meta-ti-bsp
> >>  meta-arm,git://git.yoctoproject.org/meta-arm,master,HEAD,layers=meta-arm:meta-arm-toolchain
> >>+oe-core,git://git.openembedded.org/openembedded-core,master,HEAD,layers=meta
> >>  OECORELAYERCONF=./sample-files/bblayers.conf.sample
> >>  OECORELOCALCONF=./sample-files/local-poky-v2.conf.sample
> >>-- 
> >>2.17.1

Re: [meta-arago][oe-layersetup PATCH 4/6] Rename BB_ENV_EXTRAWHITE to BB_ENV_PASSTHROUGH_ADDITIONS

[Ryan Eatmon]

On 4/26/2022 16:23, Andrew Davis wrote:
> On 4/26/22 4:08 PM, Denys Dmytriyenko wrote:
>> On Tue, Apr 26, 2022 at 03:02:49PM -0500, Andrew F. Davis via 
>> lists.yoctoproject.org wrote:
>>> This variable was renamed upstream, rename it in our setenv file.
>>
>> This will break dunfell...
>>
> 
> 
> And not doing it breaks master...
> 
> There is a check for BB_ENV_EXTRAWHITE, having it set will halt
> build (think the check is in bitbake but haven't looked).
> 

This one patch seems to be the only hold out.  We clearly cannot break 
our current production.  Do you want to resubmit the series without this 
patch and we can keep looking in this issue separately?



>>
>>> Signed-off-by: Andrew Davis <afd@ti.com>
>>> ---
>>>   oe-layertool-setup.sh | 2 +-
>>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/oe-layertool-setup.sh b/oe-layertool-setup.sh
>>> index f171f94..72416b0 100755
>>> --- a/oe-layertool-setup.sh
>>> +++ b/oe-layertool-setup.sh
>>> @@ -853,7 +853,7 @@ unset BITBAKEDIR
>>>   unset SCRIPTS
>>>   export PATH
>>>   export BUILDDIR=${builddir}
>>> -export BB_ENV_EXTRAWHITE="MACHINE DISTRO TCMODE TCLIBC http_proxy 
>>> ftp_proxy https_proxy all_proxy ALL_PROXY no_proxy SSH_AGENT_PID 
>>> SSH_AUTH_SOCK BB_SRCREV_POLICY SDKMACHINE BB_NUMBER_THREADS 
>>> PARALLEL_MAKE GIT_PROXY_COMMAND GIT_PROXY_IGNORE SOCKS5_PASSWD 
>>> SOCKS5_USER OEBASE META_SDK_PATH TOOLCHAIN_TYPE TOOLCHAIN_BRAND 
>>> TOOLCHAIN_BASE TOOLCHAIN_PATH TOOLCHAIN_PATH_ARMV5 
>>> TOOLCHAIN_PATH_ARMV7 TOOLCHAIN_PATH_ARMV8 EXTRA_TISDK_FILES 
>>> TISDK_VERSION ARAGO_BRAND ARAGO_RT_ENABLE ARAGO_SYSTEST_ENABLE 
>>> ARAGO_KERNEL_SUFFIX TI_SECURE_DEV_PKG_CAT TI_SECURE_DEV_PKG_AUTO 
>>> TI_SECURE_DEV_PKG_K3 ARAGO_SYSVINIT SYSFW_FILE"
>>> +export BB_ENV_PASSTHROUGH_ADDITIONS="MACHINE DISTRO TCMODE TCLIBC 
>>> http_proxy ftp_proxy https_proxy all_proxy ALL_PROXY no_proxy 
>>> SSH_AGENT_PID SSH_AUTH_SOCK BB_SRCREV_POLICY SDKMACHINE 
>>> BB_NUMBER_THREADS PARALLEL_MAKE GIT_PROXY_COMMAND GIT_PROXY_IGNORE 
>>> SOCKS5_PASSWD SOCKS5_USER OEBASE META_SDK_PATH TOOLCHAIN_TYPE 
>>> TOOLCHAIN_BRAND TOOLCHAIN_BASE TOOLCHAIN_PATH TOOLCHAIN_PATH_ARMV5 
>>> TOOLCHAIN_PATH_ARMV7 TOOLCHAIN_PATH_ARMV8 EXTRA_TISDK_FILES 
>>> TISDK_VERSION ARAGO_BRAND ARAGO_RT_ENABLE ARAGO_SYSTEST_ENABLE 
>>> ARAGO_KERNEL_SUFFIX TI_SECURE_DEV_PKG_CAT TI_SECURE_DEV_PKG_AUTO 
>>> TI_SECURE_DEV_PKG_K3 ARAGO_SYSVINIT SYSFW_FILE"
>>>   EOM
>>>   }

-- 
Ryan Eatmon                reatmon@ti.com
-----------------------------------------
Texas Instruments, Inc.  -  LCPD  -  MGTS

Re: [meta-arago][oe-layersetup PATCH 4/6] Rename BB_ENV_EXTRAWHITE to BB_ENV_PASSTHROUGH_ADDITIONS

[Andrew Davis]

On 4/28/22 11:11 AM, Ryan Eatmon wrote:
> 
> On 4/26/2022 16:23, Andrew Davis wrote:
>> On 4/26/22 4:08 PM, Denys Dmytriyenko wrote:
>>> On Tue, Apr 26, 2022 at 03:02:49PM -0500, Andrew F. Davis via lists.yoctoproject.org wrote:
>>>> This variable was renamed upstream, rename it in our setenv file.
>>>
>>> This will break dunfell...
>>>
>>
>>
>> And not doing it breaks master...
>>
>> There is a check for BB_ENV_EXTRAWHITE, having it set will halt
>> build (think the check is in bitbake but haven't looked).
>>
> 
> This one patch seems to be the only hold out.  We clearly cannot break our current production.  Do you want to resubmit the series without this patch and we can keep looking in this issue separately?
> 
> 


Not sure what your tooling looks like, but could you just
take all the other patch in this series skipping this one?

Or do you need a full resend?

Andrew


> 
>>>
>>>> Signed-off-by: Andrew Davis <afd@ti.com>
>>>> ---
>>>>   oe-layertool-setup.sh | 2 +-
>>>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>>>
>>>> diff --git a/oe-layertool-setup.sh b/oe-layertool-setup.sh
>>>> index f171f94..72416b0 100755
>>>> --- a/oe-layertool-setup.sh
>>>> +++ b/oe-layertool-setup.sh
>>>> @@ -853,7 +853,7 @@ unset BITBAKEDIR
>>>>   unset SCRIPTS
>>>>   export PATH
>>>>   export BUILDDIR=${builddir}
>>>> -export BB_ENV_EXTRAWHITE="MACHINE DISTRO TCMODE TCLIBC http_proxy ftp_proxy https_proxy all_proxy ALL_PROXY no_proxy SSH_AGENT_PID SSH_AUTH_SOCK BB_SRCREV_POLICY SDKMACHINE BB_NUMBER_THREADS PARALLEL_MAKE GIT_PROXY_COMMAND GIT_PROXY_IGNORE SOCKS5_PASSWD SOCKS5_USER OEBASE META_SDK_PATH TOOLCHAIN_TYPE TOOLCHAIN_BRAND TOOLCHAIN_BASE TOOLCHAIN_PATH TOOLCHAIN_PATH_ARMV5 TOOLCHAIN_PATH_ARMV7 TOOLCHAIN_PATH_ARMV8 EXTRA_TISDK_FILES TISDK_VERSION ARAGO_BRAND ARAGO_RT_ENABLE ARAGO_SYSTEST_ENABLE ARAGO_KERNEL_SUFFIX TI_SECURE_DEV_PKG_CAT TI_SECURE_DEV_PKG_AUTO TI_SECURE_DEV_PKG_K3 ARAGO_SYSVINIT SYSFW_FILE"
>>>> +export BB_ENV_PASSTHROUGH_ADDITIONS="MACHINE DISTRO TCMODE TCLIBC http_proxy ftp_proxy https_proxy all_proxy ALL_PROXY no_proxy SSH_AGENT_PID SSH_AUTH_SOCK BB_SRCREV_POLICY SDKMACHINE BB_NUMBER_THREADS PARALLEL_MAKE GIT_PROXY_COMMAND GIT_PROXY_IGNORE SOCKS5_PASSWD SOCKS5_USER OEBASE META_SDK_PATH TOOLCHAIN_TYPE TOOLCHAIN_BRAND TOOLCHAIN_BASE TOOLCHAIN_PATH TOOLCHAIN_PATH_ARMV5 TOOLCHAIN_PATH_ARMV7 TOOLCHAIN_PATH_ARMV8 EXTRA_TISDK_FILES TISDK_VERSION ARAGO_BRAND ARAGO_RT_ENABLE ARAGO_SYSTEST_ENABLE ARAGO_KERNEL_SUFFIX TI_SECURE_DEV_PKG_CAT TI_SECURE_DEV_PKG_AUTO TI_SECURE_DEV_PKG_K3 ARAGO_SYSVINIT SYSFW_FILE"
>>>>   EOM
>>>>   }
> 

Re: [meta-arago][oe-layersetup PATCH 4/6] Rename BB_ENV_EXTRAWHITE to BB_ENV_PASSTHROUGH_ADDITIONS

[Ryan Eatmon]

On 4/28/2022 11:48, Andrew Davis wrote:
> On 4/28/22 11:11 AM, Ryan Eatmon wrote:
>>
>> On 4/26/2022 16:23, Andrew Davis wrote:
>>> On 4/26/22 4:08 PM, Denys Dmytriyenko wrote:
>>>> On Tue, Apr 26, 2022 at 03:02:49PM -0500, Andrew F. Davis via 
>>>> lists.yoctoproject.org wrote:
>>>>> This variable was renamed upstream, rename it in our setenv file.
>>>>
>>>> This will break dunfell...
>>>>
>>>
>>>
>>> And not doing it breaks master...
>>>
>>> There is a check for BB_ENV_EXTRAWHITE, having it set will halt
>>> build (think the check is in bitbake but haven't looked).
>>>
>>
>> This one patch seems to be the only hold out.  We clearly cannot break 
>> our current production.  Do you want to resubmit the series without 
>> this patch and we can keep looking in this issue separately?
>>
>>
> 
> 
> Not sure what your tooling looks like, but could you just
> take all the other patch in this series skipping this one?
> 
> Or do you need a full resend?


Full resend as v2 would be best.  Cleanest for the mailing list history 
and for Patchwork.


> Andrew
> 
> 
>>
>>>>
>>>>> Signed-off-by: Andrew Davis <afd@ti.com>
>>>>> ---
>>>>>   oe-layertool-setup.sh | 2 +-
>>>>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>>>>
>>>>> diff --git a/oe-layertool-setup.sh b/oe-layertool-setup.sh
>>>>> index f171f94..72416b0 100755
>>>>> --- a/oe-layertool-setup.sh
>>>>> +++ b/oe-layertool-setup.sh
>>>>> @@ -853,7 +853,7 @@ unset BITBAKEDIR
>>>>>   unset SCRIPTS
>>>>>   export PATH
>>>>>   export BUILDDIR=${builddir}
>>>>> -export BB_ENV_EXTRAWHITE="MACHINE DISTRO TCMODE TCLIBC http_proxy 
>>>>> ftp_proxy https_proxy all_proxy ALL_PROXY no_proxy SSH_AGENT_PID 
>>>>> SSH_AUTH_SOCK BB_SRCREV_POLICY SDKMACHINE BB_NUMBER_THREADS 
>>>>> PARALLEL_MAKE GIT_PROXY_COMMAND GIT_PROXY_IGNORE SOCKS5_PASSWD 
>>>>> SOCKS5_USER OEBASE META_SDK_PATH TOOLCHAIN_TYPE TOOLCHAIN_BRAND 
>>>>> TOOLCHAIN_BASE TOOLCHAIN_PATH TOOLCHAIN_PATH_ARMV5 
>>>>> TOOLCHAIN_PATH_ARMV7 TOOLCHAIN_PATH_ARMV8 EXTRA_TISDK_FILES 
>>>>> TISDK_VERSION ARAGO_BRAND ARAGO_RT_ENABLE ARAGO_SYSTEST_ENABLE 
>>>>> ARAGO_KERNEL_SUFFIX TI_SECURE_DEV_PKG_CAT TI_SECURE_DEV_PKG_AUTO 
>>>>> TI_SECURE_DEV_PKG_K3 ARAGO_SYSVINIT SYSFW_FILE"
>>>>> +export BB_ENV_PASSTHROUGH_ADDITIONS="MACHINE DISTRO TCMODE TCLIBC 
>>>>> http_proxy ftp_proxy https_proxy all_proxy ALL_PROXY no_proxy 
>>>>> SSH_AGENT_PID SSH_AUTH_SOCK BB_SRCREV_POLICY SDKMACHINE 
>>>>> BB_NUMBER_THREADS PARALLEL_MAKE GIT_PROXY_COMMAND GIT_PROXY_IGNORE 
>>>>> SOCKS5_PASSWD SOCKS5_USER OEBASE META_SDK_PATH TOOLCHAIN_TYPE 
>>>>> TOOLCHAIN_BRAND TOOLCHAIN_BASE TOOLCHAIN_PATH TOOLCHAIN_PATH_ARMV5 
>>>>> TOOLCHAIN_PATH_ARMV7 TOOLCHAIN_PATH_ARMV8 EXTRA_TISDK_FILES 
>>>>> TISDK_VERSION ARAGO_BRAND ARAGO_RT_ENABLE ARAGO_SYSTEST_ENABLE 
>>>>> ARAGO_KERNEL_SUFFIX TI_SECURE_DEV_PKG_CAT TI_SECURE_DEV_PKG_AUTO 
>>>>> TI_SECURE_DEV_PKG_K3 ARAGO_SYSVINIT SYSFW_FILE"
>>>>>   EOM
>>>>>   }
>>

-- 
Ryan Eatmon                reatmon@ti.com
-----------------------------------------
Texas Instruments, Inc.  -  LCPD  -  MGTS