From: Ola x Nilsson <ola.x.nilsson@axis.com>
To: olani@axis.com, openembedded-core@lists.openembedded.org
Subject: [PATCH 0/4] Load devtool and reciptool plugins in a well-defined manner
Date: Tue, 25 Oct 2016 13:03:31 +0200 [thread overview]
Message-ID: <1477393415-13053-1-git-send-email-olani@axis.com> (raw)
From: Ola x Nilsson <ola.x.nilsson@axis.com>
Make devtool and recipetool load plugins in a well-defined and expected manner.
devtool and reciptool search for pluings first in scripts/lib/<tool>
and then in BBPATH. Each found plugin is loaded, which means that any
plugin found later in BBPATH shadows previously loaded plugins.
plugin_init is loaded for all plugins, no matter whether they are
later replaced. register_plugin is called once for each time a plugin
with a certain name is found, but it is always called on the last
loaded plugin.
Change the scriptutils.load_plugins function to never load a plugin
file if one with the same filename has already been loaded. This
means that the first found plugin wins.
Change the load order to first search BBPATH and then
scripts/lib/tools. This means that plugins in scripts/lib/<tool> can
be shadowed by plugins in layers.
Ola x Nilsson (4):
devtool: selftest: add test for devtool plugin loading
recipetool: selftest: Add test for recipetool plugin loading
devtool: Load plugins in a well defined order
recipetool: Load plugins in a well defined order
meta-selftest/lib/devtool/bbpath.py | 44 ++++++++++++++++++++++++++++++++++
meta-selftest/lib/recipetool/bbpath.py | 41 +++++++++++++++++++++++++++++++
meta/lib/oeqa/selftest/devtool.py | 43 +++++++++++++++++++++++++++++++++
meta/lib/oeqa/selftest/recipetool.py | 44 ++++++++++++++++++++++++++++++++++
scripts/devtool | 3 ++-
scripts/lib/scriptutils.py | 8 +++++--
scripts/recipetool | 4 ++--
7 files changed, 182 insertions(+), 5 deletions(-)
create mode 100644 meta-selftest/lib/devtool/bbpath.py
create mode 100644 meta-selftest/lib/recipetool/bbpath.py
--
2.1.4
next reply other threads:[~2016-10-25 11:03 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-10-25 11:03 Ola x Nilsson [this message]
2016-10-25 11:03 ` [PATCH 1/4] devtool: selftest: add test for devtool plugin loading Ola x Nilsson
2016-10-25 11:03 ` [PATCH 2/4] recipetool: selftest: Add test for recipetool " Ola x Nilsson
2016-10-25 11:03 ` [PATCH 3/4] devtool: Load plugins in a well defined order Ola x Nilsson
2016-10-25 11:03 ` [PATCH 4/4] recipetool: " Ola x Nilsson
2016-12-06 7:37 ` [PATCH 0/4] Load devtool and reciptool plugins in a well-defined manner Ola x Nilsson
2016-12-07 21:16 ` Paul Eggleton
2016-12-08 15:24 ` Burton, Ross
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1477393415-13053-1-git-send-email-olani@axis.com \
--to=ola.x.nilsson@axis.com \
--cc=olani@axis.com \
--cc=openembedded-core@lists.openembedded.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox