diff options
author | Patrick Williams <patrick@stwcx.xyz> | 2017-02-24 13:04:06 -0600 |
---|---|---|
committer | Patrick Williams <patrick@stwcx.xyz> | 2017-02-24 13:07:48 -0600 |
commit | 43b61b7029b4e566766ee8c36b59efe826326d74 (patch) | |
tree | ceeb616ac7a0e9378caeec41bfcf4b1f145179cb | |
parent | feb59cb0723d9b3b0afb27e05869fa6d2a42bc6c (diff) | |
download | talos-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>
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')}" |