| Message ID | 20260329-fix-confusing-appends-v1-13-df7e87b8b879@gmail.com |
|---|---|
| State | Under Review |
| Headers | show |
| Series | Confusing and invalid conditional appends | expand |
diff --git a/meta-oe/recipes-devtools/flatbuffers/flatbuffers.bb b/meta-oe/recipes-devtools/flatbuffers/flatbuffers.bb index c51a49ba01..289330ee04 100644 --- a/meta-oe/recipes-devtools/flatbuffers/flatbuffers.bb +++ b/meta-oe/recipes-devtools/flatbuffers/flatbuffers.bb @@ -25,7 +25,7 @@ inherit cmake python3native rm_flatc_cmaketarget_for_target() { rm -f "${SYSROOT_DESTDIR}/${libdir}/cmake/flatbuffers/FlatcTargets.cmake" } -SYSROOT_PREPROCESS_FUNCS:class-target += "rm_flatc_cmaketarget_for_target" +SYSROOT_PREPROCESS_FUNCS:append:class-target = " rm_flatc_cmaketarget_for_target" FILES:${PN}-compiler = "${bindir}"
While in this case `SYSROOT_PREPROCESS_FUNCS: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-oe/recipes-devtools/flatbuffers/flatbuffers.bb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)