ExternalVendorCode/tahoe-lafs
1
0
Fork 0
mirror of https://github.com/tahoe-lafs/tahoe-lafs.git synced 2025-01-19 11:16:24 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Merge pull request #176 from tahoe-lafs/2436.less-scary.0

Browse Source 
Don't show scary diagnostic warnings from --version[-and-path] if the only problem is that extra packages are found by pkg_resources.
This commit is contained in:
zooko 2015-07-21 17:59:31 +00:00
commit 49b0c686f1
2 changed files with 48 additions and 38 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
src/allmydata/__init__.py
View File

@ -377,12 +377,12 @@ def cross_check(pkg_resources_vers_and_locs, imported_vers_and_locs_list):
% (name, pr_ver, str(pr_normver), pr_loc, imp_ver, str(imp_normver), imp_loc))
imported_packages = set([p.lower() for (p, _) in imported_vers_and_locs_list])
extra_vers_and_locs_list = []
for pr_name, (pr_ver, pr_loc) in pkg_resources_vers_and_locs.iteritems():
if pr_name not in imported_packages and pr_name not in ignorable:
errors.append("Warning: dependency %r (version %r) found by pkg_resources not found by import."
% (pr_name, pr_ver))
extra_vers_and_locs_list.append( (pr_name, (pr_ver, pr_loc, "according to pkg_resources")) )
return errors
return errors, extra_vers_and_locs_list
def get_error_string(errors, debug=False):
@ -437,6 +437,12 @@ def get_package_locations():
return dict([(k, l) for k, (v, l, c) in _vers_and_locs_list])
def get_package_versions_string(show_paths=False, debug=False):
errors = []
if not hasattr(sys, 'frozen'):
global _vers_and_locs_list
errors, extra_vers_and_locs_list = cross_check_pkg_resources_versus_import()
_vers_and_locs_list += extra_vers_and_locs_list
res = []
for p, (v, loc, comment) in _vers_and_locs_list:
info = str(p) + ": " + str(v)
@ -448,9 +454,7 @@ def get_package_versions_string(show_paths=False, debug=False):
output = "\n".join(res) + "\n"
if not hasattr(sys, 'frozen'):
errors = cross_check_pkg_resources_versus_import()
if errors:
output += get_error_string(errors, debug=debug)
if errors:
output += get_error_string(errors, debug=debug)
return output

68
src/allmydata/test/test_version.py
View File

