summaryrefslogtreecommitdiffstats
path: root/meta-phosphor/recipes-phosphor/state
diff options
context:
space:
mode:
authorBrad Bishop <bradleyb@fuzziesquirrel.com>2019-03-22 23:41:42 -0400
committerBrad Bishop <bradleyb@fuzziesquirrel.com>2019-03-28 14:44:54 -0400
commit44f3cceaa77d532e9475eafb2a6d4e83bc00fc4b (patch)
tree1dcd7e9fb1a0624a63b30cd14b84124baf670926 /meta-phosphor/recipes-phosphor/state
parentd108709597c5981f1f13c4729ff6644f73a93197 (diff)
downloadtalos-openbmc-44f3cceaa77d532e9475eafb2a6d4e83bc00fc4b.tar.gz
talos-openbmc-44f3cceaa77d532e9475eafb2a6d4e83bc00fc4b.zip
meta-phosphor: Fix explicit dependencies antipattern
Out of the box bitbake examines built applications, automatically adds runtime dependencies and thus ensures any library packages dependencies are automatically added to images, sdks, etc. There is no need to list them explicitly in a recipe. Dependencies change over time, and listing them explicitly is likely prone to errors - the net effect being unnecessary shared library packages being installed into images. Consult https://www.yoctoproject.org/docs/latest/mega-manual/mega-manual.html#var-RDEPENDS for information on when to use explicit runtime dependencies. (From meta-phosphor rev: 26873c99d30b55e21a920c9851f21c3e2eab19f7) Change-Id: I38640d75a2d834c07d45ddcdb0162f1a23f848d9 Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
Diffstat (limited to 'meta-phosphor/recipes-phosphor/state')
-rw-r--r--meta-phosphor/recipes-phosphor/state/phosphor-state-manager_git.bb9
1 files changed, 1 insertions, 8 deletions
diff --git a/meta-phosphor/recipes-phosphor/state/phosphor-state-manager_git.bb b/meta-phosphor/recipes-phosphor/state/phosphor-state-manager_git.bb
index f25117faf..5408b4750 100644
--- a/meta-phosphor/recipes-phosphor/state/phosphor-state-manager_git.bb
+++ b/meta-phosphor/recipes-phosphor/state/phosphor-state-manager_git.bb
@@ -41,15 +41,8 @@ DEPENDS += "sdbusplus"
DEPENDS += "sdeventplus"
DEPENDS += "phosphor-logging"
DEPENDS += "phosphor-dbus-interfaces"
+DEPENDS += "systemd"
DEPENDS += "libcereal"
-RDEPENDS_${PN} += "sdbusplus"
-
-RDEPENDS_${PN}-host += "libsystemd phosphor-dbus-interfaces"
-RDEPENDS_${PN}-chassis += "libsystemd phosphor-dbus-interfaces"
-RDEPENDS_${PN}-bmc += "libsystemd phosphor-dbus-interfaces"
-RDEPENDS_${PN}-discover += "libsystemd phosphor-dbus-interfaces"
-RDEPENDS_${PN}-host-check += "libsystemd phosphor-dbus-interfaces"
-RDEPENDS_${PN}-reset-sensor-states += "libsystemd phosphor-dbus-interfaces"
FILES_${PN}-host = "${sbindir}/phosphor-host-state-manager"
DBUS_SERVICE_${PN}-host += "xyz.openbmc_project.State.Host.service"
OpenPOWER on IntegriCloud