[OE-core] [PATCH v2] base.bbclass: add named SRCREVs to the sstate hash

2019-05-27 Thread Michael Ho
Several fetchers support named sources that require setting a SRCREV with the source name as a suffix. These named SRCREV variables are not captured in the sstate hash calculation because they're only referenced within the bitbake fetcher function. Several recipes avoid this bug by adding the

[OE-core] [PATCH] base.bbclass: add named SRCREVs to the sstate hash

2019-05-08 Thread Michael Ho
Several fetchers support named sources that require setting a SRCREV with the source name as a suffix. These named SRCREV variables are not captured in the sstate hash calculation because they're only referenced within the bitbake fetcher function. Add a snippet to the base.bbclass anonymous

[OE-core] cmake find_program and hosttools_dir

2019-08-14 Thread Michael Ho
his a good solution or is there something I'm missing here? Thanks. Kind regards, Michael Ho $ cat files/CMakeLists.txt cmake_minimum_required (VERSION 2.6) project (Tutorial) find_program(HOST_GIT git) if(NOT HOST_GIT) message(FATAL_ERROR "No git binary found") endif() $ cat

[OE-core] [PATCH] cmake.bbclass: add HOSTTOOLS_DIR to CMAKE_FIND_ROOT_PATH

2019-08-14 Thread Michael Ho
The find_program command will fail if it is used on a tool that is listed in ASSUME_PROVIDED. This is because these tools are in the hosttools directory which is not listed in CMAKE_FIND_ROOT_PATH so cmake will not find them. Adding the directory HOSTTOOLS_DIR to the CMAKE_FIND_ROOT_PATH variable

Re: [OE-core] [PATCH] package_rpm.bbclass: respect package overrides for the main package

2020-05-28 Thread Michael Ho
there are in place in the project. Should I in the future address patches to the dunfell branch? (Where might I find the guidelines for submitting to dunfell vs master). Thanks. Kind regards, Michael -- BMW Car IT GmbH Michael Ho Spezialist Entwicklung – Build and Release Engineering Lise

[OE-core] [PATCH 1/5] package.bbclass: add PACKAGE_ADD_METADATA to sstate variables

2020-05-25 Thread Michael Ho
From: Michael Ho The PACKAGE_ADD_METADATA can be used to define additional metadata for packages. Changes to this variable should affect the packaging sstate hash so packages are re-generated when this variable changes. This variable is added to both PKGDATA_VARS and PACKAGEVARS. It is needed

[OE-core] [PATCH 3/5] package_deb.bbclass: add PACKAGE_ADD_METADATA_DEB to sstate variables

2020-05-25 Thread Michael Ho
From: Michael Ho Refactor the addition of vardeps for this package class to use DEBEXTRAVARS and pass the items through gen_packagevar so package name specific overrides are accounted for. This matches the style used in package_ipk.bbclass. Additionally add PACKAGE_ADD_METADATA_DEB

[OE-core] [PATCH 2/5] package_ipk.bbclass: add PACKAGE_ADD_METADATA_IPK to sstate variables

2020-05-25 Thread Michael Ho
From: Michael Ho Add PACKAGE_ADD_METADATA_IPK to IPKEXTRAVARS so it affects the sstate hash of do_package_write_ipk. Signed-off-by: Michael Ho --- meta/classes/package_ipk.bbclass | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/meta/classes/package_ipk.bbclass b/meta

[OE-core] [PATCH 4/5] package_rpm.bbclass: add PACKAGE_ADD_METADATA_RPM to sstate variables

2020-05-25 Thread Michael Ho
From: Michael Ho Refactor the addition of vardeps for this package class to use RPMEXTRAVARS and pass the items through gen_packagevar so package name specific overrides are accounted for. This matches the style used in package_ipk.bbclass. Since do_package_write_rpm does not have any "

[OE-core] [PATCH 5/5] ref-manual: add PACKAGE_ADD_METADATA documentation

2020-05-25 Thread Michael Ho
From: Michael Ho Add a basic variable definition and a small section to the development tasks manual for using PACKAGE_ADD_METADATA to add custom metadata to packages. Signed-off-by: Michael Ho --- .../dev-manual/dev-manual-common-tasks.xml | 58 ++ documentation

[OE-core] [PATCH] package_rpm.bbclass: respect package overrides for the main package

2020-05-25 Thread Michael Ho
From: Michael Ho Apply ${PN} to OVERRIDES when determining the base package spec variables. Without this, there is a mismatch in behaviour where overrides for the base package have no effect while overrides for subpackages do. The subpackages should not be affected by this override as they apply

[OE-core] [PATCH v2 2/4] package_ipk.bbclass: add PACKAGE_ADD_METADATA_IPK to sstate variables

