code.ossystems Code Review - openembedded-core.git/blob

   1 From a2dd127b4163aff6cc35af0d0251321964232ad4 Mon Sep 17 00:00:00 2001
   2 From: Alexander Kanavin <alex.kanavin@gmail.com>
   3 Date: Mon, 7 Oct 2019 13:22:14 +0200
   4 Subject: [PATCH] setup.py: do not report missing dependencies for disabled
   5  modules
   6
   7 Reporting those missing dependencies is misleading as the modules would not
   8 have been built anyway. This particularly matters in oe-core's automated
   9 build completeness checker which relies on the report.
  10
  11 Upstream-Status: Inappropriate [oe-core specific]
  12 Signed-off-by: Alexander Kanavin <alex.kanavin@gmail.com>
  13 Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
  14 Signed-off-by: Alejandro Hernandez Samaniego <alejandro@enedino.org>
  15
  16 ---
  17  setup.py | 4 ++++
  18  1 file changed, 4 insertions(+)
  19
  20 diff --git a/setup.py b/setup.py
  21 index 7691258..ec3f2a4 100644
  22 --- a/setup.py
  23 +++ b/setup.py
  24 @@ -408,6 +408,14 @@ class PyBuildExt(build_ext):
  25                  print("%-*s   %-*s   %-*s" % (longest, e, longest, f,
  26                                                longest, g))
  27
  28 +        # There is no need to report missing module dependencies,
  29 +        # if the modules have been disabled in the first place.
  30 +        # cannot use mods_disabled here, because remove_configured_extensions adds
  31 +        # only disabled extensions into it (doesn't cover _dbm, _gdbm, readline
  32 +        # we support disabling through PACKAGECONFIG)
  33 +        sysconf_dis = sysconfig.get_config_var('MODDISABLED_NAMES').split()
  34 +        self.missing = list(set(self.missing) - set(sysconf_dis))
  35 +
  36          if self.missing:
  37              print()
  38              print("Python build finished successfully!")