| Message ID | 20260329-fix-confusing-appends-v1-6-df7e87b8b879@gmail.com |
|---|---|
| State | Under Review |
| Headers | show |
| Series | Confusing and invalid conditional appends | expand |
diff --git a/meta-networking/recipes-daemons/postfix/postfix_3.10.8.bb b/meta-networking/recipes-daemons/postfix/postfix_3.10.8.bb index 2423e17169..a66ecc0b5d 100644 --- a/meta-networking/recipes-daemons/postfix/postfix_3.10.8.bb +++ b/meta-networking/recipes-daemons/postfix/postfix_3.10.8.bb @@ -240,8 +240,8 @@ pkg_postinst_ontarget:${PN}-cfg () { } PACKAGES =+ "${PN}-cfg" -RDEPENDS:${PN}-cfg:class-target += "${PN}-bin" -RDEPENDS:${PN}:class-target += "${PN}-cfg" +RDEPENDS:${PN}-cfg:append:class-target = " ${PN}-bin" +RDEPENDS:${PN}:append:class-target = " ${PN}-cfg" # Exclude .debug directories from the main package FILES:${PN}-bin += "${localstatedir} ${bindir}/* ${sbindir}/* \ ${libexecdir}/* ${systemd_unitdir}/*"
While in this case `RDEPENDS:class-target +=` wouldn't result in any unwanted override, there is no guarantee there won't be a change, which would be hidden by this override. To avoid any surprises in the future let's use `:append:class-target =` syntax here. Signed-off-by: Michal Sieron <michalwsieron@gmail.com> --- meta-networking/recipes-daemons/postfix/postfix_3.10.8.bb | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)