summaryrefslogtreecommitdiffstats
path: root/meta/lib
diff options
context:
space:
mode:
authorRoss Burton <ross.burton@intel.com>2018-03-01 18:26:29 +0000
committerRichard Purdie <richard.purdie@linuxfoundation.org>2018-03-26 15:07:11 +0100
commit59b982e6b8cf18fa4a301d18cd79eee3bbd76250 (patch)
treec9df17708072b2412729f926ca6ee388667ae3eb /meta/lib
parentb0184b60821ab46e1b73fe9365b0c1f6b01b4228 (diff)
downloadpoky-59b982e6b8cf18fa4a301d18cd79eee3bbd76250.tar.gz
package_manager: improve install_complementary
- No need to use bb.utils.which() as subprocess will search $PATH - Clarity flow by moving the install inside the try/except (From OE-Core rev: f4d22b7195dd8f08fe26dd353c7e860208e87d6a) (From OE-Core rev: 92a0359b1e2558b175374a81a1d6146724cd1a9e) (From OE-Core rev: 409e2e8c2b8f99d6f3d5930047f29649a1e62888) Signed-off-by: Ross Burton <ross.burton@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/lib')
-rw-r--r--meta/lib/oe/package_manager.py4
1 files changed, 2 insertions, 2 deletions
diff --git a/meta/lib/oe/package_manager.py b/meta/lib/oe/package_manager.py
index bd6dc99542..4b7163e82e 100644
--- a/meta/lib/oe/package_manager.py
+++ b/meta/lib/oe/package_manager.py
@@ -402,7 +402,7 @@ class PackageManager(object, metaclass=ABCMeta):
402 installed_pkgs.write(output) 402 installed_pkgs.write(output)
403 installed_pkgs.flush() 403 installed_pkgs.flush()
404 404
405 cmd = [bb.utils.which(os.getenv('PATH'), "oe-pkgdata-util"), 405 cmd = ["oe-pkgdata-util",
406 "-p", self.d.getVar('PKGDATA_DIR'), "glob", installed_pkgs.name, 406 "-p", self.d.getVar('PKGDATA_DIR'), "glob", installed_pkgs.name,
407 globs] 407 globs]
408 exclude = self.d.getVar('PACKAGE_EXCLUDE_COMPLEMENTARY') 408 exclude = self.d.getVar('PACKAGE_EXCLUDE_COMPLEMENTARY')
@@ -412,11 +412,11 @@ class PackageManager(object, metaclass=ABCMeta):
412 bb.note("Installing complementary packages ...") 412 bb.note("Installing complementary packages ...")
413 bb.note('Running %s' % cmd) 413 bb.note('Running %s' % cmd)
414 complementary_pkgs = subprocess.check_output(cmd, stderr=subprocess.STDOUT).decode("utf-8") 414 complementary_pkgs = subprocess.check_output(cmd, stderr=subprocess.STDOUT).decode("utf-8")
415 self.install(complementary_pkgs.split(), attempt_only=True)
415 except subprocess.CalledProcessError as e: 416 except subprocess.CalledProcessError as e:
416 bb.fatal("Could not compute complementary packages list. Command " 417 bb.fatal("Could not compute complementary packages list. Command "
417 "'%s' returned %d:\n%s" % 418 "'%s' returned %d:\n%s" %
418 (' '.join(cmd), e.returncode, e.output.decode("utf-8"))) 419 (' '.join(cmd), e.returncode, e.output.decode("utf-8")))
419 self.install(complementary_pkgs.split(), attempt_only=True)
420 420
421 def deploy_dir_lock(self): 421 def deploy_dir_lock(self):
422 if self.deploy_dir is None: 422 if self.deploy_dir is None: