Re: [PATCH 1/1] expand last partition supports btrfs

[Henning Schild <henning.schild@siemens.com>]

Am Fri, 30 Sep 2022 19:37:09 +0200
schrieb "Roberto A. Foglietta" <roberto.foglietta@gmail.com>:

> Il giorno ven 30 set 2022 alle ore 13:42 Henning Schild <
> henning.schild@siemens.com> ha scritto:  
> 
> >
> > Feel free to apply that patch in your layer, you can likely bbappend
> > somehow and do not actually have to patch isar.
> >  
> 
> Geniale!

That sounds like the trick works for you. Make sure to revert it once
the systemd patches are merged and you switch to a new isar. But i
assume the version will be bumped so the append will tell you that it
does not work any longer.

> However the fact that DEBIAN_DEPENDS requires a comma separation that
> upsets me.

We once tried to make that more OE but that is hard. Here you just have
a string that makes it into the control file. It can contain things
like "or" operators and versions. Too debian-specific to find a good
way to apply OE standards on it.

> $ cat
> recipes-support/expand-on-first-boot/expand-on-first-boot_1.1.bbappend
> 
> DEBIAN_DEPENDS += ", btrfs-progs"
> 
> FILESEXTRAPATHS_prepend := "${THISDIR}:"
> 
> SRC_URI += "file://expand-last-partition.sh"
> 
> The second thing that upset me is that ${THISDIR}: search always on
> /debx86 which is the name of the layer

Mhh you layer.conf should have LAYERDIR_debx86 defined. And you can go

FILESEXTRAPATHS_prepend :=
"${LAYERDIR_debx86}/recipes-support/expand-on-first-boot/files:"

regards,
Henning

> /build/../repo/recipes-support/expand-on-first-boot/debx86
> 
> In case I use this one  ${THISDIR}/files: then the path would be
> debx86/files - mumble mumble
> 
> Thanks, R.