@ -73,65 +73,71 @@ class CheckRequirement(unittest.TestCase):
# The bug in #1355 is triggered when a version string from either pkg_resources or import
# is not parseable at all by normalized_version.
res = cross_check({"foo": ("unparseable", "")}, [("foo", ("1.0", "", None))])
self.failUnlessEqual(len(res), 1)
self.failUnlessIn("by pkg_resources could not be parsed", res[0])
(errors, extras) = cross_check({"foo": ("unparseable", "")}, [("foo", ("1.0", "", None))])
self.failUnlessEqual(extras, [])
self.failUnlessEqual(len(errors), 1)
self.failUnlessIn("by pkg_resources could not be parsed", errors[0])
res = cross_check({"foo": ("1.0", "")}, [("foo", ("unparseable", "", None))])
self.failUnlessEqual(len(res), 1)
self.failUnlessIn(") could not be parsed", res[0])
(errors, extras) = cross_check({"foo": ("1.0", "")}, [("foo", ("unparseable", "", None))])
self.failUnlessEqual(extras, [])
self.failUnlessEqual(len(errors), 1)
self.failUnlessIn(") could not be parsed", errors[0])
def test_cross_check(self):
res = cross_check({}, [])
self.failUnlessEqual(res, [])
self.failUnlessEqual(res, ([], []))
res = cross_check({}, [("allmydata-tahoe", ("1.0", "", "blah"))])
self.failUnlessEqual(res, [])
self.failUnlessEqual(res, ([], []))
res = cross_check({"foo": ("unparseable", "")}, [])
self.failUnlessEqual(len(res), 1)
self.failUnlessIn("not found by import", res[0])
self.failUnlessEqual(res, ([], [("foo", ("unparseable", "", "according to pkg_resources"))]))
res = cross_check({"argparse": ("unparseable", "")}, [])
self.failUnlessEqual(len(res), 0)
self.failUnlessEqual(res, ([], []))
res = cross_check({}, [("foo", ("unparseable", "", None))])
self.failUnlessEqual(len(res), 1)
self.failUnlessIn("not found by pkg_resources", res[0])
(errors, extras) = cross_check({}, [("foo", ("unparseable", "", None))])
self.failUnlessEqual(extras, [])
self.failUnlessEqual(len(errors), 1)
self.failUnlessIn("was not found by pkg_resources", errors[0])
res = cross_check({"distribute": ("1.0", "/somewhere")}, [("setuptools", ("2.0", "/somewhere", "distribute"))])
self.failUnlessEqual(len(res), 0)
self.failUnlessEqual(res, ([], []))
res = cross_check({"distribute": ("1.0", "/somewhere")}, [("setuptools", ("2.0", "/somewhere", None))])
self.failUnlessEqual(len(res), 1)
self.failUnlessIn("location mismatch", res[0])
(errors, extras) = cross_check({"distribute": ("1.0", "/somewhere")}, [("setuptools", ("2.0", "/somewhere", None))])
self.failUnlessEqual(extras, [])
self.failUnlessEqual(len(errors), 1)
self.failUnlessIn("location mismatch", errors[0])
res = cross_check({"distribute": ("1.0", "/somewhere")}, [("setuptools", ("2.0", "/somewhere_different", None))])
self.failUnlessEqual(len(res), 1)
self.failUnlessIn("location mismatch", res[0])
(errors, extras) = cross_check({"distribute": ("1.0", "/somewhere")}, [("setuptools", ("2.0", "/somewhere_different", None))])
self.failUnlessEqual(extras, [])
self.failUnlessEqual(len(errors), 1)
self.failUnlessIn("location mismatch", errors[0])
res = cross_check({"zope.interface": ("1.0", "")}, [("zope.interface", ("unknown", "", None))])
self.failUnlessEqual(len(res), 0)
self.failUnlessEqual(res, ([], []))
res = cross_check({"foo": ("1.0", "")}, [("foo", ("unknown", "", None))])
self.failUnlessEqual(len(res), 1)
self.failUnlessIn("could not find a version number", res[0])
(errors, extras) = cross_check({"foo": ("1.0", "")}, [("foo", ("unknown", "", None))])
self.failUnlessEqual(extras, [])
self.failUnlessEqual(len(errors), 1)
self.failUnlessIn("could not find a version number", errors[0])
# When pkg_resources and import both find a package, there is only a warning if both
# the version and the path fail to match.
res = cross_check({"foo": ("1.0", "/somewhere")}, [("foo", ("2.0", "/somewhere", None))])
self.failUnlessEqual(len(res), 0)
self.failUnlessEqual(res, ([], []))
res = cross_check({"foo": ("1.0", "/somewhere")}, [("foo", ("1.0", "/somewhere_different", None))])
self.failUnlessEqual(len(res), 0)
self.failUnlessEqual(res, ([], []))
res = cross_check({"foo": ("1.0-r123", "/somewhere")}, [("foo", ("1.0.post123", "/somewhere_different", None))])
self.failUnlessEqual(len(res), 0)
self.failUnlessEqual(res, ([], []))
res = cross_check({"foo": ("1.0", "/somewhere")}, [("foo", ("2.0", "/somewhere_different", None))])
self.failUnlessEqual(len(res), 1)
self.failUnlessIn("but version '2.0'", res[0])
(errors, extras) = cross_check({"foo": ("1.0", "/somewhere")}, [("foo", ("2.0", "/somewhere_different", None))])
self.failUnlessEqual(extras, [])
self.failUnlessEqual(len(errors), 1)
self.failUnlessIn("but version '2.0'", errors[0])
def test_extract_openssl_version(self):
self.failUnlessEqual(extract_openssl_version(MockSSL("")),