diff options
| author | Kun Yi <kunyi@google.com> | 2017-10-10 09:17:08 +0300 |
|---|---|---|
| committer | Kun Yi <kunyi@google.com> | 2017-10-19 00:18:50 +0300 |
| commit | cdc3b23921bc2804b06abd6812839c84e01cec80 (patch) | |
| tree | 6d3f0b21d4d641b34ca50d044c1f1ff66af94249 /meta-phosphor/common/recipes-phosphor/settings | |
| parent | 2a595c5906c85313829b026e3c995e59d43d11a2 (diff) | |
| download | openbmc-cdc3b23921bc2804b06abd6812839c84e01cec80.tar.xz | |
Fix YAML overriding.
bitbake tasks use python3, under which filter() returns a filter object
which doesn't enumerate as list. As a result none of the overriding YAMLs
takes effect. Use list expansion instead.
Discussion for context:
https://lists.ozlabs.org/pipermail/openbmc/2017-October/009521.html
Fixes openbmc/openbmc#2469.
Change-Id: I37cc7fc14b6f91c4eb1fb199d7cff52d93c9ad2a
Signed-off-by: Kun Yi <kunyi@google.com>
Diffstat (limited to 'meta-phosphor/common/recipes-phosphor/settings')
| -rw-r--r-- | meta-phosphor/common/recipes-phosphor/settings/phosphor-settings-manager.bb | 2 | ||||
| -rw-r--r-- | meta-phosphor/common/recipes-phosphor/settings/phosphor-settings.bb | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/meta-phosphor/common/recipes-phosphor/settings/phosphor-settings-manager.bb b/meta-phosphor/common/recipes-phosphor/settings/phosphor-settings-manager.bb index 525a9fdffe..10938c9e60 100644 --- a/meta-phosphor/common/recipes-phosphor/settings/phosphor-settings-manager.bb +++ b/meta-phosphor/common/recipes-phosphor/settings/phosphor-settings-manager.bb @@ -48,7 +48,7 @@ python do_merge_settings () { cmd.append(os.path.join(settingsdir, 'defaults.yaml')) fetch = bb.fetch2.Fetch([], d) - override_urls = filter(lambda f: f.endswith('.override.yml'), fetch.urls) + override_urls = [url for url in fetch.urls if url.endswith('.override.yml')] for url in override_urls: bb.debug(2, 'Overriding with source: ' + url) local_base = os.path.basename(fetch.localpath(url)) diff --git a/meta-phosphor/common/recipes-phosphor/settings/phosphor-settings.bb b/meta-phosphor/common/recipes-phosphor/settings/phosphor-settings.bb index 191a389aea..8e94d8f62f 100644 --- a/meta-phosphor/common/recipes-phosphor/settings/phosphor-settings.bb +++ b/meta-phosphor/common/recipes-phosphor/settings/phosphor-settings.bb @@ -36,7 +36,7 @@ python do_merge_settings () { cmd.append(os.path.join(d.getVar('S', True), 'settings.yaml')) fetch = bb.fetch2.Fetch([], d) - override_urls = filter(lambda f: f.endswith('.override.yml'), fetch.urls) + override_urls = [url for url in fetch.urls if url.endswith('.override.yml')] for url in override_urls: bb.debug(2, 'Overriding with source: ' + url) local_base = os.path.basename(fetch.localpath(url)) |