From 0916235adf36f5c7d3d4b2f613ef9ff8b690f983 Mon Sep 17 00:00:00 2001 From: Olof Johansson Date: Tue, 19 Jan 2016 14:45:32 +0100 Subject: rpm: remove bashisms: [ x == x ] -> [ x = x ] The postinst and postrm of rpm contained a bashism, that could in some situations potentially cause ldconfig not be triggered when it should be. If you use dash on host, test would fail because of syntax errors. But on host, it should fail because of the comparison. On target, you often use busybox ash, and it supports == as an alias for =. So in practice, only if you use a shell like dash on target, you'll run into issues with this. (From OE-Core rev: 4fd33ca846ba9f61e72a176a6462443c9fb5ddc2) Signed-off-by: Olof Johansson Signed-off-by: Richard Purdie --- meta/recipes-devtools/rpm/rpm_4.11.2.bb | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/meta/recipes-devtools/rpm/rpm_4.11.2.bb b/meta/recipes-devtools/rpm/rpm_4.11.2.bb index a48852f875..3683971cd5 100644 --- a/meta/recipes-devtools/rpm/rpm_4.11.2.bb +++ b/meta/recipes-devtools/rpm/rpm_4.11.2.bb @@ -101,8 +101,7 @@ do_install_append() { } pkg_postinst_${PN}() { - - [ "x\$D" == "x" ] && ldconfig + [ "x\$D" = "x" ] && ldconfig test -f ${localstatedir}/lib/rpm/Packages || rpm --initdb rm -f ${localstatedir}/lib/rpm/Filemd5s \ ${localstatedir}/lib/rpm/Filedigests \ @@ -112,7 +111,7 @@ pkg_postinst_${PN}() { } pkg_postrm_${PN}() { - [ "x\$D" == "x" ] && ldconfig + [ "x\$D" = "x" ] && ldconfig } -- cgit 1.2.3-korg