2020-05-25 Thread Michael Ho
From: Michael Ho Add PACKAGE_ADD_METADATA_IPK to IPKEXTRAVARS so it affects the sstate hash of do_package_write_ipk. Signed-off-by: Michael Ho --- meta/classes/package_ipk.bbclass | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/meta/classes/package_ipk.bbclass b/meta

[OE-core] [PATCH v2 4/4] package_rpm.bbclass: add PACKAGE_ADD_METADATA_RPM to sstate variables

2020-05-25 Thread Michael Ho
From: Michael Ho Refactor the addition of vardeps for this package class to use RPMEXTRAVARS and pass the items through gen_packagevar so package name specific overrides are accounted for. This matches the style used in package_ipk.bbclass. Since do_package_write_rpm does not have any "

[OE-core] [PATCH v2 3/4] package_deb.bbclass: add PACKAGE_ADD_METADATA_DEB to sstate variables

2020-05-25 Thread Michael Ho
From: Michael Ho Refactor the addition of vardeps for this package class to use DEBEXTRAVARS and pass the items through gen_packagevar so package name specific overrides are accounted for. This matches the style used in package_ipk.bbclass. Additionally add PACKAGE_ADD_METADATA_DEB

[OE-core] [PATCH v2 1/4] package.bbclass: add PACKAGE_ADD_METADATA to sstate variables

2020-05-25 Thread Michael Ho
From: Michael Ho The PACKAGE_ADD_METADATA can be used to define additional metadata for packages. Changes to this variable should affect the packaging sstate hash so packages are re-generated when this variable changes. This variable is added to both PKGDATA_VARS and PACKAGEVARS. It is needed

Re: [OE-core] [meta-oe][PATCH] 2/2] package_{ipk, deb, rpm}.bbclass: support per package user-defined metadata.

2020-05-22 Thread Michael Ho
Hi Paulo, I actually have some patches in my "to send upstream list" to address the documentation and some sstate-caching bugs with these variables so it's fresh to me at the moment. Correct me if I'm wrong but I think this is already available in the code (applying metadata specific to a certain

Re: [OE-core] [meta-oe][PATCH] 2/2] package_{ipk, deb, rpm}.bbclass: support per package user-defined metadata.

2020-05-22 Thread Michael Ho
d.setVarFlag(key, "separator", "\\n") > metadata_fields = [field.strip() for field in > oe.data.typed_value(key, d)] > return "\n".join(metadata_fields).strip() > > From my understanding, i do not see how the per package override &g

[OE-core] [PATCH] rootfs_ipk: allow do_populate_sdk in parallel to do_rootfs

2021-01-11 Thread Michael Ho
From: Michael Ho Switch do_populate_sdk for the ipk package manager to use a separate target opkg config file and separate the lockfiles restricting do_rootfs and do_populate_sdk from running in parallel. This way if an image recipe includes a dependency to do_populate_sdk by default

Re: [OE-core] [PATCH] rootfs_ipk: allow do_populate_sdk in parallel to do_rootfs

2021-01-11 Thread Michael Ho
-linux/core-image-minimal/1.0-r0/recipe-sysroot-native/var/cache/apt/archives/lock - open (11: Resource temporarily unavailable) -- BMW Car IT GmbH Michael Ho Spezialist Entwicklung – Build and Release Engineering Lise-Meitner-Straße 14 89081 Ulm Tel.: ­+49-731-37804-071 Mobil: +49-152-54980-471 Fax

[OE-core] [dunfell][PATCH] rootfs_ipk: allow do_populate_sdk in parallel to do_rootfs

2021-01-11 Thread Michael Ho
From: Michael Ho Switch do_populate_sdk for the ipk package manager to use a separate target opkg config file and separate the lockfiles restricting do_rootfs and do_populate_sdk from running in parallel. This way if an image recipe includes a dependency to do_populate_sdk by default

Re: [OE-core] [PATCH] rootfs_ipk: allow do_populate_sdk in parallel to do_rootfs

2021-01-11 Thread Michael Ho
Hello Alexander, Did a quick test and it seems the rpm packaging classes don’t have this issue. The rootfs_deb bbclass however has the same lock files like with rootfs_ipk. Should I try to patch this too to keep them aligned? Kind regards, Michael -- BMW Car IT GmbH Michael Ho Spezialist

[OE-core] [PATCH] license_image.bbclass: fix missing recipeinfo on self

2021-01-12 Thread Michael Ho
From: Michael Ho Resolve a build bug where image recipes with a do_deploy task will fail. If the image recipe inheriting license_image.bbclass has a deploy task, then the function get_deployed_dependencies will add itself to the list of recipes to get license information for. However, image

Re: [OE-core] Question about security_flags.inc and CC_ARCH

