* [PATCH 0/1] devtool: package: don't try to initialise tinfoil twice
@ 2016-12-14 8:49 Paul Eggleton
2016-12-14 8:49 ` [PATCH 1/1] " Paul Eggleton
0 siblings, 1 reply; 2+ messages in thread
From: Paul Eggleton @ 2016-12-14 8:49 UTC (permalink / raw)
To: openembedded-core
This patch is on top of the tinfoil2 set but doesn't actually require it -
it fixes an issue that's been there for a while, it just slowed things
down a bit instead of failing as it does with tinfoil2.
The following changes since commit 2eea6a65b009e9d47c03b7d82d2699f88228fc62:
oe-selftest: add basic tinfoil tests (2016-12-13 19:55:02 +1300)
are available in the git repository at:
git://git.openembedded.org/openembedded-core-contrib paule/tinfoil2-extra-fix
http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=paule/tinfoil2-extra-fix
Paul Eggleton (1):
devtool: package: don't try to initialise tinfoil twice
scripts/lib/devtool/package.py | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
--
2.5.5
^ permalink raw reply [flat|nested] 2+ messages in thread
* [PATCH 1/1] devtool: package: don't try to initialise tinfoil twice
2016-12-14 8:49 [PATCH 0/1] devtool: package: don't try to initialise tinfoil twice Paul Eggleton
@ 2016-12-14 8:49 ` Paul Eggleton
0 siblings, 0 replies; 2+ messages in thread
From: Paul Eggleton @ 2016-12-14 8:49 UTC (permalink / raw)
To: openembedded-core
setup_tinfoil() already calls prepare(), we don't need to call it again
ourselves and doing so with tinfoil2 results in "ERROR: Only one copy of
bitbake should be run against a build directory". Calling prepare()
twice should probably still be allowed, so that ought to be fixed
separately, but in the mean time this code is still wrong so fix it
here.
Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
---
scripts/lib/devtool/package.py | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/scripts/lib/devtool/package.py b/scripts/lib/devtool/package.py
index afb5809..4764064 100644
--- a/scripts/lib/devtool/package.py
+++ b/scripts/lib/devtool/package.py
@@ -28,10 +28,8 @@ def package(args, config, basepath, workspace):
"""Entry point for the devtool 'package' subcommand"""
check_workspace_recipe(workspace, args.recipename)
- tinfoil = setup_tinfoil(basepath=basepath)
+ tinfoil = setup_tinfoil(basepath=basepath, config_only=True)
try:
- tinfoil.prepare(config_only=True)
-
image_pkgtype = config.get('Package', 'image_pkgtype', '')
if not image_pkgtype:
image_pkgtype = tinfoil.config_data.getVar('IMAGE_PKGTYPE', True)
--
2.5.5
^ permalink raw reply related [flat|nested] 2+ messages in thread
end of thread, other threads:[~2016-12-14 8:50 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-12-14 8:49 [PATCH 0/1] devtool: package: don't try to initialise tinfoil twice Paul Eggleton
2016-12-14 8:49 ` [PATCH 1/1] " Paul Eggleton
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.