Hi Richard,

On 3/4/24 at 16:48, Richard Purdie wrote:
On Mon, 2024-03-04 at 16:34 +0100, Michael Opdenacker wrote:
Hi Richard

Many thanks for the patch!

On 3/4/24 at 16:14, Richard Purdie wrote:
Since we plan to have better "binary distro" testing on the autobuilder
and we plan to primarily use ipk for this, having ipk based images available
as part of our builds is desireable.

Since we want systemd based images with ipk, it makes sense to leave standard
poky using rpm images but switch poky-altcfg to use ipk.

Whilst late in the release cycle, doing this now for scarthgap gives us a good
platform to build off in the long term too.

Signed-off-by: Richard Purdie <richard.pur...@linuxfoundation.org>
---
   config.json | 5 +++++
   1 file changed, 5 insertions(+)

diff --git a/config.json b/config.json
index fd8335f..4d0f258 100644
--- a/config.json
+++ b/config.json
@@ -127,6 +127,7 @@
               "BUILDINFO" : true,
               "BUILDHISTORY" : true,
               "extravars" : [
+                "PACKAGE_CLASSES = 'package_ipk package_rpm package_deb'",
                   "IMAGE_INSTALL:append = ' ssh-pregen-hostkeys'"
               ],
               "extratools" : "${EXTRATOOLS_URL}",
@@ -140,6 +141,7 @@
               "BUILDINFO" : true,
               "BUILDHISTORY" : true,
               "extravars" : [
+                "PACKAGE_CLASSES = 'package_ipk package_rpm package_deb'",
                   "IMAGE_INSTALL:append = ' ssh-pregen-hostkeys'"
               ],
               "extratools" : "${EXTRATOOLS_URL}",
@@ -198,6 +200,9 @@
           "altcfg-hw" : {
               "DISTRO" : "poky-altcfg",
               "BUILDINFO" : true,
+            "extravars" : [
+                "PACKAGE_CLASSES = 'package_ipk package_rpm package_deb'"
+            ],
               "step1" : {
                   "BBTARGETS" : "core-image-full-cmdline core-image-sato 
core-image-sato-sdk"
               }

Actually, to use these images for testing package upgrades through
feeds, we also need enough free space in the images.
Could we also add something like IMAGE_ROOTFS_EXTRA_SPACE = "262144"?
Setting that "globally" for those templates will potentially break other
images the usual way to do it would be:

IMAGE_ROOTFS_EXTRA_SPACE:append = "+ 102400"

but I still worry about side effects, e.g. for minimal or initramfs images.
Which images do we plan to test? core-image-full-cmdline and what else?

We probably need a set of definitions like:

IMAGE_ROOTFS_EXTRA_SPACE:append:pn-core-image-full-cmdline = " + 262144"

but something like:

IMAGE_ROOTFS_EXTRA_SPACE:append = "${@bb.utils.contains("IMAGE_FEATURES, "package-management", " + 
262144", "", d)}"

might work.


Ouch, good catch.
I'd vote for the second option. When we have package management, it makes sense to keep enough space for downloading and applying updates.
Cheers
Michael.

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

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#62673): https://lists.yoctoproject.org/g/yocto/message/62673
Mute This Topic: https://lists.yoctoproject.org/mt/104723845/21656
Group Owner: yocto+ow...@lists.yoctoproject.org
Unsubscribe: https://lists.yoctoproject.org/g/yocto/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to