diff mbox series

[meta-rockchip] fix sed invocation

Message ID 20251104174516.31259-1-twoerner@gmail.com
State New
Headers show
Series [meta-rockchip] fix sed invocation | expand

Commit Message

Trevor Woerner Nov. 4, 2025, 5:45 p.m. UTC
"sed -ie" is different than "sed -i -e". Although both run the script on
the intended file in-place, the first one creates a backup. These two
were accidentally merged into one option which can sometimes work, but
not in this case.

Signed-off-by: Trevor Woerner <twoerner@gmail.com>
---
 .../rk-rauc-demo/recipes-core/rauc/rauc-conf.bbappend           | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Quentin Schulz Nov. 4, 2025, 5:56 p.m. UTC | #1
Hi Trevor,

On 11/4/25 6:45 PM, Trevor Woerner via lists.yoctoproject.org wrote:
> "sed -ie" is different than "sed -i -e". Although both run the script on

Nice catch!

> the intended file in-place, the first one creates a backup. These two
> were accidentally merged into one option which can sometimes work, but
> not in this case.
> 

Reviewed-by: Quentin Schulz <quentin.schulz@cherry.de>

Thanks!
Quentin
diff mbox series

Patch

diff --git a/dynamic-layers/rk-rauc-demo/recipes-core/rauc/rauc-conf.bbappend b/dynamic-layers/rk-rauc-demo/recipes-core/rauc/rauc-conf.bbappend
index 323b63b01f40..82dec84a8d5c 100644
--- a/dynamic-layers/rk-rauc-demo/recipes-core/rauc/rauc-conf.bbappend
+++ b/dynamic-layers/rk-rauc-demo/recipes-core/rauc/rauc-conf.bbappend
@@ -5,5 +5,5 @@  PACKAGE_ARCH:rk-rauc-demo = "${MACHINE_ARCH}"
 FILESEXTRAPATHS:prepend:rk-rauc-demo := "${THISDIR}/files:"
 
 do_install:prepend:rk-rauc-demo() {
-	sed -ie 's!@MACHINE@!${MACHINE}!g' ${UNPACKDIR}/system.conf
+	sed -i -e 's!@MACHINE@!${MACHINE}!g' ${UNPACKDIR}/system.conf
 }