From 523afbd0142dcb40ef3265499944f178c601dddf Mon Sep 17 00:00:00 2001 From: Kevin Carter Date: Wed, 27 Jul 2016 08:29:46 -0500 Subject: [PATCH] Fix py_pkgs lookup to not include optional packages This change fixes a regression created when we Updated lookup plugin to allow it to also index requirement files. The offending change is here: "I5e981c818a1da50aa0ae15630655a8dfac4f3db7". This simply ensures that only non-optional packages are part of the specific role requirements. Change-Id: I1b65a595d8691fdd38ffd568db71a1bcc06207c9 Signed-off-by: Kevin Carter --- lookup/py_pkgs.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lookup/py_pkgs.py b/lookup/py_pkgs.py index 0a14429..85be981 100644 --- a/lookup/py_pkgs.py +++ b/lookup/py_pkgs.py @@ -458,8 +458,8 @@ class DependencyFileProcessor(object): pkgs = role_pkgs.get(var_name, list()) if 'optional' not in var_name: - pkgs.extend(packages) - pkg_index[role_name][var_name] = self._py_pkg_extend(packages, pkgs) + pkgs = self._py_pkg_extend(packages, pkgs) + pkg_index[role_name][var_name] = pkgs else: for k, v in pkg_index.items(): for item_name in v.keys():