summaryrefslogtreecommitdiffstats
path: root/meta-phosphor
diff options
context:
space:
mode:
authorEd Tanous <ed.tanous@intel.com>2019-04-04 16:29:04 -0700
committerBrad Bishop <bradleyb@fuzziesquirrel.com>2019-04-15 23:00:48 -0400
commit74c48cb2504bf76bd162a28aa8e7688a3d4a8f28 (patch)
tree1a9f7073104cb0cecb5d722835ab805961b72c69 /meta-phosphor
parent5818b24d410f82b75c1a3bfc46818bdd42de5273 (diff)
downloadtalos-openbmc-74c48cb2504bf76bd162a28aa8e7688a3d4a8f28.tar.gz
talos-openbmc-74c48cb2504bf76bd162a28aa8e7688a3d4a8f28.zip
meta-phosphor: Don't build debug tarball implicitly
In previous builds, the debug tarball was built as part of the obmc-phosphor-image implicitly. This increases the build times by a trivial amount, which matters, but is not that important. It also makes debugging image dependencies (using bitbake -g) a bear, given that it includes all dependencies for the debug tarball, as well as the actual image. Given that it's pretty trivial to build this manually via bitbake obmc-debug-tarball and (at least in my opinion) injecting the task seems like a bit of a violation of the yocto intracacies, and gives a surprising result to the user, this patchset removes the task add, and makes obmc-phosphor-debug-tarball a truly separate build target. (From meta-phosphor rev: 5c848ab3436bd0ed1062c4657f35a6a6890992ad) Signed-off-by: Ed Tanous <ed.tanous@intel.com> Change-Id: Ie2294f8e324249a44ef73aa10d57f332e8f54168 Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
Diffstat (limited to 'meta-phosphor')
-rw-r--r--meta-phosphor/classes/obmc-phosphor-image.bbclass2
1 files changed, 0 insertions, 2 deletions
diff --git a/meta-phosphor/classes/obmc-phosphor-image.bbclass b/meta-phosphor/classes/obmc-phosphor-image.bbclass
index 17cbdcd25..807c9c42a 100644
--- a/meta-phosphor/classes/obmc-phosphor-image.bbclass
+++ b/meta-phosphor/classes/obmc-phosphor-image.bbclass
@@ -98,8 +98,6 @@ CORE_IMAGE_EXTRA_INSTALL_append = " bash \
OBMC_IMAGE_EXTRA_INSTALL ?= ""
-do_image_complete[depends] += "obmc-phosphor-debug-tarball:do_image_complete"
-
# The /etc/version file is misleading and not useful. Remove it.
# Users should instead rely on /etc/os-release.
remove_etc_version() {
OpenPOWER on IntegriCloud