diff options
| author | Alexander Kanavin <alex@linutronix.de> | 2024-07-17 20:22:13 +0200 |
|---|---|---|
| committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2024-07-22 16:53:06 +0100 |
| commit | 597b87a46855a792d0d78209db1a85b96a50ac8e (patch) | |
| tree | b9874015bba89bc33007bcc3deae5251884a3673 /meta/lib/oe | |
| parent | def276f0c4c43fa4b95b7b38a944de7cc7e9286f (diff) | |
| download | poky-597b87a46855a792d0d78209db1a85b96a50ac8e.tar.gz | |
lib/oe/recipeutils: return a dict in get_recipe_upgrade_status() instead of a tuple
Putting various things in a tuple is an anti-pattern of sorts, as the consumers
have to unpack it into local variables for readability, or access items directly
with indexes, which makes code pretty much unreadable.
(From OE-Core rev: e86aa26d209eb9809198f6dd40cd058366318e3d)
Signed-off-by: Alexander Kanavin <alex@linutronix.de>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/lib/oe')
| -rw-r--r-- | meta/lib/oe/recipeutils.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/meta/lib/oe/recipeutils.py b/meta/lib/oe/recipeutils.py index 2d69a33113..f9d7dfe253 100644 --- a/meta/lib/oe/recipeutils.py +++ b/meta/lib/oe/recipeutils.py | |||
| @@ -1112,7 +1112,7 @@ def _get_recipe_upgrade_status(data): | |||
| 1112 | maintainer = data.getVar('RECIPE_MAINTAINER') | 1112 | maintainer = data.getVar('RECIPE_MAINTAINER') |
| 1113 | no_upgrade_reason = data.getVar('RECIPE_NO_UPDATE_REASON') | 1113 | no_upgrade_reason = data.getVar('RECIPE_NO_UPDATE_REASON') |
| 1114 | 1114 | ||
| 1115 | return (pn, status, cur_ver, next_ver, maintainer, revision, no_upgrade_reason) | 1115 | return {'pn':pn, 'status':status, 'cur_ver':cur_ver, 'next_ver':next_ver, 'maintainer':maintainer, 'revision':revision, 'no_upgrade_reason':no_upgrade_reason} |
| 1116 | 1116 | ||
| 1117 | def get_recipe_upgrade_status(recipes=None): | 1117 | def get_recipe_upgrade_status(recipes=None): |
| 1118 | pkgs_list = [] | 1118 | pkgs_list = [] |
