meta-fii: meta-mori: Launch apps by correct path to avoid anti-pattern
As specified by OpenBMC anti-pattern document, calls to a binary/script
should be the fully qualified path to the application being launched.
https://github.com/openbmc/docs/blob/717d273cf8bceac3868a278aeb34d19e8d74c1b3/anti-patterns.md#resolution-4
This follows the systemd recommendation to use the full
path to avoid ambiguity.
https://www.freedesktop.org/software/systemd/man/systemd.service.html#:~:text=Using%20an%20absolute%20path%20is%20recommended%20to%20avoid%20ambiguity
In addition, OpenBMC applications that are intended to be used by users
should be located in /usr/bin.
Changes:
Place user scripts in /usr/bin instead of /usr/sbin. Start scripts by
their absolute path in service files.
Testing:
The services that are modified can find and start their scripts. The
scripts that had their locations modified can be called by the user
without providing their full path.
Change-Id: Ibe7fbb1644aa1b0df0f296ea1f1a8ceaf444fe35
Signed-off-by: Kyle Nieman <kyle.nieman@fii-na.com>
diff --git a/meta-fii/meta-mori/recipes-mori/mori-fw-utility/mori-bios-update/phosphor-ipmi-flash-bios-update.service b/meta-fii/meta-mori/recipes-mori/mori-fw-utility/mori-bios-update/phosphor-ipmi-flash-bios-update.service
index bf819a3..6f75c28 100644
--- a/meta-fii/meta-mori/recipes-mori/mori-fw-utility/mori-bios-update/phosphor-ipmi-flash-bios-update.service
+++ b/meta-fii/meta-mori/recipes-mori/mori-fw-utility/mori-bios-update/phosphor-ipmi-flash-bios-update.service
@@ -2,5 +2,5 @@
Description=Phosphor-ipmi-flash update BIOS firmware service
[Service]
-ExecStart=mori-fw.sh bios /run/initramfs/bios-image
+ExecStart=/usr/bin/mori-fw.sh bios /run/initramfs/bios-image
Type=oneshot
diff --git a/meta-fii/meta-mori/recipes-mori/mori-fw-utility/mori-cpld-update/phosphor-ipmi-flash-bmccpld-update.service b/meta-fii/meta-mori/recipes-mori/mori-fw-utility/mori-cpld-update/phosphor-ipmi-flash-bmccpld-update.service
index 2ed0e2f..0e03f6f 100644
--- a/meta-fii/meta-mori/recipes-mori/mori-fw-utility/mori-cpld-update/phosphor-ipmi-flash-bmccpld-update.service
+++ b/meta-fii/meta-mori/recipes-mori/mori-fw-utility/mori-cpld-update/phosphor-ipmi-flash-bmccpld-update.service
@@ -2,7 +2,7 @@
Description=Phosphor-ipmi-flash update BMC CPLD firmware service
[Service]
-ExecStart=mori-fw.sh bmccpld /run/initramfs/bmc-cpld.svf
+ExecStart=/usr/bin/mori-fw.sh bmccpld /run/initramfs/bmc-cpld.svf
Type=oneshot
[Install]
diff --git a/meta-fii/meta-mori/recipes-mori/mori-fw-utility/mori-cpld-update/phosphor-ipmi-flash-mbcpld-update.service b/meta-fii/meta-mori/recipes-mori/mori-fw-utility/mori-cpld-update/phosphor-ipmi-flash-mbcpld-update.service
index 0036606..20f6c51 100644
--- a/meta-fii/meta-mori/recipes-mori/mori-fw-utility/mori-cpld-update/phosphor-ipmi-flash-mbcpld-update.service
+++ b/meta-fii/meta-mori/recipes-mori/mori-fw-utility/mori-cpld-update/phosphor-ipmi-flash-mbcpld-update.service
@@ -2,7 +2,7 @@
Description=Phosphor-ipmi-flash update MB CPLD firmware service
[Service]
-ExecStart=mori-fw.sh mbcpld /run/initramfs/mb-cpld.svf
+ExecStart=/usr/bin/mori-fw.sh mbcpld /run/initramfs/mb-cpld.svf
Type=oneshot
[Install]
diff --git a/meta-fii/meta-mori/recipes-mori/mori-fw-utility/mori-fw.bb b/meta-fii/meta-mori/recipes-mori/mori-fw-utility/mori-fw.bb
index 71c69b0..4121135 100644
--- a/meta-fii/meta-mori/recipes-mori/mori-fw-utility/mori-fw.bb
+++ b/meta-fii/meta-mori/recipes-mori/mori-fw-utility/mori-fw.bb
@@ -17,9 +17,9 @@
inherit systemd obmc-phosphor-systemd
do_install () {
- install -d ${D}${sbindir}
+ install -d ${D}${bindir}
install -d ${D}${libexecdir}/${PN}
- install -m 0755 ${WORKDIR}/mori-fw.sh ${D}${sbindir}/mori-fw.sh
+ install -m 0755 ${WORKDIR}/mori-fw.sh ${D}${bindir}/mori-fw.sh
install -m 0755 ${WORKDIR}/mori-fw-ver.sh \
${D}${libexecdir}/${PN}/mori-fw-ver.sh
install -m 0755 ${WORKDIR}/mori-lib.sh ${D}${libexecdir}/${PN}/mori-lib.sh
diff --git a/meta-fii/meta-mori/recipes-mori/mori-sys-utility/mori-cmd.bb b/meta-fii/meta-mori/recipes-mori/mori-sys-utility/mori-cmd.bb
index 3fb90a2..a664d78 100644
--- a/meta-fii/meta-mori/recipes-mori/mori-sys-utility/mori-cmd.bb
+++ b/meta-fii/meta-mori/recipes-mori/mori-sys-utility/mori-cmd.bb
@@ -7,8 +7,8 @@
SRC_URI = "file://mori.sh"
do_install () {
- install -d ${D}${sbindir}
- install -m 0755 ${WORKDIR}/mori.sh ${D}${sbindir}/mori.sh
+ install -d ${D}${bindir}
+ install -m 0755 ${WORKDIR}/mori.sh ${D}${bindir}/mori.sh
}
RDEPENDS:${PN}:append = " bash"