2021-01-25 Thread Michael Ho
advice! Kind regards, Michael Ho -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#147220): https://lists.openembedded.org/g/openembedded-core/message/147220 Mute This Topic: https://lists.openembedded.org/mt/79701669/21656 Group Owner: openembedded

[OE-core] [pseudo][PATCH] pseudo_client.c: Rebuild passwd paths after chroot

2021-01-26 Thread Michael Ho
From: Michael Ho Pseudo will calculate the search paths used for passwd operations such as lckpwdf and ulckpwdf using build_passwd_paths when it initiates. This takes into account the chroot at the time. The problem is that after a chroot is performed, pseudo continues to use the search paths

[OE-core] Question about security_flags.inc and CC_ARCH

2021-01-15 Thread Michael Ho
and CXXFLAGS. Is that right? Would there be some objection to moving the security flags to CFLAGS/CXXFLAGS for the cross-canadian target (sdk)? Thanks for any insights people can share! Kind regards, Michael Ho [1] https://wiki.debian.org/Hardening#Using_Hardening_Options

Re: [OE-core] [PATCH] rootfs_ipk: allow do_populate_sdk in parallel to do_rootfs

2021-01-12 Thread Michael Ho
and try running that a few hundred times? Kind regards, Michael -- BMW Car IT GmbH Michael Ho Spezialist Entwicklung – Build and Release Engineering Lise-Meitner-Straße 14 89081 Ulm Tel.: ­+49-731-37804-071 Mobil: +49-152-54980-471 Fax: +49-731-37804-001 Mail: michael...@bmw-carit.de Web: http

Re: [OE-core] [PATCH] license_image: Introduce SKIP_LICENSE_MANIFEST_RECIPES variable to prevent from trying to use license recipeinfo file from image recipes

2021-02-04 Thread Michael Ho
a license) then my patch should probably be skipped over for yours. Kind regards, Michael Ho -- BMW Car IT GmbH Michael Ho Spezialist Entwicklung – Build and Release Engineering Lise-Meitner-Straße 14 89081 Ulm Tel.: ­+49-731-37804-071 Mobil: +49-152-54980-471 Fax: +49-731-37804-001 Mail: michael

Re: [OE-core] [PATCH] license_image: Introduce SKIP_LICENSE_MANIFEST_RECIPES variable to prevent from trying to use license recipeinfo file from image recipes

2021-02-04 Thread Michael Ho
(Sorry, didn't read your patch correctly, it already included my change). I guess I mean should my change (not dep[0] == pn) be removed and yours take its place as yours should cover that case. Kind regards, Michael Ho -- BMW Car IT GmbH Michael Ho Spezialist Entwicklung – Build and Release

[OE-core] [PATCH] sstate.bbclass: fix errors about read-only sstate mirrors

2021-06-24 Thread Michael Ho
From: Michael Ho If a read-only sstate mirror is used in conjunction with hash equiv, then OSError will be raised when an sstate-cache hit is achieved. This is because sstate_task_postfunc will try to "touch" the symlinks that point to the read-only sstate mirror when sstate_report_u

[OE-core] [PATCH v2] sstate.bbclass: fix errors about read-only sstate mirrors

2021-06-28 Thread Michael Ho
From: Michael Ho If a read-only sstate mirror is used in conjunction with hash equiv, then OSError will be raised when an sstate-cache hit is achieved. This is because sstate_task_postfunc will try to "touch" the symlinks that point to the read-only sstate mirror when sstate_report_u

[OE-core] [PATCH] rootfs.py: expose prepared_index and ipk_repo_workdir for OpkgPM

2021-02-05 Thread Michael Ho
From: Michael Ho The OpkgPM class has several constructor options that allows for skipping the ipk repository set up and controlling the ipk repository directory. This commit exposes these option via the OpkgRootfs class so others can make use of them if they wish to. Adds the bitbake variables

[OE-core] [PATCH] RFC: rootfs.py: expose prepared_index and ipk_repo_workdir for OpkgPM

2021-02-05 Thread Michael Ho
From: Michael Ho Hi, The OpkgPM class has two arguments for advanced control over the ipk repository creation/use that I would like to make use of from outside of OpkgRootfs (which wraps OpkgPM). This is for purposes beyond just standard image generation so it's more of an advanced interface

Re: [OE-core] [PATCH] rootfs.py: expose prepared_index and ipk_repo_workdir for OpkgPM

2021-02-05 Thread Michael Ho
Hi RP, Thanks for the comments. Sounds good to me. I'll send another patchset. I'll see if I can also set up a renaming patch for the other IPKG variables for review comments. Kind regards, Michael -- BMW Car IT GmbH Michael Ho Spezialist Entwicklung – Build and Release Engineering Lise