summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPatrick Williams <patrick@stwcx.xyz>2017-02-24 13:04:06 -0600
committerPatrick Williams <patrick@stwcx.xyz>2017-02-24 13:07:48 -0600
commit43b61b7029b4e566766ee8c36b59efe826326d74 (patch)
treeceeb616ac7a0e9378caeec41bfcf4b1f145179cb
parentfeb59cb0723d9b3b0afb27e05869fa6d2a42bc6c (diff)
downloadtalos-openbmc-43b61b7029b4e566766ee8c36b59efe826326d74.tar.gz
talos-openbmc-43b61b7029b4e566766ee8c36b59efe826326d74.zip
ipmi-whitelist: fix compile issue
ipmi-whitelists were not working correctly when restored from sstate due to incorrect path locations in the bbclass. Files being installed with the do_install step were not associated with the package because they were placed in the wrong directory. By placing in the right directory, they are placed into the sstate file and restored correctly. The files were also installed into /etc and move to /usr/share. This matches what is being done for the mrw files (see phosphor-ipmi-fru.bbclass for example). Change-Id: I1881d383d4d9d7d3098411fa83c85e49c3a08560 Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
-rw-r--r--meta-phosphor/classes/phosphor-ipmi-host-whitelist.bbclass4
-rw-r--r--meta-phosphor/common/recipes-phosphor/ipmi/phosphor-ipmi-host.bb2
-rw-r--r--meta-phosphor/common/recipes-phosphor/packagegroups/packagegroup-obmc-ipmid-providers.bb12
3 files changed, 3 insertions, 15 deletions
diff --git a/meta-phosphor/classes/phosphor-ipmi-host-whitelist.bbclass b/meta-phosphor/classes/phosphor-ipmi-host-whitelist.bbclass
index a13b0b157..f2b041fe9 100644
--- a/meta-phosphor/classes/phosphor-ipmi-host-whitelist.bbclass
+++ b/meta-phosphor/classes/phosphor-ipmi-host-whitelist.bbclass
@@ -2,7 +2,7 @@
# packages
WHITELIST ?= "${PN}.conf"
-INSTALLDIR = "${sysconfdir}/phosphor-ipmi-host/"
+INSTALLDIR = "${D}${datadir}/phosphor-ipmi-host/"
python phosphor_ipmi_host_whitelist_postinstall() {
def install_whitelist(d):
@@ -24,4 +24,4 @@ python phosphor_ipmi_host_whitelist_postinstall() {
install_whitelist(d)
}
-do_populate_sysroot[postfuncs] += "phosphor_ipmi_host_whitelist_postinstall"
+do_install[postfuncs] += "phosphor_ipmi_host_whitelist_postinstall"
diff --git a/meta-phosphor/common/recipes-phosphor/ipmi/phosphor-ipmi-host.bb b/meta-phosphor/common/recipes-phosphor/ipmi/phosphor-ipmi-host.bb
index 641aa4d6a..2b400651d 100644
--- a/meta-phosphor/common/recipes-phosphor/ipmi/phosphor-ipmi-host.bb
+++ b/meta-phosphor/common/recipes-phosphor/ipmi/phosphor-ipmi-host.bb
@@ -27,7 +27,7 @@ SRCREV = "51cb8eef48ffc2bbe165cb276a38c391b4185704"
# Setup IPMI Whitelist Conf files
WHITELIST_CONF = " \
- ${STAGING_ETCDIR_NATIVE}/phosphor-ipmi-host/*.conf \
+ ${STAGING_DATADIR_NATIVE}/phosphor-ipmi-host/*.conf \
${S}/host-ipmid-whitelist.conf \
"
EXTRA_OECONF = " \
diff --git a/meta-phosphor/common/recipes-phosphor/packagegroups/packagegroup-obmc-ipmid-providers.bb b/meta-phosphor/common/recipes-phosphor/packagegroups/packagegroup-obmc-ipmid-providers.bb
index 375e38620..87d17f967 100644
--- a/meta-phosphor/common/recipes-phosphor/packagegroups/packagegroup-obmc-ipmid-providers.bb
+++ b/meta-phosphor/common/recipes-phosphor/packagegroups/packagegroup-obmc-ipmid-providers.bb
@@ -16,15 +16,3 @@ WHITELIST_FORMAT = "{0}-whitelist-native"
OBMC_IPMID_WHITELISTS = "${@compose_list(d, 'WHITELIST_FORMAT', 'OBMC_IPMID_PROVIDERS')}"
DEPENDS_append = "${OBMC_IPMID_WHITELISTS}"
-
-# Normally the dependency is between do_configure and
-# DEPENDS:do_populate_sysroot, but when a package is in the SSTATE cache the
-# do_configure step is not re-ran. The do_populate_sysroot tasks can themselves
-# be performed in parallel by bitbake.
-# Since the do_populate_sysroot task of this recipe doesn't provide the
-# whitelists itself, we need an explicit dependency on the whitelist's
-# do_populate_sysroot task so that the whitelists are present in time for an
-# application that needs them to be do_configured.
-
-WHITELIST_TASK_FORMAT = "{0}:do_populate_sysroot"
-do_populate_sysroot[depends] = "${@compose_list(d, 'WHITELIST_TASK_FORMAT', 'OBMC_IPMID_WHITELISTS')}"
OpenPOWER on IntegriCloud