* [PATCH 1/8] isar-boostrap-helper: move 'HOST_ARCH' and 'HOST_DISTRO' to base.bbclass
2019-04-16 13:26 [PATCH 0/8] Cleanup rootfs creation claudius.heine.ext
@ 2019-04-16 13:26 ` claudius.heine.ext
2019-04-16 13:26 ` [PATCH 2/8] move 'HOST_DISTRO_APT_SOURCES' from bootstrap-helper to isar-bootstrap claudius.heine.ext
` (7 subsequent siblings)
8 siblings, 0 replies; 15+ messages in thread
From: claudius.heine.ext @ 2019-04-16 13:26 UTC (permalink / raw)
To: isar-users; +Cc: Claudius Heine
From: Claudius Heine <ch@denx.de>
This also sets the 'HOST_DISTRO' to 'DISTRO' per default. (jessie
support was dropped.)
Signed-off-by: Claudius Heine <ch@denx.de>
---
meta/classes/base.bbclass | 9 +++++++++
meta/classes/isar-bootstrap-helper.bbclass | 12 ------------
2 files changed, 9 insertions(+), 12 deletions(-)
diff --git a/meta/classes/base.bbclass b/meta/classes/base.bbclass
index 9a1837f..ca80bf8 100644
--- a/meta/classes/base.bbclass
+++ b/meta/classes/base.bbclass
@@ -20,6 +20,15 @@
THISDIR = "${@os.path.dirname(d.getVar('FILE', True))}"
+def get_deb_host_arch():
+ import subprocess
+ host_arch = subprocess.check_output(
+ ["dpkg", "--print-architecture"]
+ ).decode('utf-8').strip()
+ return host_arch
+HOST_ARCH ??= "${@get_deb_host_arch()}"
+HOST_DISTRO ??= "${DISTRO}"
+
die() {
bbfatal "$*"
}
diff --git a/meta/classes/isar-bootstrap-helper.bbclass b/meta/classes/isar-bootstrap-helper.bbclass
index 38d90fb..59c447b 100644
--- a/meta/classes/isar-bootstrap-helper.bbclass
+++ b/meta/classes/isar-bootstrap-helper.bbclass
@@ -7,18 +7,6 @@
IMAGE_TRANSIENT_PACKAGES ??= ""
-def get_deb_host_arch():
- import subprocess
- host_arch = subprocess.check_output(
- ["dpkg", "--print-architecture"]
- ).decode('utf-8').strip()
- return host_arch
-
-#Debian Distribution for SDK host
-HOST_DISTRO ?= "debian-stretch"
-#Determine SDK host architecture if not explicitly set
-HOST_ARCH ?= "${@get_deb_host_arch()}"
-
HOST_DISTRO_APT_SOURCES += "conf/distro/${HOST_DISTRO}.list"
def reverse_bb_array(d, varname):
--
2.20.1
^ permalink raw reply [flat|nested] 15+ messages in thread* [PATCH 2/8] move 'HOST_DISTRO_APT_SOURCES' from bootstrap-helper to isar-bootstrap
2019-04-16 13:26 [PATCH 0/8] Cleanup rootfs creation claudius.heine.ext
2019-04-16 13:26 ` [PATCH 1/8] isar-boostrap-helper: move 'HOST_ARCH' and 'HOST_DISTRO' to base.bbclass claudius.heine.ext
@ 2019-04-16 13:26 ` claudius.heine.ext
2019-04-16 13:26 ` [PATCH 3/8] buildchroot.bbclass: only cross build if HOST_ARCH != DISTRO_ARCH claudius.heine.ext
` (6 subsequent siblings)
8 siblings, 0 replies; 15+ messages in thread
From: claudius.heine.ext @ 2019-04-16 13:26 UTC (permalink / raw)
To: isar-users; +Cc: Claudius Heine
From: Claudius Heine <ch@denx.de>
This variable is only used in 'isar-bootstrap-host.bb' and mentioned in
'isar-bootstrap.inc'. So it does not make sense to have it available
anywhere else.
Signed-off-by: Claudius Heine <ch@denx.de>
---
meta/classes/isar-bootstrap-helper.bbclass | 2 --
meta/recipes-core/isar-bootstrap/isar-bootstrap.inc | 1 +
2 files changed, 1 insertion(+), 2 deletions(-)
diff --git a/meta/classes/isar-bootstrap-helper.bbclass b/meta/classes/isar-bootstrap-helper.bbclass
index 59c447b..0d77929 100644
--- a/meta/classes/isar-bootstrap-helper.bbclass
+++ b/meta/classes/isar-bootstrap-helper.bbclass
@@ -7,8 +7,6 @@
IMAGE_TRANSIENT_PACKAGES ??= ""
-HOST_DISTRO_APT_SOURCES += "conf/distro/${HOST_DISTRO}.list"
-
def reverse_bb_array(d, varname):
array = d.getVar(varname, True)
if array is None:
diff --git a/meta/recipes-core/isar-bootstrap/isar-bootstrap.inc b/meta/recipes-core/isar-bootstrap/isar-bootstrap.inc
index c1b571a..0ec2e2b 100644
--- a/meta/recipes-core/isar-bootstrap/isar-bootstrap.inc
+++ b/meta/recipes-core/isar-bootstrap/isar-bootstrap.inc
@@ -27,6 +27,7 @@ APTKEYRING = "${WORKDIR}/apt-keyring.gpg"
DEBOOTSTRAP_KEYRING = ""
DEPLOY_ISAR_BOOTSTRAP ?= ""
DISTRO_BOOTSTRAP_BASE_PACKAGES = "locales"
+HOST_DISTRO_APT_SOURCES += "conf/distro/${HOST_DISTRO}.list"
DISTRO_APT_PREMIRRORS ?= "${@ "http://ftp\.(\S+\.)?debian.org file:///${REPO_BASE_DIR} \n" if bb.utils.to_boolean(d.getVar('ISAR_USE_CACHED_BASE_REPO')) else "" }"
--
2.20.1
^ permalink raw reply [flat|nested] 15+ messages in thread* [PATCH 3/8] buildchroot.bbclass: only cross build if HOST_ARCH != DISTRO_ARCH
2019-04-16 13:26 [PATCH 0/8] Cleanup rootfs creation claudius.heine.ext
2019-04-16 13:26 ` [PATCH 1/8] isar-boostrap-helper: move 'HOST_ARCH' and 'HOST_DISTRO' to base.bbclass claudius.heine.ext
2019-04-16 13:26 ` [PATCH 2/8] move 'HOST_DISTRO_APT_SOURCES' from bootstrap-helper to isar-bootstrap claudius.heine.ext
@ 2019-04-16 13:26 ` claudius.heine.ext
2019-04-16 13:26 ` [PATCH 4/8] isar-bootstrap/buildchroot/sdkchroot: refactor PF and WORKDIR claudius.heine.ext
` (5 subsequent siblings)
8 siblings, 0 replies; 15+ messages in thread
From: claudius.heine.ext @ 2019-04-16 13:26 UTC (permalink / raw)
To: isar-users; +Cc: Claudius Heine
From: Claudius Heine <ch@denx.de>
Signed-off-by: Claudius Heine <ch@denx.de>
---
meta/classes/buildchroot.bbclass | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/meta/classes/buildchroot.bbclass b/meta/classes/buildchroot.bbclass
index 1a87bf4..ea1538a 100644
--- a/meta/classes/buildchroot.bbclass
+++ b/meta/classes/buildchroot.bbclass
@@ -8,7 +8,7 @@ ISAR_CROSS_COMPILE ??= "0"
# Add dependency from the correct buildchroot: host or target
python __anonymous() {
mode = d.getVar('ISAR_CROSS_COMPILE', True)
- if mode == "0":
+ if mode == "0" or d.getVar('HOST_ARCH') == d.getVar('DISTRO_ARCH'):
dep = "buildchroot-target:do_build"
rootfs = d.getVar('BUILDCHROOT_TARGET_DIR', True)
else:
--
2.20.1
^ permalink raw reply [flat|nested] 15+ messages in thread* [PATCH 4/8] isar-bootstrap/buildchroot/sdkchroot: refactor PF and WORKDIR
2019-04-16 13:26 [PATCH 0/8] Cleanup rootfs creation claudius.heine.ext
` (2 preceding siblings ...)
2019-04-16 13:26 ` [PATCH 3/8] buildchroot.bbclass: only cross build if HOST_ARCH != DISTRO_ARCH claudius.heine.ext
@ 2019-04-16 13:26 ` claudius.heine.ext
2019-04-16 13:43 ` Andreas Reichel
2019-04-16 13:26 ` [PATCH 5/8] bitbake.conf: remove unneeded and differently used variables claudius.heine.ext
` (4 subsequent siblings)
8 siblings, 1 reply; 15+ messages in thread
From: claudius.heine.ext @ 2019-04-16 13:26 UTC (permalink / raw)
To: isar-users; +Cc: Claudius Heine
From: Claudius Heine <ch@denx.de>
Each of those are dependent on both HOST_ARCH and DISTRO_ARCH, so the
stamps etc. should not be shared.
Signed-off-by: Claudius Heine <ch@denx.de>
---
meta/classes/image-sdk-extension.bbclass | 2 +-
meta/conf/bitbake.conf | 4 ++--
.../isar-bootstrap/isar-bootstrap-host.bb | 11 +++--------
.../isar-bootstrap/isar-bootstrap-target.bb | 6 +-----
meta/recipes-devtools/buildchroot/buildchroot-host.bb | 1 +
meta/recipes-devtools/buildchroot/buildchroot.inc | 3 +--
meta/recipes-devtools/sdkchroot/sdkchroot.bb | 3 +--
7 files changed, 10 insertions(+), 20 deletions(-)
diff --git a/meta/classes/image-sdk-extension.bbclass b/meta/classes/image-sdk-extension.bbclass
index 1838080..8df3ed1 100644
--- a/meta/classes/image-sdk-extension.bbclass
+++ b/meta/classes/image-sdk-extension.bbclass
@@ -5,7 +5,7 @@
#
# This class extends the image.bbclass to supply the creation of a sdk
-SDKCHROOT_DIR = "${TMPDIR}/work/${DISTRO}-${DISTRO_ARCH}/sdkchroot-${HOST_DISTRO}-${HOST_ARCH}"
+SDKCHROOT_DIR = "${TMPDIR}/work/${DISTRO}-${DISTRO_ARCH}/sdkchroot-${HOST_DISTRO}-${HOST_ARCH}-${DISTRO_ARCH}"
do_populate_sdk[stamp-extra-info] = "${DISTRO}-${MACHINE}"
do_populate_sdk[depends] = "sdkchroot:do_build"
diff --git a/meta/conf/bitbake.conf b/meta/conf/bitbake.conf
index 0e521bb..7c1c89f 100644
--- a/meta/conf/bitbake.conf
+++ b/meta/conf/bitbake.conf
@@ -50,8 +50,8 @@ DEPLOY_DIR_BOOTSTRAP = "${DEPLOY_DIR}/bootstrap"
DEPLOY_DIR_IMAGE = "${DEPLOY_DIR}/images/${MACHINE}"
DL_DIR ?= "${TOPDIR}/downloads"
SSTATE_DIR ?= "${TMPDIR}/sstate-cache"
-BUILDCHROOT_HOST_DIR = "${TMPDIR}/work/${DISTRO}-${DISTRO_ARCH}/buildchroot-host/rootfs"
-BUILDCHROOT_TARGET_DIR = "${TMPDIR}/work/${DISTRO}-${DISTRO_ARCH}/buildchroot-target/rootfs"
+BUILDCHROOT_HOST_DIR = "${TMPDIR}/work/${DISTRO}-${DISTRO_ARCH}/buildchroot-host-${DISTRO}-${HOST_ARCH}-${DISTRO_ARCH}/rootfs"
+BUILDCHROOT_TARGET_DIR = "${TMPDIR}/work/${DISTRO}-${DISTRO_ARCH}/buildchroot-target-${DISTRO}-${DISTRO_ARCH}/rootfs"
CACHE = "${TMPDIR}/cache"
OVERRIDES = "${DISTRO_ARCH}:${MACHINE}:${DISTRO}:forcevariable"
diff --git a/meta/recipes-core/isar-bootstrap/isar-bootstrap-host.bb b/meta/recipes-core/isar-bootstrap/isar-bootstrap-host.bb
index 08b068f..6d0f5f5 100644
--- a/meta/recipes-core/isar-bootstrap/isar-bootstrap-host.bb
+++ b/meta/recipes-core/isar-bootstrap/isar-bootstrap-host.bb
@@ -6,17 +6,13 @@
# SPDX-License-Identifier: MIT
Description = "Minimal host Debian root file system"
+PF = "${PN}-${HOST_DISTRO}-${HOST_ARCH}-${DISTRO_ARCH}"
-WORKDIR = "${TMPDIR}/work/${DISTRO}-${DISTRO_ARCH}/${PN}-${HOST_DISTRO}-${HOST_ARCH}"
-DEPLOY_ISAR_BOOTSTRAP = "${DEPLOY_DIR_BOOTSTRAP}/${HOST_DISTRO}-${HOST_ARCH}"
-ISAR_BOOTSTRAP_LOCK = "${DEPLOY_DIR_BOOTSTRAP}/${HOST_DISTRO}-${HOST_ARCH}.lock"
+DEPLOY_ISAR_BOOTSTRAP = "${DEPLOY_DIR_BOOTSTRAP}/${HOST_DISTRO}-${HOST_ARCH}-${DISTRO_ARCH}"
+ISAR_BOOTSTRAP_LOCK = "${DEPLOY_DIR_BOOTSTRAP}/${HOST_DISTRO}-${HOST_ARCH}-${DISTRO_ARCH}.lock"
require isar-bootstrap.inc
-inherit isar-bootstrap-helper
-do_generate_keyring[stamp-extra-info] = "${DISTRO}-${DISTRO_ARCH}"
-
-do_apt_config_prepare[stamp-extra-info] = "${DISTRO}-${DISTRO_ARCH}"
do_apt_config_prepare[dirs] = "${WORKDIR}"
do_apt_config_prepare[vardeps] += "\
APTPREFS \
@@ -47,7 +43,6 @@ addtask apt_config_prepare before do_bootstrap after do_unpack
OVERRIDES_append = ":${@get_distro_needs_https_support(d, True)}"
-do_bootstrap[stamp-extra-info] = "${HOST_DISTRO}-${HOST_ARCH}"
do_bootstrap[vardeps] += "HOST_DISTRO_APT_SOURCES"
do_bootstrap() {
isar_bootstrap --host
diff --git a/meta/recipes-core/isar-bootstrap/isar-bootstrap-target.bb b/meta/recipes-core/isar-bootstrap/isar-bootstrap-target.bb
index 79f3e34..552a317 100644
--- a/meta/recipes-core/isar-bootstrap/isar-bootstrap-target.bb
+++ b/meta/recipes-core/isar-bootstrap/isar-bootstrap-target.bb
@@ -6,16 +6,13 @@
# SPDX-License-Identifier: MIT
Description = "Minimal target Debian root file system"
+PF = "${PN}-${DISTRO}-${DISTRO_ARCH}"
-WORKDIR = "${TMPDIR}/work/${DISTRO}-${DISTRO_ARCH}/${PN}"
DEPLOY_ISAR_BOOTSTRAP = "${DEPLOY_DIR_BOOTSTRAP}/${DISTRO}-${DISTRO_ARCH}"
ISAR_BOOTSTRAP_LOCK = "${DEPLOY_DIR_BOOTSTRAP}/${DISTRO}-${DISTRO_ARCH}.lock"
require isar-bootstrap.inc
-do_generate_keyring[stamp-extra-info] = "${DISTRO}-${DISTRO_ARCH}"
-
-do_apt_config_prepare[stamp-extra-info] = "${DISTRO}-${DISTRO_ARCH}"
do_apt_config_prepare[dirs] = "${WORKDIR}"
do_apt_config_prepare[vardeps] += "\
APTPREFS \
@@ -44,7 +41,6 @@ addtask apt_config_prepare before do_bootstrap after do_unpack
OVERRIDES_append = ":${@get_distro_needs_https_support(d, False)}"
-do_bootstrap[stamp-extra-info] = "${DISTRO}-${DISTRO_ARCH}"
do_bootstrap[vardeps] += "DISTRO_APT_SOURCES"
do_bootstrap() {
isar_bootstrap
diff --git a/meta/recipes-devtools/buildchroot/buildchroot-host.bb b/meta/recipes-devtools/buildchroot/buildchroot-host.bb
index c8c10c0..e8f7f69 100644
--- a/meta/recipes-devtools/buildchroot/buildchroot-host.bb
+++ b/meta/recipes-devtools/buildchroot/buildchroot-host.bb
@@ -6,6 +6,7 @@
DESCRIPTION = "Isar development filesystem for host"
require buildchroot.inc
+PF = "${PN}-${HOST_DISTRO}-${HOST_ARCH}-${DISTRO_ARCH}"
BUILDCHROOT_PREINSTALL ?= "make \
debhelper \
diff --git a/meta/recipes-devtools/buildchroot/buildchroot.inc b/meta/recipes-devtools/buildchroot/buildchroot.inc
index 764bc5e..cdb4c07 100644
--- a/meta/recipes-devtools/buildchroot/buildchroot.inc
+++ b/meta/recipes-devtools/buildchroot/buildchroot.inc
@@ -12,13 +12,12 @@ SRC_URI = "file://configscript.sh \
file://common.sh \
file://deps.sh"
PV = "1.0"
+PF = "${PN}-${DISTRO}-${DISTRO_ARCH}"
inherit isar-bootstrap-helper
-WORKDIR = "${TMPDIR}/work/${DISTRO}-${DISTRO_ARCH}/${PN}"
BUILDCHROOT_DIR = "${WORKDIR}/rootfs"
-do_build[stamp-extra-info] = "${DISTRO}-${DISTRO_ARCH}"
do_build[root_cleandirs] = "${BUILDCHROOT_DIR} \
${BUILDCHROOT_DIR}/isar-apt \
${BUILDCHROOT_DIR}/downloads \
diff --git a/meta/recipes-devtools/sdkchroot/sdkchroot.bb b/meta/recipes-devtools/sdkchroot/sdkchroot.bb
index 895a454..249fa94 100644
--- a/meta/recipes-devtools/sdkchroot/sdkchroot.bb
+++ b/meta/recipes-devtools/sdkchroot/sdkchroot.bb
@@ -14,6 +14,7 @@ SRC_URI = " \
PV = "0.1"
inherit isar-bootstrap-helper
+PF = "${PN}-${ROOTFS_DISTRO}-${HOST_ARCH}-${DISTRO_ARCH}"
SDKCHROOT_PREINSTALL := "debhelper \
autotools-dev \
@@ -25,11 +26,9 @@ SDKCHROOT_PREINSTALL := "debhelper \
devscripts \
equivs"
-WORKDIR = "${TMPDIR}/work/${DISTRO}-${DISTRO_ARCH}/${PN}-${HOST_DISTRO}-${HOST_ARCH}"
S = "${WORKDIR}/rootfs"
do_build[dirs] = "${DEPLOY_DIR_IMAGE}"
-do_build[stamp-extra-info] = "${HOST_DISTRO}-${HOST_ARCH}"
do_build[root_cleandirs] = "${S} \
${S}/isar-apt"
--
2.20.1
^ permalink raw reply [flat|nested] 15+ messages in thread* Re: [PATCH 4/8] isar-bootstrap/buildchroot/sdkchroot: refactor PF and WORKDIR
2019-04-16 13:26 ` [PATCH 4/8] isar-bootstrap/buildchroot/sdkchroot: refactor PF and WORKDIR claudius.heine.ext
@ 2019-04-16 13:43 ` Andreas Reichel
2019-04-16 13:59 ` Claudius Heine
0 siblings, 1 reply; 15+ messages in thread
From: Andreas Reichel @ 2019-04-16 13:43 UTC (permalink / raw)
To: [ext] claudius.heine.ext@siemens.com; +Cc: isar-users, Claudius Heine
On Tue, Apr 16, 2019 at 03:26:42PM +0200, [ext] claudius.heine.ext@siemens.com wrote:
> From: Claudius Heine <ch@denx.de>
>
> Each of those are dependent on both HOST_ARCH and DISTRO_ARCH, so the
> stamps etc. should not be shared.
>
> Signed-off-by: Claudius Heine <ch@denx.de>
> ---
> meta/classes/image-sdk-extension.bbclass | 2 +-
> meta/conf/bitbake.conf | 4 ++--
> .../isar-bootstrap/isar-bootstrap-host.bb | 11 +++--------
> .../isar-bootstrap/isar-bootstrap-target.bb | 6 +-----
> meta/recipes-devtools/buildchroot/buildchroot-host.bb | 1 +
> meta/recipes-devtools/buildchroot/buildchroot.inc | 3 +--
> meta/recipes-devtools/sdkchroot/sdkchroot.bb | 3 +--
> 7 files changed, 10 insertions(+), 20 deletions(-)
>
> diff --git a/meta/classes/image-sdk-extension.bbclass b/meta/classes/image-sdk-extension.bbclass
> index 1838080..8df3ed1 100644
> --- a/meta/classes/image-sdk-extension.bbclass
> +++ b/meta/classes/image-sdk-extension.bbclass
> @@ -5,7 +5,7 @@
> #
> # This class extends the image.bbclass to supply the creation of a sdk
>
> -SDKCHROOT_DIR = "${TMPDIR}/work/${DISTRO}-${DISTRO_ARCH}/sdkchroot-${HOST_DISTRO}-${HOST_ARCH}"
> +SDKCHROOT_DIR = "${TMPDIR}/work/${DISTRO}-${DISTRO_ARCH}/sdkchroot-${HOST_DISTRO}-${HOST_ARCH}-${DISTRO_ARCH}"
>
> do_populate_sdk[stamp-extra-info] = "${DISTRO}-${MACHINE}"
> do_populate_sdk[depends] = "sdkchroot:do_build"
> diff --git a/meta/conf/bitbake.conf b/meta/conf/bitbake.conf
> index 0e521bb..7c1c89f 100644
> --- a/meta/conf/bitbake.conf
> +++ b/meta/conf/bitbake.conf
> @@ -50,8 +50,8 @@ DEPLOY_DIR_BOOTSTRAP = "${DEPLOY_DIR}/bootstrap"
> DEPLOY_DIR_IMAGE = "${DEPLOY_DIR}/images/${MACHINE}"
> DL_DIR ?= "${TOPDIR}/downloads"
> SSTATE_DIR ?= "${TMPDIR}/sstate-cache"
> -BUILDCHROOT_HOST_DIR = "${TMPDIR}/work/${DISTRO}-${DISTRO_ARCH}/buildchroot-host/rootfs"
> -BUILDCHROOT_TARGET_DIR = "${TMPDIR}/work/${DISTRO}-${DISTRO_ARCH}/buildchroot-target/rootfs"
> +BUILDCHROOT_HOST_DIR = "${TMPDIR}/work/${DISTRO}-${DISTRO_ARCH}/buildchroot-host-${DISTRO}-${HOST_ARCH}-${DISTRO_ARCH}/rootfs"
> +BUILDCHROOT_TARGET_DIR = "${TMPDIR}/work/${DISTRO}-${DISTRO_ARCH}/buildchroot-target-${DISTRO}-${DISTRO_ARCH}/rootfs"
maybe first define a variable for ${DISTRO}-${DISTRO_ARCH} and then
prepend this with ${HOST_ARCH}- where needed makes the code more
readable.
> CACHE = "${TMPDIR}/cache"
>
> OVERRIDES = "${DISTRO_ARCH}:${MACHINE}:${DISTRO}:forcevariable"
> diff --git a/meta/recipes-core/isar-bootstrap/isar-bootstrap-host.bb b/meta/recipes-core/isar-bootstrap/isar-bootstrap-host.bb
> index 08b068f..6d0f5f5 100644
> --- a/meta/recipes-core/isar-bootstrap/isar-bootstrap-host.bb
> +++ b/meta/recipes-core/isar-bootstrap/isar-bootstrap-host.bb
> @@ -6,17 +6,13 @@
> # SPDX-License-Identifier: MIT
>
> Description = "Minimal host Debian root file system"
> +PF = "${PN}-${HOST_DISTRO}-${HOST_ARCH}-${DISTRO_ARCH}"
>
> -WORKDIR = "${TMPDIR}/work/${DISTRO}-${DISTRO_ARCH}/${PN}-${HOST_DISTRO}-${HOST_ARCH}"
> -DEPLOY_ISAR_BOOTSTRAP = "${DEPLOY_DIR_BOOTSTRAP}/${HOST_DISTRO}-${HOST_ARCH}"
> -ISAR_BOOTSTRAP_LOCK = "${DEPLOY_DIR_BOOTSTRAP}/${HOST_DISTRO}-${HOST_ARCH}.lock"
> +DEPLOY_ISAR_BOOTSTRAP = "${DEPLOY_DIR_BOOTSTRAP}/${HOST_DISTRO}-${HOST_ARCH}-${DISTRO_ARCH}"
> +ISAR_BOOTSTRAP_LOCK = "${DEPLOY_DIR_BOOTSTRAP}/${HOST_DISTRO}-${HOST_ARCH}-${DISTRO_ARCH}.lock"
>
Same here
> require isar-bootstrap.inc
> -inherit isar-bootstrap-helper
>
> -do_generate_keyring[stamp-extra-info] = "${DISTRO}-${DISTRO_ARCH}"
> -
> -do_apt_config_prepare[stamp-extra-info] = "${DISTRO}-${DISTRO_ARCH}"
> do_apt_config_prepare[dirs] = "${WORKDIR}"
> do_apt_config_prepare[vardeps] += "\
> APTPREFS \
> @@ -47,7 +43,6 @@ addtask apt_config_prepare before do_bootstrap after do_unpack
>
> OVERRIDES_append = ":${@get_distro_needs_https_support(d, True)}"
>
> -do_bootstrap[stamp-extra-info] = "${HOST_DISTRO}-${HOST_ARCH}"
> do_bootstrap[vardeps] += "HOST_DISTRO_APT_SOURCES"
> do_bootstrap() {
> isar_bootstrap --host
> diff --git a/meta/recipes-core/isar-bootstrap/isar-bootstrap-target.bb b/meta/recipes-core/isar-bootstrap/isar-bootstrap-target.bb
> index 79f3e34..552a317 100644
> --- a/meta/recipes-core/isar-bootstrap/isar-bootstrap-target.bb
> +++ b/meta/recipes-core/isar-bootstrap/isar-bootstrap-target.bb
> @@ -6,16 +6,13 @@
> # SPDX-License-Identifier: MIT
>
> Description = "Minimal target Debian root file system"
> +PF = "${PN}-${DISTRO}-${DISTRO_ARCH}"
>
> -WORKDIR = "${TMPDIR}/work/${DISTRO}-${DISTRO_ARCH}/${PN}"
> DEPLOY_ISAR_BOOTSTRAP = "${DEPLOY_DIR_BOOTSTRAP}/${DISTRO}-${DISTRO_ARCH}"
> ISAR_BOOTSTRAP_LOCK = "${DEPLOY_DIR_BOOTSTRAP}/${DISTRO}-${DISTRO_ARCH}.lock"
>
> require isar-bootstrap.inc
>
> -do_generate_keyring[stamp-extra-info] = "${DISTRO}-${DISTRO_ARCH}"
> -
> -do_apt_config_prepare[stamp-extra-info] = "${DISTRO}-${DISTRO_ARCH}"
> do_apt_config_prepare[dirs] = "${WORKDIR}"
> do_apt_config_prepare[vardeps] += "\
> APTPREFS \
> @@ -44,7 +41,6 @@ addtask apt_config_prepare before do_bootstrap after do_unpack
>
> OVERRIDES_append = ":${@get_distro_needs_https_support(d, False)}"
>
> -do_bootstrap[stamp-extra-info] = "${DISTRO}-${DISTRO_ARCH}"
> do_bootstrap[vardeps] += "DISTRO_APT_SOURCES"
> do_bootstrap() {
> isar_bootstrap
> diff --git a/meta/recipes-devtools/buildchroot/buildchroot-host.bb b/meta/recipes-devtools/buildchroot/buildchroot-host.bb
> index c8c10c0..e8f7f69 100644
> --- a/meta/recipes-devtools/buildchroot/buildchroot-host.bb
> +++ b/meta/recipes-devtools/buildchroot/buildchroot-host.bb
> @@ -6,6 +6,7 @@
> DESCRIPTION = "Isar development filesystem for host"
>
> require buildchroot.inc
> +PF = "${PN}-${HOST_DISTRO}-${HOST_ARCH}-${DISTRO_ARCH}"
>
> BUILDCHROOT_PREINSTALL ?= "make \
> debhelper \
> diff --git a/meta/recipes-devtools/buildchroot/buildchroot.inc b/meta/recipes-devtools/buildchroot/buildchroot.inc
> index 764bc5e..cdb4c07 100644
> --- a/meta/recipes-devtools/buildchroot/buildchroot.inc
> +++ b/meta/recipes-devtools/buildchroot/buildchroot.inc
> @@ -12,13 +12,12 @@ SRC_URI = "file://configscript.sh \
> file://common.sh \
> file://deps.sh"
> PV = "1.0"
> +PF = "${PN}-${DISTRO}-${DISTRO_ARCH}"
>
> inherit isar-bootstrap-helper
>
> -WORKDIR = "${TMPDIR}/work/${DISTRO}-${DISTRO_ARCH}/${PN}"
> BUILDCHROOT_DIR = "${WORKDIR}/rootfs"
>
> -do_build[stamp-extra-info] = "${DISTRO}-${DISTRO_ARCH}"
> do_build[root_cleandirs] = "${BUILDCHROOT_DIR} \
> ${BUILDCHROOT_DIR}/isar-apt \
> ${BUILDCHROOT_DIR}/downloads \
> diff --git a/meta/recipes-devtools/sdkchroot/sdkchroot.bb b/meta/recipes-devtools/sdkchroot/sdkchroot.bb
> index 895a454..249fa94 100644
> --- a/meta/recipes-devtools/sdkchroot/sdkchroot.bb
> +++ b/meta/recipes-devtools/sdkchroot/sdkchroot.bb
> @@ -14,6 +14,7 @@ SRC_URI = " \
> PV = "0.1"
>
> inherit isar-bootstrap-helper
> +PF = "${PN}-${ROOTFS_DISTRO}-${HOST_ARCH}-${DISTRO_ARCH}"
>
> SDKCHROOT_PREINSTALL := "debhelper \
> autotools-dev \
> @@ -25,11 +26,9 @@ SDKCHROOT_PREINSTALL := "debhelper \
> devscripts \
> equivs"
>
> -WORKDIR = "${TMPDIR}/work/${DISTRO}-${DISTRO_ARCH}/${PN}-${HOST_DISTRO}-${HOST_ARCH}"
> S = "${WORKDIR}/rootfs"
>
> do_build[dirs] = "${DEPLOY_DIR_IMAGE}"
> -do_build[stamp-extra-info] = "${HOST_DISTRO}-${HOST_ARCH}"
> do_build[root_cleandirs] = "${S} \
> ${S}/isar-apt"
>
> --
> 2.20.1
>
> --
> You received this message because you are subscribed to the Google Groups "isar-users" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to isar-users+unsubscribe@googlegroups.com.
> To post to this group, send email to isar-users@googlegroups.com.
> To view this discussion on the web visit https://groups.google.com/d/msgid/isar-users/20190416132647.3074-5-claudius.heine.ext%40siemens.com.
> For more options, visit https://groups.google.com/d/optout.
--
Andreas Reichel
Dipl.-Phys. (Univ.)
Software Consultant
Andreas.Reichel@tngtech.com, +49-174-3180074
TNG Technology Consulting GmbH, Betastr. 13a, 85774 Unterfoehring
Geschaeftsfuehrer: Henrik Klagges, Dr. Robert Dahlke, Gerhard Mueller
Sitz: Unterfoehring * Amtsgericht Muenchen * HRB 135082
^ permalink raw reply [flat|nested] 15+ messages in thread* Re: [PATCH 4/8] isar-bootstrap/buildchroot/sdkchroot: refactor PF and WORKDIR
2019-04-16 13:43 ` Andreas Reichel
@ 2019-04-16 13:59 ` Claudius Heine
0 siblings, 0 replies; 15+ messages in thread
From: Claudius Heine @ 2019-04-16 13:59 UTC (permalink / raw)
To: Andreas Reichel; +Cc: isar-users, Claudius Heine
Hi Andreas,
On 16/04/2019 15.43, Andreas Reichel wrote:
> On Tue, Apr 16, 2019 at 03:26:42PM +0200, [ext] claudius.heine.ext@siemens.com wrote:
>> From: Claudius Heine <ch@denx.de>
>>
>> Each of those are dependent on both HOST_ARCH and DISTRO_ARCH, so the
>> stamps etc. should not be shared.
>>
>> Signed-off-by: Claudius Heine <ch@denx.de>
>> ---
>> meta/classes/image-sdk-extension.bbclass | 2 +-
>> meta/conf/bitbake.conf | 4 ++--
>> .../isar-bootstrap/isar-bootstrap-host.bb | 11 +++--------
>> .../isar-bootstrap/isar-bootstrap-target.bb | 6 +-----
>> meta/recipes-devtools/buildchroot/buildchroot-host.bb | 1 +
>> meta/recipes-devtools/buildchroot/buildchroot.inc | 3 +--
>> meta/recipes-devtools/sdkchroot/sdkchroot.bb | 3 +--
>> 7 files changed, 10 insertions(+), 20 deletions(-)
>>
>> diff --git a/meta/classes/image-sdk-extension.bbclass b/meta/classes/image-sdk-extension.bbclass
>> index 1838080..8df3ed1 100644
>> --- a/meta/classes/image-sdk-extension.bbclass
>> +++ b/meta/classes/image-sdk-extension.bbclass
>> @@ -5,7 +5,7 @@
>> #
>> # This class extends the image.bbclass to supply the creation of a sdk
>>
>> -SDKCHROOT_DIR = "${TMPDIR}/work/${DISTRO}-${DISTRO_ARCH}/sdkchroot-${HOST_DISTRO}-${HOST_ARCH}"
>> +SDKCHROOT_DIR = "${TMPDIR}/work/${DISTRO}-${DISTRO_ARCH}/sdkchroot-${HOST_DISTRO}-${HOST_ARCH}-${DISTRO_ARCH}"
>>
>> do_populate_sdk[stamp-extra-info] = "${DISTRO}-${MACHINE}"
>> do_populate_sdk[depends] = "sdkchroot:do_build"
>> diff --git a/meta/conf/bitbake.conf b/meta/conf/bitbake.conf
>> index 0e521bb..7c1c89f 100644
>> --- a/meta/conf/bitbake.conf
>> +++ b/meta/conf/bitbake.conf
>> @@ -50,8 +50,8 @@ DEPLOY_DIR_BOOTSTRAP = "${DEPLOY_DIR}/bootstrap"
>> DEPLOY_DIR_IMAGE = "${DEPLOY_DIR}/images/${MACHINE}"
>> DL_DIR ?= "${TOPDIR}/downloads"
>> SSTATE_DIR ?= "${TMPDIR}/sstate-cache"
>> -BUILDCHROOT_HOST_DIR = "${TMPDIR}/work/${DISTRO}-${DISTRO_ARCH}/buildchroot-host/rootfs"
>> -BUILDCHROOT_TARGET_DIR = "${TMPDIR}/work/${DISTRO}-${DISTRO_ARCH}/buildchroot-target/rootfs"
>> +BUILDCHROOT_HOST_DIR = "${TMPDIR}/work/${DISTRO}-${DISTRO_ARCH}/buildchroot-host-${DISTRO}-${HOST_ARCH}-${DISTRO_ARCH}/rootfs"
>> +BUILDCHROOT_TARGET_DIR = "${TMPDIR}/work/${DISTRO}-${DISTRO_ARCH}/buildchroot-target-${DISTRO}-${DISTRO_ARCH}/rootfs"
>
> maybe first define a variable for ${DISTRO}-${DISTRO_ARCH} and then
> prepend this with ${HOST_ARCH}- where needed makes the code more
> readable.
I understand your point, but this is the 'bitbake.conf' so any variable
will be available everywhere. So I would like to avoid creating any
helper variables that are of limited use anywhere else and create mostly
noise.
Also changing the order would lead to more inconsistencies IMO.
Everywhere the 'DISTRO' code word is named first, then followed by the
native architecture of that root file system. This change just adds the
architecture of the target machine to the last place for the
cross-tool-chain.
Thanks,
Claudius
>
>> CACHE = "${TMPDIR}/cache"
>>
>> OVERRIDES = "${DISTRO_ARCH}:${MACHINE}:${DISTRO}:forcevariable"
>> diff --git a/meta/recipes-core/isar-bootstrap/isar-bootstrap-host.bb b/meta/recipes-core/isar-bootstrap/isar-bootstrap-host.bb
>> index 08b068f..6d0f5f5 100644
>> --- a/meta/recipes-core/isar-bootstrap/isar-bootstrap-host.bb
>> +++ b/meta/recipes-core/isar-bootstrap/isar-bootstrap-host.bb
>> @@ -6,17 +6,13 @@
>> # SPDX-License-Identifier: MIT
>>
>> Description = "Minimal host Debian root file system"
>> +PF = "${PN}-${HOST_DISTRO}-${HOST_ARCH}-${DISTRO_ARCH}"
>>
>> -WORKDIR = "${TMPDIR}/work/${DISTRO}-${DISTRO_ARCH}/${PN}-${HOST_DISTRO}-${HOST_ARCH}"
>> -DEPLOY_ISAR_BOOTSTRAP = "${DEPLOY_DIR_BOOTSTRAP}/${HOST_DISTRO}-${HOST_ARCH}"
>> -ISAR_BOOTSTRAP_LOCK = "${DEPLOY_DIR_BOOTSTRAP}/${HOST_DISTRO}-${HOST_ARCH}.lock"
>> +DEPLOY_ISAR_BOOTSTRAP = "${DEPLOY_DIR_BOOTSTRAP}/${HOST_DISTRO}-${HOST_ARCH}-${DISTRO_ARCH}"
>> +ISAR_BOOTSTRAP_LOCK = "${DEPLOY_DIR_BOOTSTRAP}/${HOST_DISTRO}-${HOST_ARCH}-${DISTRO_ARCH}.lock"
>>
>
> Same here
>
>> require isar-bootstrap.inc
>> -inherit isar-bootstrap-helper
>>
>> -do_generate_keyring[stamp-extra-info] = "${DISTRO}-${DISTRO_ARCH}"
>> -
>> -do_apt_config_prepare[stamp-extra-info] = "${DISTRO}-${DISTRO_ARCH}"
>> do_apt_config_prepare[dirs] = "${WORKDIR}"
>> do_apt_config_prepare[vardeps] += "\
>> APTPREFS \
>> @@ -47,7 +43,6 @@ addtask apt_config_prepare before do_bootstrap after do_unpack
>>
>> OVERRIDES_append = ":${@get_distro_needs_https_support(d, True)}"
>>
>> -do_bootstrap[stamp-extra-info] = "${HOST_DISTRO}-${HOST_ARCH}"
>> do_bootstrap[vardeps] += "HOST_DISTRO_APT_SOURCES"
>> do_bootstrap() {
>> isar_bootstrap --host
>> diff --git a/meta/recipes-core/isar-bootstrap/isar-bootstrap-target.bb b/meta/recipes-core/isar-bootstrap/isar-bootstrap-target.bb
>> index 79f3e34..552a317 100644
>> --- a/meta/recipes-core/isar-bootstrap/isar-bootstrap-target.bb
>> +++ b/meta/recipes-core/isar-bootstrap/isar-bootstrap-target.bb
>> @@ -6,16 +6,13 @@
>> # SPDX-License-Identifier: MIT
>>
>> Description = "Minimal target Debian root file system"
>> +PF = "${PN}-${DISTRO}-${DISTRO_ARCH}"
>>
>> -WORKDIR = "${TMPDIR}/work/${DISTRO}-${DISTRO_ARCH}/${PN}"
>> DEPLOY_ISAR_BOOTSTRAP = "${DEPLOY_DIR_BOOTSTRAP}/${DISTRO}-${DISTRO_ARCH}"
>> ISAR_BOOTSTRAP_LOCK = "${DEPLOY_DIR_BOOTSTRAP}/${DISTRO}-${DISTRO_ARCH}.lock"
>>
>> require isar-bootstrap.inc
>>
>> -do_generate_keyring[stamp-extra-info] = "${DISTRO}-${DISTRO_ARCH}"
>> -
>> -do_apt_config_prepare[stamp-extra-info] = "${DISTRO}-${DISTRO_ARCH}"
>> do_apt_config_prepare[dirs] = "${WORKDIR}"
>> do_apt_config_prepare[vardeps] += "\
>> APTPREFS \
>> @@ -44,7 +41,6 @@ addtask apt_config_prepare before do_bootstrap after do_unpack
>>
>> OVERRIDES_append = ":${@get_distro_needs_https_support(d, False)}"
>>
>> -do_bootstrap[stamp-extra-info] = "${DISTRO}-${DISTRO_ARCH}"
>> do_bootstrap[vardeps] += "DISTRO_APT_SOURCES"
>> do_bootstrap() {
>> isar_bootstrap
>> diff --git a/meta/recipes-devtools/buildchroot/buildchroot-host.bb b/meta/recipes-devtools/buildchroot/buildchroot-host.bb
>> index c8c10c0..e8f7f69 100644
>> --- a/meta/recipes-devtools/buildchroot/buildchroot-host.bb
>> +++ b/meta/recipes-devtools/buildchroot/buildchroot-host.bb
>> @@ -6,6 +6,7 @@
>> DESCRIPTION = "Isar development filesystem for host"
>>
>> require buildchroot.inc
>> +PF = "${PN}-${HOST_DISTRO}-${HOST_ARCH}-${DISTRO_ARCH}"
>>
>> BUILDCHROOT_PREINSTALL ?= "make \
>> debhelper \
>> diff --git a/meta/recipes-devtools/buildchroot/buildchroot.inc b/meta/recipes-devtools/buildchroot/buildchroot.inc
>> index 764bc5e..cdb4c07 100644
>> --- a/meta/recipes-devtools/buildchroot/buildchroot.inc
>> +++ b/meta/recipes-devtools/buildchroot/buildchroot.inc
>> @@ -12,13 +12,12 @@ SRC_URI = "file://configscript.sh \
>> file://common.sh \
>> file://deps.sh"
>> PV = "1.0"
>> +PF = "${PN}-${DISTRO}-${DISTRO_ARCH}"
>>
>> inherit isar-bootstrap-helper
>>
>> -WORKDIR = "${TMPDIR}/work/${DISTRO}-${DISTRO_ARCH}/${PN}"
>> BUILDCHROOT_DIR = "${WORKDIR}/rootfs"
>>
>> -do_build[stamp-extra-info] = "${DISTRO}-${DISTRO_ARCH}"
>> do_build[root_cleandirs] = "${BUILDCHROOT_DIR} \
>> ${BUILDCHROOT_DIR}/isar-apt \
>> ${BUILDCHROOT_DIR}/downloads \
>> diff --git a/meta/recipes-devtools/sdkchroot/sdkchroot.bb b/meta/recipes-devtools/sdkchroot/sdkchroot.bb
>> index 895a454..249fa94 100644
>> --- a/meta/recipes-devtools/sdkchroot/sdkchroot.bb
>> +++ b/meta/recipes-devtools/sdkchroot/sdkchroot.bb
>> @@ -14,6 +14,7 @@ SRC_URI = " \
>> PV = "0.1"
>>
>> inherit isar-bootstrap-helper
>> +PF = "${PN}-${ROOTFS_DISTRO}-${HOST_ARCH}-${DISTRO_ARCH}"
>>
>> SDKCHROOT_PREINSTALL := "debhelper \
>> autotools-dev \
>> @@ -25,11 +26,9 @@ SDKCHROOT_PREINSTALL := "debhelper \
>> devscripts \
>> equivs"
>>
>> -WORKDIR = "${TMPDIR}/work/${DISTRO}-${DISTRO_ARCH}/${PN}-${HOST_DISTRO}-${HOST_ARCH}"
>> S = "${WORKDIR}/rootfs"
>>
>> do_build[dirs] = "${DEPLOY_DIR_IMAGE}"
>> -do_build[stamp-extra-info] = "${HOST_DISTRO}-${HOST_ARCH}"
>> do_build[root_cleandirs] = "${S} \
>> ${S}/isar-apt"
>>
>> --
>> 2.20.1
>>
>> --
>> You received this message because you are subscribed to the Google Groups "isar-users" group.
>> To unsubscribe from this group and stop receiving emails from it, send an email to isar-users+unsubscribe@googlegroups.com.
>> To post to this group, send email to isar-users@googlegroups.com.
>> To view this discussion on the web visit https://groups.google.com/d/msgid/isar-users/20190416132647.3074-5-claudius.heine.ext%40siemens.com.
>> For more options, visit https://groups.google.com/d/optout.
>
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-54 Fax: (+49)-8142-66989-80 Email: ch@denx.de
^ permalink raw reply [flat|nested] 15+ messages in thread
* [PATCH 5/8] bitbake.conf: remove unneeded and differently used variables
2019-04-16 13:26 [PATCH 0/8] Cleanup rootfs creation claudius.heine.ext
` (3 preceding siblings ...)
2019-04-16 13:26 ` [PATCH 4/8] isar-bootstrap/buildchroot/sdkchroot: refactor PF and WORKDIR claudius.heine.ext
@ 2019-04-16 13:26 ` claudius.heine.ext
2019-04-16 13:26 ` [PATCH 6/8] image.bbclass: make IMAGE_ROOTFS overwritable claudius.heine.ext
` (3 subsequent siblings)
8 siblings, 0 replies; 15+ messages in thread
From: claudius.heine.ext @ 2019-04-16 13:26 UTC (permalink / raw)
To: isar-users; +Cc: Claudius Heine
From: Claudius Heine <ch@denx.de>
IMAGE_CMD and TARGET_ARCH are not used in Isar or any bitbake code.
IMAGE_ROOTFS is used as the directory where image recipes store their
root file system. Pointing that to a global path like '${TMPDIR}/rootfs'
does not make sense for Isar.
Signed-off-by: Claudius Heine <ch@denx.de>
---
meta/conf/bitbake.conf | 3 ---
1 file changed, 3 deletions(-)
diff --git a/meta/conf/bitbake.conf b/meta/conf/bitbake.conf
index 7c1c89f..21d13ef 100644
--- a/meta/conf/bitbake.conf
+++ b/meta/conf/bitbake.conf
@@ -26,8 +26,6 @@ DEPLOY_DIR = "${TMPDIR}/deploy"
FILESPATH = "${FILE_DIRNAME}/${PF}:${FILE_DIRNAME}/${P}:${FILE_DIRNAME}/${PN}:${FILE_DIRNAME}/files:${FILE_DIRNAME}"
FILE_DIRNAME = "${@os.path.dirname(d.getVar('FILE', False))}"
GITDIR = "${DL_DIR}/git"
-IMAGE_CMD = "_NO_DEFINED_IMAGE_TYPES_"
-IMAGE_ROOTFS = "${TMPDIR}/rootfs"
P = "${PN}-${PV}"
PF = "${PN}-${PV}-${PR}"
PN = "${@bb.parse.BBHandler.vars_from_file(d.getVar('FILE', False),d)[0] or 'defaultpkgname'}"
@@ -39,7 +37,6 @@ SRC_URI = "file://${FILE}"
STAMP = "${TMPDIR}/stamps/${PF}"
SVNDIR = "${DL_DIR}/svn"
T = "${WORKDIR}/temp"
-TARGET_ARCH = "${BUILD_ARCH}"
TMPDIR = "${TOPDIR}/tmp"
PERSISTENT_DIR = "${TMPDIR}/cache"
GITPKGV = "${@bb.fetch2.get_srcrev(d, 'gitpkgv_revision')}"
--
2.20.1
^ permalink raw reply [flat|nested] 15+ messages in thread* [PATCH 6/8] image.bbclass: make IMAGE_ROOTFS overwritable
2019-04-16 13:26 [PATCH 0/8] Cleanup rootfs creation claudius.heine.ext
` (4 preceding siblings ...)
2019-04-16 13:26 ` [PATCH 5/8] bitbake.conf: remove unneeded and differently used variables claudius.heine.ext
@ 2019-04-16 13:26 ` claudius.heine.ext
2019-04-16 13:45 ` Andreas Reichel
2019-04-16 13:26 ` [PATCH 7/8] bitbake.conf: set default QEMU_ARCH variables claudius.heine.ext
` (2 subsequent siblings)
8 siblings, 1 reply; 15+ messages in thread
From: claudius.heine.ext @ 2019-04-16 13:26 UTC (permalink / raw)
To: isar-users; +Cc: Claudius Heine
From: Claudius Heine <ch@denx.de>
This way recipes that inherit image.bbclass can set the this path
themselves. This was previously not possible, because the bitbake.conf
set this variable with '='.
Signed-off-by: Claudius Heine <ch@denx.de>
---
meta/classes/image.bbclass | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/meta/classes/image.bbclass b/meta/classes/image.bbclass
index f9a3052..6881d37 100644
--- a/meta/classes/image.bbclass
+++ b/meta/classes/image.bbclass
@@ -5,7 +5,7 @@ PF = "${PN}-${DISTRO}-${MACHINE}"
IMAGE_INSTALL ?= ""
IMAGE_TYPE ?= "ext4-img"
-IMAGE_ROOTFS = "${WORKDIR}/rootfs"
+IMAGE_ROOTFS ?= "${WORKDIR}/rootfs"
IMAGE_INSTALL += "${@ ("linux-image-" + d.getVar("KERNEL_NAME", True)) if d.getVar("KERNEL_NAME", True) else ""}"
--
2.20.1
^ permalink raw reply [flat|nested] 15+ messages in thread* Re: [PATCH 6/8] image.bbclass: make IMAGE_ROOTFS overwritable
2019-04-16 13:26 ` [PATCH 6/8] image.bbclass: make IMAGE_ROOTFS overwritable claudius.heine.ext
@ 2019-04-16 13:45 ` Andreas Reichel
0 siblings, 0 replies; 15+ messages in thread
From: Andreas Reichel @ 2019-04-16 13:45 UTC (permalink / raw)
To: [ext] claudius.heine.ext@siemens.com; +Cc: isar-users, Claudius Heine
On Tue, Apr 16, 2019 at 03:26:44PM +0200, [ext] claudius.heine.ext@siemens.com wrote:
> From: Claudius Heine <ch@denx.de>
>
> This way recipes that inherit image.bbclass can set the this path
the this
> themselves. This was previously not possible, because the bitbake.conf
> set this variable with '='.
>
> Signed-off-by: Claudius Heine <ch@denx.de>
> ---
> meta/classes/image.bbclass | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta/classes/image.bbclass b/meta/classes/image.bbclass
> index f9a3052..6881d37 100644
> --- a/meta/classes/image.bbclass
> +++ b/meta/classes/image.bbclass
> @@ -5,7 +5,7 @@ PF = "${PN}-${DISTRO}-${MACHINE}"
>
> IMAGE_INSTALL ?= ""
> IMAGE_TYPE ?= "ext4-img"
> -IMAGE_ROOTFS = "${WORKDIR}/rootfs"
> +IMAGE_ROOTFS ?= "${WORKDIR}/rootfs"
>
> IMAGE_INSTALL += "${@ ("linux-image-" + d.getVar("KERNEL_NAME", True)) if d.getVar("KERNEL_NAME", True) else ""}"
>
> --
> 2.20.1
>
> --
> You received this message because you are subscribed to the Google Groups "isar-users" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to isar-users+unsubscribe@googlegroups.com.
> To post to this group, send email to isar-users@googlegroups.com.
> To view this discussion on the web visit https://groups.google.com/d/msgid/isar-users/20190416132647.3074-7-claudius.heine.ext%40siemens.com.
> For more options, visit https://groups.google.com/d/optout.
--
Andreas Reichel
Dipl.-Phys. (Univ.)
Software Consultant
Andreas.Reichel@tngtech.com, +49-174-3180074
TNG Technology Consulting GmbH, Betastr. 13a, 85774 Unterfoehring
Geschaeftsfuehrer: Henrik Klagges, Dr. Robert Dahlke, Gerhard Mueller
Sitz: Unterfoehring * Amtsgericht Muenchen * HRB 135082
^ permalink raw reply [flat|nested] 15+ messages in thread
* [PATCH 7/8] bitbake.conf: set default QEMU_ARCH variables
2019-04-16 13:26 [PATCH 0/8] Cleanup rootfs creation claudius.heine.ext
` (5 preceding siblings ...)
2019-04-16 13:26 ` [PATCH 6/8] image.bbclass: make IMAGE_ROOTFS overwritable claudius.heine.ext
@ 2019-04-16 13:26 ` claudius.heine.ext
2019-04-16 13:26 ` [PATCH 8/8] buildchroot/configscript: make creation of builder uid/gid idempotent claudius.heine.ext
2019-04-22 13:28 ` [PATCH 0/8] Cleanup rootfs creation Maxim Yu. Osipov
8 siblings, 0 replies; 15+ messages in thread
From: claudius.heine.ext @ 2019-04-16 13:26 UTC (permalink / raw)
To: isar-users; +Cc: Claudius Heine
From: Claudius Heine <ch@denx.de>
Signed-off-by: Claudius Heine <ch@denx.de>
---
meta/conf/bitbake.conf | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/meta/conf/bitbake.conf b/meta/conf/bitbake.conf
index 21d13ef..37f1963 100644
--- a/meta/conf/bitbake.conf
+++ b/meta/conf/bitbake.conf
@@ -53,6 +53,12 @@ CACHE = "${TMPDIR}/cache"
OVERRIDES = "${DISTRO_ARCH}:${MACHINE}:${DISTRO}:forcevariable"
+# Setting default QEMU_ARCH variables for different DISTRO_ARCH:
+QEMU_ARCH_amd64 = "x86_64"
+QEMU_ARCH_i386 = "i386"
+QEMU_ARCH_armhf = "arm"
+QEMU_ARCH_arm64 = "aarch64"
+
# Codename of the repository created by the caching class
DEBDISTRONAME = "isar"
--
2.20.1
^ permalink raw reply [flat|nested] 15+ messages in thread* [PATCH 8/8] buildchroot/configscript: make creation of builder uid/gid idempotent
2019-04-16 13:26 [PATCH 0/8] Cleanup rootfs creation claudius.heine.ext
` (6 preceding siblings ...)
2019-04-16 13:26 ` [PATCH 7/8] bitbake.conf: set default QEMU_ARCH variables claudius.heine.ext
@ 2019-04-16 13:26 ` claudius.heine.ext
2019-04-22 13:28 ` [PATCH 0/8] Cleanup rootfs creation Maxim Yu. Osipov
8 siblings, 0 replies; 15+ messages in thread
From: claudius.heine.ext @ 2019-04-16 13:26 UTC (permalink / raw)
To: isar-users; +Cc: Claudius Heine
From: Claudius Heine <ch@denx.de>
Signed-off-by: Claudius Heine <ch@denx.de>
---
meta/recipes-devtools/buildchroot/files/configscript.sh | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/meta/recipes-devtools/buildchroot/files/configscript.sh b/meta/recipes-devtools/buildchroot/files/configscript.sh
index 25a21ef..c4d8cf8 100644
--- a/meta/recipes-devtools/buildchroot/files/configscript.sh
+++ b/meta/recipes-devtools/buildchroot/files/configscript.sh
@@ -10,6 +10,6 @@ locales locales/locales_to_be_generated multiselect en_US.UTF-8 UTF-8
locales locales/default_environment_locale select en_US.UTF-8
END
-groupadd --system builder -o --gid $2
-useradd --system -o --uid $1 --gid builder --no-create-home --home /home/builder --no-user-group --comment "Isar buildchroot build user" builder
+grep -q "builder:x:$2" /etc/group || groupadd -f --system builder -o --gid $2
+grep -q "builder:x:$1" /etc/passwd || useradd --system -o --uid $1 --gid builder --no-create-home --home /home/builder --no-user-group --comment "Isar buildchroot build user" builder
chown -R builder:builder /home/builder
--
2.20.1
^ permalink raw reply [flat|nested] 15+ messages in thread* Re: [PATCH 0/8] Cleanup rootfs creation
2019-04-16 13:26 [PATCH 0/8] Cleanup rootfs creation claudius.heine.ext
` (7 preceding siblings ...)
2019-04-16 13:26 ` [PATCH 8/8] buildchroot/configscript: make creation of builder uid/gid idempotent claudius.heine.ext
@ 2019-04-22 13:28 ` Maxim Yu. Osipov
2019-04-24 7:20 ` Claudius Heine
8 siblings, 1 reply; 15+ messages in thread
From: Maxim Yu. Osipov @ 2019-04-22 13:28 UTC (permalink / raw)
To: claudius.heine.ext, isar-users; +Cc: Claudius Heine
[-- Attachment #1: Type: text/plain, Size: 5886 bytes --]
Hi,
Tried the series against the current 'next'.
Creation of SDK has been failed (my host is Debian 9) (see log below).
I've double checked - in the current 'next' 'bitbake -c do_populate_sdk
multiconfig:qemuarm-stretch:isar-image-base' works fine.
My local.conf is attached for convenience.
Maxim.
===============
bitbake -c do_populate_sdk multiconfig:qemuarm-stretch:isar-image-base
Parsing recipes: 100%
|#######################################################################################################################################################################################################|
Time: 0:00:02
Parsing of 26 .bb files complete (0 cached, 26 parsed). 390 targets, 0
skipped, 0 masked, 0 errors.
NOTE: Resolving any missing task queue dependencies
NOTE: Resolving any missing task queue dependencies
NOTE: Resolving any missing task queue dependencies
NOTE: Resolving any missing task queue dependencies
NOTE: Resolving any missing task queue dependencies
NOTE: Resolving any missing task queue dependencies
NOTE: Resolving any missing task queue dependencies
NOTE: Resolving any missing task queue dependencies
NOTE: Resolving any missing task queue dependencies
NOTE: Resolving any missing task queue dependencies
NOTE: Resolving any missing task queue dependencies
NOTE: Resolving any missing task queue dependencies
NOTE: Resolving any missing task queue dependencies
NOTE: Resolving any missing task queue dependencies
NOTE: Resolving any missing task queue dependencies
Initialising tasks: 100%
|####################################################################################################################################################################################################|
Time: 0:00:03
NOTE: Executing RunQueue Tasks
ERROR: mc:qemuarm-stretch:sdkchroot-${ROOTFS_DISTRO}-amd64-armhf
do_build: Function failed: do_build (log file is located at
/home/myo/work/isar/src/trunk/isar/build/tmp/work/debian-stretch-armhf/sdkchroot-${ROOTFS_DISTRO}-amd64-armhf/temp/log.do_build.24544)
ERROR: Logfile of failure stored in:
/home/myo/work/isar/src/trunk/isar/build/tmp/work/debian-stretch-armhf/sdkchroot-${ROOTFS_DISTRO}-amd64-armhf/temp/log.do_build.24544
Log data follows:
| DEBUG: Executing shell function do_build
| cp: cannot stat
'/home/myo/work/isar/src/trunk/isar/build/tmp/deploy/bootstrap/debian-stretch-amd64/':
No such file or directory
| WARNING: exit code 1 from a shell command.
| ERROR: Function failed: do_build (log file is located at
/home/myo/work/isar/src/trunk/isar/build/tmp/work/debian-stretch-armhf/sdkchroot-${ROOTFS_DISTRO}-amd64-armhf/temp/log.do_build.24544)
ERROR: Task
(multiconfig:qemuarm-stretch:/home/myo/work/isar/src/trunk/isar/meta/recipes-devtools/sdkchroot/sdkchroot.bb:do_build)
failed with exit code '1'
ERROR: mc:qemuarm-stretch:buildchroot-host-debian-stretch-amd64-armhf
do_build: Function failed: do_build (log file is located at
/home/myo/work/isar/src/trunk/isar/build/tmp/work/debian-stretch-armhf/buildchroot-host-debian-stretch-amd64-armhf/temp/log.do_build.24543)
ERROR: Logfile of failure stored in:
/home/myo/work/isar/src/trunk/isar/build/tmp/work/debian-stretch-armhf/buildchroot-host-debian-stretch-amd64-armhf/temp/log.do_build.24543
Log data follows:
| DEBUG: Executing shell function do_build
| cp: cannot stat
'/home/myo/work/isar/src/trunk/isar/build/tmp/deploy/bootstrap/debian-stretch-amd64/':
No such file or directory
| WARNING: exit code 1 from a shell command.
| ERROR: Function failed: do_build (log file is located at
/home/myo/work/isar/src/trunk/isar/build/tmp/work/debian-stretch-armhf/buildchroot-host-debian-stretch-amd64-armhf/temp/log.do_build.24543)
ERROR: Task
(multiconfig:qemuarm-stretch:/home/myo/work/isar/src/trunk/isar/meta/recipes-devtools/buildchroot/buildchroot-host.bb:do_build)
failed with exit code '1'
=====
On 4/16/19 3:26 PM, claudius.heine.ext@siemens.com wrote:
> From: Claudius Heine <ch@denx.de>
>
> Hi,
>
> this patchset contains some patches that where developed while implementing
> the preprocessing image pipeline. They are universally useful, but do prepare
> for the next steps.
>
> Cheers,
> Claudius
>
> Claudius Heine (8):
> isar-boostrap-helper: move 'HOST_ARCH' and 'HOST_DISTRO' to
> base.bbclass
> move 'HOST_DISTRO_APT_SOURCES' from bootstrap-helper to isar-bootstrap
> buildchroot.bbclass: only cross build if HOST_ARCH != DISTRO_ARCH
> isar-bootstrap/buildchroot/sdkchroot: refactor PF and WORKDIR
> bitbake.conf: remove unneeded and differently used variables
> image.bbclass: make IMAGE_ROOTFS overwritable
> bitbake.conf: set default QEMU_ARCH variables
> buildchroot/configscript: make creation of builder uid/gid idempotent
>
> meta/classes/base.bbclass | 9 +++++++++
> meta/classes/buildchroot.bbclass | 2 +-
> meta/classes/image-sdk-extension.bbclass | 2 +-
> meta/classes/image.bbclass | 2 +-
> meta/classes/isar-bootstrap-helper.bbclass | 14 --------------
> meta/conf/bitbake.conf | 13 ++++++++-----
> .../isar-bootstrap/isar-bootstrap-host.bb | 11 +++--------
> .../isar-bootstrap/isar-bootstrap-target.bb | 6 +-----
> .../recipes-core/isar-bootstrap/isar-bootstrap.inc | 1 +
> .../buildchroot/buildchroot-host.bb | 1 +
> meta/recipes-devtools/buildchroot/buildchroot.inc | 3 +--
> .../buildchroot/files/configscript.sh | 4 ++--
> meta/recipes-devtools/sdkchroot/sdkchroot.bb | 3 +--
> 13 files changed, 30 insertions(+), 41 deletions(-)
>
--
Maxim Osipov
ilbers GmbH
Maria-Merian-Str. 8
85521 Ottobrunn
Germany
+49 (151) 6517 6917
mosipov@ilbers.de
http://ilbers.de/
Commercial register Munich, HRB 214197
General Manager: Baurzhan Ismagulov
[-- Attachment #2: local.conf --]
[-- Type: text/plain, Size: 7090 bytes --]
#
# This file is your local configuration file and is where all local user settings
# are placed. The comments in this file give some guide to the options a new user
# to the system might want to change but pretty much any configuration option can
# be set in this file. More adventurous users can look at local.conf.extended
# which contains other examples of configuration which can be placed in this file
# but new users likely won't need any of them initially.
#
# Lines starting with the '#' character are commented out and in some cases the
# default values are provided as comments to show people example syntax. Enabling
# the option is a question of removing the # character and making any change to the
# variable as required.
#
# Machine Selection
#
# You need to select a specific machine to target the build with. There are a selection
# of emulated machines available which can boot and run in the QEMU emulator:
#
# This sets the default machine to be qemuarm if no other machine is selected:
MACHINE ??= "qemuarm"
#
# Isar Configuration Selection
#
# You need to select a specific distribution configuration which will used for both:
# generation of buildchroot environment and target root filesystem.
#
# This sets the default distribution configuration:
DISTRO ??= "debian-stretch"
DISTRO_ARCH ??= "armhf"
#
# Multiple Configuration Selection
#
# If you want to use multiple configuration files for the build, list them in the
# following option.
#
# This sets the default multiple configurations used:
BBMULTICONFIG = " \
qemuarm-stretch \
qemuarm-buster \
qemuarm64-stretch \
qemuarm64-buster \
qemui386-stretch \
qemui386-buster \
qemuamd64-stretch \
bananapi-stretch \
de0-nano-soc-stretch \
hikey-stretch \
qemuamd64-buster \
qemuamd64-buster-tgz \
nand-ubi-demo-buster \
rpi-stretch \
"
#
# Where to place downloads
#
# During a first build the system will download many different source code tarballs
# from various upstream projects. This can take a while, particularly if your network
# connection is slow. These are all stored in DL_DIR. When wiping and rebuilding you
# can preserve this directory to speed up this part of subsequent builds. This directory
# is safe to share between multiple builds on the same machine too.
#
# The default is a downloads directory under TOPDIR which is the build directory.
#
#DL_DIR ?= "${TOPDIR}/downloads"
#
# Where to place shared-state files
#
# BitBake has the capability to accelerate builds based on previously built output.
# This is done using "shared state" files which can be thought of as cache objects
# and this option determines where those files are placed.
#
# You can wipe out TMPDIR leaving this directory intact and the build would regenerate
# from these files if no changes were made to the configuration. If changes were made
# to the configuration, only shared state files where the state was still valid would
# be used (done using checksums).
#
# The default is a sstate-cache directory under TOPDIR.
#
#SSTATE_DIR ?= "${TOPDIR}/sstate-cache"
#
# Where to place the build output
#
# This option specifies where the bulk of the building work should be done and
# 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.
#
# The default is a tmp directory under TOPDIR.
#
#TMPDIR = "${TOPDIR}/tmp"
#
# Interactive shell configuration
#
# Under certain circumstances the system may need input from you and to do this it
# can launch an interactive shell. It needs to do this since the build is
# multithreaded and needs to be able to handle the case where more than one parallel
# process may require the user's attention. The default is iterate over the available
# terminal types to find one that works.
#
# Examples of the occasions this may happen are when resolving patches which cannot
# be applied, to use the devshell or the kernel menuconfig
#
# Supported values are auto, gnome, xfce, rxvt, screen, konsole (KDE 3.x only), none
# Note: currently, Konsole support only works for KDE 3.x due to the way
# newer Konsole versions behave
#OE_TERMINAL = "auto"
# By default disable interactive patch resolution (tasks will just fail instead):
PATCHRESOLVE = "noop"
#
# Disk Space Monitoring during the build
#
# Monitor the disk space during the build. If there is less that 1GB of space or less
# than 100K inodes in any key build location (TMPDIR, DL_DIR, SSTATE_DIR), gracefully
# shutdown the build. If there is less that 100MB or 1K inodes, perform a hard abort
# of the build. The reason for this is that running completely out of space can corrupt
# files and damages the build in ways which may not be easily recoverable.
# It's necesary to monitor /tmp, if there is no space left the build will fail
# with very exotic errors.
BB_DISKMON_DIRS = "\
STOPTASKS,${TMPDIR},1G,100K \
STOPTASKS,${DL_DIR},1G,100K \
STOPTASKS,${SSTATE_DIR},1G,100K \
STOPTASKS,/tmp,100M,100K \
ABORT,${TMPDIR},100M,1K \
ABORT,${DL_DIR},100M,1K \
ABORT,${SSTATE_DIR},100M,1K \
ABORT,/tmp,10M,1K"
#
# Shared-state files from other locations
#
# As mentioned above, shared state files are prebuilt cache data objects which can
# used to accelerate build time. This variable can be used to configure the system
# to search other mirror locations for these objects before it builds the data itself.
#
# This can be a filesystem directory, or a remote url such as http or ftp. These
# would contain the sstate-cache results from previous builds (possibly from other
# machines). This variable works like fetcher MIRRORS/PREMIRRORS and points to the
# cache locations to check for the shared objects.
# NOTE: if the mirror uses the same structure as SSTATE_DIR, you need to add PATH
# at the end as shown in the examples below. This will be substituted with the
# correct path within the directory structure.
#SSTATE_MIRRORS ?= "\
#file://.* http://someserver.tld/share/sstate/PATH;downloadfilename=PATH \n \
#file://.* file:///some/local/dir/sstate/PATH"
# CONF_VERSION is increased each time build/conf/ changes incompatibly and is used to
# track the version of this file when it was generated. This can safely be ignored if
# this doesn't mean anything to you.
CONF_VERSION = "1"
#
# The default list of extra packages to be installed.
IMAGE_INSTALL = "hello-isar example-raw example-module-${KERNEL_NAME} enable-fsck"
#
# Enable cross-compilation support
# NOTE: this works on build host >= stretch for armhf, arm64 and amd64 targets for now.
ISAR_CROSS_COMPILE ?= "1"
#
# Uncomment this to enable use of cached base repository
#ISAR_USE_CACHED_BASE_REPO ?= "1"
# Set root password to 'root'
# Password was encrypted using following command:
# mkpasswd -m sha512crypt -R 10000
# mkpasswd is part of the 'whois' package of Debian
CFG_ROOT_PW ?= "$6$rounds=10000$RXeWrnFmkY$DtuS/OmsAS2cCEDo0BF5qQsizIrq6jPgXnwv3PHqREJeKd1sXdHX/ayQtuQWVDHe0KIO0/sVH8dvQm1KthF0d/"
^ permalink raw reply [flat|nested] 15+ messages in thread* Re: [PATCH 0/8] Cleanup rootfs creation
2019-04-22 13:28 ` [PATCH 0/8] Cleanup rootfs creation Maxim Yu. Osipov
@ 2019-04-24 7:20 ` Claudius Heine
2019-04-24 7:45 ` Maxim Yu. Osipov
0 siblings, 1 reply; 15+ messages in thread
From: Claudius Heine @ 2019-04-24 7:20 UTC (permalink / raw)
To: Maxim Yu. Osipov, isar-users; +Cc: Claudius Heine
Hi Maxim,
On 22/04/2019 15.28, Maxim Yu. Osipov wrote:
> Hi,
>
> Tried the series against the current 'next'.
>
> Creation of SDK has been failed (my host is Debian 9) (see log below).
>
> I've double checked - in the current 'next' 'bitbake -c do_populate_sdk
> multiconfig:qemuarm-stretch:isar-image-base' works fine.
Thanks! That is a good find. That issue happened because I tried to
split up the pre-processing patchset.
Is that tested in CI?
Claudius
>
> My local.conf is attached for convenience.
>
> Maxim.
>
> ===============
> bitbake -c do_populate_sdk multiconfig:qemuarm-stretch:isar-image-base
> Parsing recipes: 100%
> |#######################################################################################################################################################################################################|
> Time: 0:00:02
> Parsing of 26 .bb files complete (0 cached, 26 parsed). 390 targets, 0
> skipped, 0 masked, 0 errors.
> NOTE: Resolving any missing task queue dependencies
> NOTE: Resolving any missing task queue dependencies
> NOTE: Resolving any missing task queue dependencies
> NOTE: Resolving any missing task queue dependencies
> NOTE: Resolving any missing task queue dependencies
> NOTE: Resolving any missing task queue dependencies
> NOTE: Resolving any missing task queue dependencies
> NOTE: Resolving any missing task queue dependencies
> NOTE: Resolving any missing task queue dependencies
> NOTE: Resolving any missing task queue dependencies
> NOTE: Resolving any missing task queue dependencies
> NOTE: Resolving any missing task queue dependencies
> NOTE: Resolving any missing task queue dependencies
> NOTE: Resolving any missing task queue dependencies
> NOTE: Resolving any missing task queue dependencies
> Initialising tasks: 100%
> |####################################################################################################################################################################################################|
> Time: 0:00:03
> NOTE: Executing RunQueue Tasks
> ERROR: mc:qemuarm-stretch:sdkchroot-${ROOTFS_DISTRO}-amd64-armhf
> do_build: Function failed: do_build (log file is located at
> /home/myo/work/isar/src/trunk/isar/build/tmp/work/debian-stretch-armhf/sdkchroot-${ROOTFS_DISTRO}-amd64-armhf/temp/log.do_build.24544)
>
> ERROR: Logfile of failure stored in:
> /home/myo/work/isar/src/trunk/isar/build/tmp/work/debian-stretch-armhf/sdkchroot-${ROOTFS_DISTRO}-amd64-armhf/temp/log.do_build.24544
>
> Log data follows:
> | DEBUG: Executing shell function do_build
> | cp: cannot stat
> '/home/myo/work/isar/src/trunk/isar/build/tmp/deploy/bootstrap/debian-stretch-amd64/':
> No such file or directory
> | WARNING: exit code 1 from a shell command.
> | ERROR: Function failed: do_build (log file is located at
> /home/myo/work/isar/src/trunk/isar/build/tmp/work/debian-stretch-armhf/sdkchroot-${ROOTFS_DISTRO}-amd64-armhf/temp/log.do_build.24544)
>
> ERROR: Task
> (multiconfig:qemuarm-stretch:/home/myo/work/isar/src/trunk/isar/meta/recipes-devtools/sdkchroot/sdkchroot.bb:do_build)
> failed with exit code '1'
> ERROR: mc:qemuarm-stretch:buildchroot-host-debian-stretch-amd64-armhf
> do_build: Function failed: do_build (log file is located at
> /home/myo/work/isar/src/trunk/isar/build/tmp/work/debian-stretch-armhf/buildchroot-host-debian-stretch-amd64-armhf/temp/log.do_build.24543)
>
> ERROR: Logfile of failure stored in:
> /home/myo/work/isar/src/trunk/isar/build/tmp/work/debian-stretch-armhf/buildchroot-host-debian-stretch-amd64-armhf/temp/log.do_build.24543
>
> Log data follows:
> | DEBUG: Executing shell function do_build
> | cp: cannot stat
> '/home/myo/work/isar/src/trunk/isar/build/tmp/deploy/bootstrap/debian-stretch-amd64/':
> No such file or directory
> | WARNING: exit code 1 from a shell command.
> | ERROR: Function failed: do_build (log file is located at
> /home/myo/work/isar/src/trunk/isar/build/tmp/work/debian-stretch-armhf/buildchroot-host-debian-stretch-amd64-armhf/temp/log.do_build.24543)
>
> ERROR: Task
> (multiconfig:qemuarm-stretch:/home/myo/work/isar/src/trunk/isar/meta/recipes-devtools/buildchroot/buildchroot-host.bb:do_build)
> failed with exit code '1'
>
> =====
>
>
> On 4/16/19 3:26 PM, claudius.heine.ext@siemens.com wrote:
>> From: Claudius Heine <ch@denx.de>
>>
>> Hi,
>>
>> this patchset contains some patches that where developed while
>> implementing
>> the preprocessing image pipeline. They are universally useful, but do
>> prepare
>> for the next steps.
>>
>> Cheers,
>> Claudius
>>
>> Claudius Heine (8):
>> isar-boostrap-helper: move 'HOST_ARCH' and 'HOST_DISTRO' to
>> base.bbclass
>> move 'HOST_DISTRO_APT_SOURCES' from bootstrap-helper to isar-bootstrap
>> buildchroot.bbclass: only cross build if HOST_ARCH != DISTRO_ARCH
>> isar-bootstrap/buildchroot/sdkchroot: refactor PF and WORKDIR
>> bitbake.conf: remove unneeded and differently used variables
>> image.bbclass: make IMAGE_ROOTFS overwritable
>> bitbake.conf: set default QEMU_ARCH variables
>> buildchroot/configscript: make creation of builder uid/gid idempotent
>>
>> meta/classes/base.bbclass | 9 +++++++++
>> meta/classes/buildchroot.bbclass | 2 +-
>> meta/classes/image-sdk-extension.bbclass | 2 +-
>> meta/classes/image.bbclass | 2 +-
>> meta/classes/isar-bootstrap-helper.bbclass | 14 --------------
>> meta/conf/bitbake.conf | 13 ++++++++-----
>> .../isar-bootstrap/isar-bootstrap-host.bb | 11 +++--------
>> .../isar-bootstrap/isar-bootstrap-target.bb | 6 +-----
>> .../recipes-core/isar-bootstrap/isar-bootstrap.inc | 1 +
>> .../buildchroot/buildchroot-host.bb | 1 +
>> meta/recipes-devtools/buildchroot/buildchroot.inc | 3 +--
>> .../buildchroot/files/configscript.sh | 4 ++--
>> meta/recipes-devtools/sdkchroot/sdkchroot.bb | 3 +--
>> 13 files changed, 30 insertions(+), 41 deletions(-)
>>
>
>
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-54 Fax: (+49)-8142-66989-80 Email: ch@denx.de
^ permalink raw reply [flat|nested] 15+ messages in thread* Re: [PATCH 0/8] Cleanup rootfs creation
2019-04-24 7:20 ` Claudius Heine
@ 2019-04-24 7:45 ` Maxim Yu. Osipov
0 siblings, 0 replies; 15+ messages in thread
From: Maxim Yu. Osipov @ 2019-04-24 7:45 UTC (permalink / raw)
To: Claudius Heine, isar-users; +Cc: Claudius Heine
Hi Claudius,
On 4/24/19 9:20 AM, Claudius Heine wrote:
> Hi Maxim,
>
> On 22/04/2019 15.28, Maxim Yu. Osipov wrote:
>> Hi,
>>
>> Tried the series against the current 'next'.
>>
>> Creation of SDK has been failed (my host is Debian 9) (see log below).
>>
>> I've double checked - in the current 'next' 'bitbake -c
>> do_populate_sdk multiconfig:qemuarm-stretch:isar-image-base' works fine.
>
> Thanks! That is a good find. That issue happened because I tried to
> split up the pre-processing patchset.
>
> Is that tested in CI?
Unfortunately at the moment no - I'll include it into CI.
Maxim.
> Claudius
>
>>
>> My local.conf is attached for convenience.
>>
>> Maxim.
>>
>> ===============
>> bitbake -c do_populate_sdk multiconfig:qemuarm-stretch:isar-image-base
>> Parsing recipes: 100%
>> |#######################################################################################################################################################################################################|
>> Time: 0:00:02
>> Parsing of 26 .bb files complete (0 cached, 26 parsed). 390 targets, 0
>> skipped, 0 masked, 0 errors.
>> NOTE: Resolving any missing task queue dependencies
>> NOTE: Resolving any missing task queue dependencies
>> NOTE: Resolving any missing task queue dependencies
>> NOTE: Resolving any missing task queue dependencies
>> NOTE: Resolving any missing task queue dependencies
>> NOTE: Resolving any missing task queue dependencies
>> NOTE: Resolving any missing task queue dependencies
>> NOTE: Resolving any missing task queue dependencies
>> NOTE: Resolving any missing task queue dependencies
>> NOTE: Resolving any missing task queue dependencies
>> NOTE: Resolving any missing task queue dependencies
>> NOTE: Resolving any missing task queue dependencies
>> NOTE: Resolving any missing task queue dependencies
>> NOTE: Resolving any missing task queue dependencies
>> NOTE: Resolving any missing task queue dependencies
>> Initialising tasks: 100%
>> |####################################################################################################################################################################################################|
>> Time: 0:00:03
>> NOTE: Executing RunQueue Tasks
>> ERROR: mc:qemuarm-stretch:sdkchroot-${ROOTFS_DISTRO}-amd64-armhf
>> do_build: Function failed: do_build (log file is located at
>> /home/myo/work/isar/src/trunk/isar/build/tmp/work/debian-stretch-armhf/sdkchroot-${ROOTFS_DISTRO}-amd64-armhf/temp/log.do_build.24544)
>>
>> ERROR: Logfile of failure stored in:
>> /home/myo/work/isar/src/trunk/isar/build/tmp/work/debian-stretch-armhf/sdkchroot-${ROOTFS_DISTRO}-amd64-armhf/temp/log.do_build.24544
>>
>> Log data follows:
>> | DEBUG: Executing shell function do_build
>> | cp: cannot stat
>> '/home/myo/work/isar/src/trunk/isar/build/tmp/deploy/bootstrap/debian-stretch-amd64/':
>> No such file or directory
>> | WARNING: exit code 1 from a shell command.
>> | ERROR: Function failed: do_build (log file is located at
>> /home/myo/work/isar/src/trunk/isar/build/tmp/work/debian-stretch-armhf/sdkchroot-${ROOTFS_DISTRO}-amd64-armhf/temp/log.do_build.24544)
>>
>> ERROR: Task
>> (multiconfig:qemuarm-stretch:/home/myo/work/isar/src/trunk/isar/meta/recipes-devtools/sdkchroot/sdkchroot.bb:do_build)
>> failed with exit code '1'
>> ERROR: mc:qemuarm-stretch:buildchroot-host-debian-stretch-amd64-armhf
>> do_build: Function failed: do_build (log file is located at
>> /home/myo/work/isar/src/trunk/isar/build/tmp/work/debian-stretch-armhf/buildchroot-host-debian-stretch-amd64-armhf/temp/log.do_build.24543)
>>
>> ERROR: Logfile of failure stored in:
>> /home/myo/work/isar/src/trunk/isar/build/tmp/work/debian-stretch-armhf/buildchroot-host-debian-stretch-amd64-armhf/temp/log.do_build.24543
>>
>> Log data follows:
>> | DEBUG: Executing shell function do_build
>> | cp: cannot stat
>> '/home/myo/work/isar/src/trunk/isar/build/tmp/deploy/bootstrap/debian-stretch-amd64/':
>> No such file or directory
>> | WARNING: exit code 1 from a shell command.
>> | ERROR: Function failed: do_build (log file is located at
>> /home/myo/work/isar/src/trunk/isar/build/tmp/work/debian-stretch-armhf/buildchroot-host-debian-stretch-amd64-armhf/temp/log.do_build.24543)
>>
>> ERROR: Task
>> (multiconfig:qemuarm-stretch:/home/myo/work/isar/src/trunk/isar/meta/recipes-devtools/buildchroot/buildchroot-host.bb:do_build)
>> failed with exit code '1'
>>
>> =====
>>
>>
>> On 4/16/19 3:26 PM, claudius.heine.ext@siemens.com wrote:
>>> From: Claudius Heine <ch@denx.de>
>>>
>>> Hi,
>>>
>>> this patchset contains some patches that where developed while
>>> implementing
>>> the preprocessing image pipeline. They are universally useful, but do
>>> prepare
>>> for the next steps.
>>>
>>> Cheers,
>>> Claudius
>>>
>>> Claudius Heine (8):
>>> isar-boostrap-helper: move 'HOST_ARCH' and 'HOST_DISTRO' to
>>> base.bbclass
>>> move 'HOST_DISTRO_APT_SOURCES' from bootstrap-helper to
>>> isar-bootstrap
>>> buildchroot.bbclass: only cross build if HOST_ARCH != DISTRO_ARCH
>>> isar-bootstrap/buildchroot/sdkchroot: refactor PF and WORKDIR
>>> bitbake.conf: remove unneeded and differently used variables
>>> image.bbclass: make IMAGE_ROOTFS overwritable
>>> bitbake.conf: set default QEMU_ARCH variables
>>> buildchroot/configscript: make creation of builder uid/gid idempotent
>>>
>>> meta/classes/base.bbclass | 9 +++++++++
>>> meta/classes/buildchroot.bbclass | 2 +-
>>> meta/classes/image-sdk-extension.bbclass | 2 +-
>>> meta/classes/image.bbclass | 2 +-
>>> meta/classes/isar-bootstrap-helper.bbclass | 14 --------------
>>> meta/conf/bitbake.conf | 13 ++++++++-----
>>> .../isar-bootstrap/isar-bootstrap-host.bb | 11 +++--------
>>> .../isar-bootstrap/isar-bootstrap-target.bb | 6 +-----
>>> .../recipes-core/isar-bootstrap/isar-bootstrap.inc | 1 +
>>> .../buildchroot/buildchroot-host.bb | 1 +
>>> meta/recipes-devtools/buildchroot/buildchroot.inc | 3 +--
>>> .../buildchroot/files/configscript.sh | 4 ++--
>>> meta/recipes-devtools/sdkchroot/sdkchroot.bb | 3 +--
>>> 13 files changed, 30 insertions(+), 41 deletions(-)
>>>
>>
>>
>
--
Maxim Osipov
ilbers GmbH
Maria-Merian-Str. 8
85521 Ottobrunn
Germany
+49 (151) 6517 6917
mosipov@ilbers.de
http://ilbers.de/
Commercial register Munich, HRB 214197
General Manager: Baurzhan Ismagulov
^ permalink raw reply [flat|nested] 15+ messages in thread