commit:     a050ce84b3cf99236a8d452c4688d5d5b69bc1ea
Author:     Michael Palimaka <kensington <AT> gentoo <DOT> org>
AuthorDate: Thu Dec 28 13:42:59 2017 +0000
Commit:     Michael Palimaka <kensington <AT> gentoo <DOT> org>
CommitDate: Thu Dec 28 13:43:38 2017 +0000
URL:        https://gitweb.gentoo.org/proj/kde.git/commit/?id=a050ce84

Documentation: add script to compare CMakeLists and ebuild dependencies

 Documentation/maintainers/cmake_ebuild_check.py | 146 ++++++++++++++++++++++++
 1 file changed, 146 insertions(+)

diff --git a/Documentation/maintainers/cmake_ebuild_check.py 
b/Documentation/maintainers/cmake_ebuild_check.py
new file mode 100755
index 0000000000..27786cf4cb
--- /dev/null
+++ b/Documentation/maintainers/cmake_ebuild_check.py
@@ -0,0 +1,146 @@
+#!/usr/bin/env python3
+#
+# Copyright (c) 2017 Michael Palimaka <kensing...@gentoo.org>
+#
+# Permission is hereby granted, free of charge, to any person obtaining a copy
+# of this software and associated documentation files (the "Software"), to deal
+# in the Software without restriction, including without limitation the rights
+# to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+# copies of the Software, and to permit persons to whom the Software is
+# furnished to do so, subject to the following conditions:
+#
+# The above copyright notice and this permission notice shall be included in
+# all copies or substantial portions of the Software.
+#
+# THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+# IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+# FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+# AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+# LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+# OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
+# THE SOFTWARE.
+
+import argparse
+import os
+import sys
+import subprocess
+import cmake_dep_check as cdc
+import portage
+
+
+def qatom(atom):
+       output = subprocess.check_output(['qatom', '--format', 
'%{CATEGORY}|%{PN}|%{PV}|%[PR]|%[SLOT]|%[pfx]|%[sfx]', atom])
+       return output.decode('utf-8').split('|')
+
+
+def get_cmake_deps(ebuild, repo):
+       cpv = qatom(ebuild)
+       settings = portage.config(config_root='/')
+
+       tmpdir = os.path.join(settings.get('PORTAGE_TMPDIR'), 'portage')
+       tmp_path = os.path.join(tmpdir, cpv[0], cpv[1] + '-' + cpv[2])
+       repo_path = 
portage.db['/']["vartree"].settings.repositories.treemap.get(repo)
+       ebuild_path = os.path.join(repo_path, cpv[0], cpv[1], cpv[1] + '-' + 
cpv[2] + '.ebuild')
+
+       subprocess.call(['ebuild', ebuild_path, 'clean', 'prepare'], 
stdout=subprocess.DEVNULL)
+
+       deps = cdc.getDeps(os.path.join(tmp_path))
+
+       subprocess.call(['ebuild', ebuild_path, 'clean'])
+
+       return deps
+
+
+def get_ebuild_deps(ebuild):
+       porttree = portage.db[portage.root]['porttree']
+       depstr = porttree.dbapi.aux_get(ebuild, ['DEPEND', 'RDEPEND', 
'PDEPEND'])
+
+       depend = portage.dep.use_reduce(depstr[0], matchall=True, flat=True)
+       rdepend = portage.dep.use_reduce(depstr[1], matchall=True, flat=True)
+       pdepend = portage.dep.use_reduce(depstr[2], matchall=True, flat=True)
+
+       all_depend = set(depend + rdepend + pdepend)
+
+       return(all_depend)
+
+
+def get_ebuild_repo(ebuild):
+       porttree = portage.db[portage.root]['porttree']
+       repo = porttree.dbapi.aux_get(ebuild, ['repository'])
+
+       return repo[0]
+
+
+def clean_dep(dep):
+       if dep.startswith('!') or dep == '||':
+               return False
+
+       parts = qatom(dep)
+       cp = parts[0] + '/' + parts[1]
+
+       return cp
+
+
+def main():
+       parser = argparse.ArgumentParser(description='Compare CMakeLists.txt 
and ebuild dependencies')
+       parser.add_argument('ebuild', help='app-foo/bar-1.2.3', nargs=1)
+       parser.add_argument('--possible-bogus-ebuild', dest='bogusebuild', 
action='store_const', const=True, help='Show dependencies that appear in 
CMakeLists.txt but not the ebuild')
+
+       args = parser.parse_args()
+
+       ebuild = args.ebuild[0]
+
+       ebuild_deps = get_ebuild_deps(ebuild)
+       cmake_deps = get_cmake_deps(ebuild, get_ebuild_repo(ebuild))
+
+       cmake_deps_merged = []
+
+       for cmake, deps in cmake_deps.items():
+               for dep in deps:
+                       if dep.startswith('ERROR'):
+                               print(dep)
+                               continue
+                       if len(dep) == 0:
+                               continue
+                       clean = clean_dep(dep)
+                       if clean:
+                               cmake_deps_merged.append(clean)
+
+       ebuild_deps_clean = []
+
+       for dep in ebuild_deps:
+               clean = clean_dep(dep)
+               if clean:
+                       ebuild_deps_clean.append(clean)
+
+       missing_cmake = []
+       missing_ebuild = []
+
+       for dep in cmake_deps_merged:
+               if dep not in ebuild_deps_clean:
+                       missing_ebuild.append(dep)
+
+       for dep in ebuild_deps_clean:
+               if dep not in cmake_deps_merged:
+                       missing_cmake.append(dep)
+
+       missing_cmake = sorted(set(missing_cmake))
+       missing_ebuild = sorted(set(missing_ebuild))
+
+       if len(missing_ebuild) > 0:
+               print('DEPENDENCIES IN CMAKELISTS BUT NOT EBUILD:')
+               print('==========================================')
+               for dep in missing_ebuild:
+                       print(dep)
+       else:
+               print('Did not find any dependencies in CMakeLists that do not 
appear in the ebuild')
+
+       if len(missing_cmake) > 0 and args.bogusebuild is True:
+               print('DEPENDENCIES IN EBUILD BUT NOT IN CMAKELISTS:')
+               print('=============================================')
+               for dep in missing_cmake:
+                       print(dep)
+
+
+if __name__ == '__main__':
+       sys.exit(main())

Reply via email to