From 564f6c799ad882f6503ce2764dc9ad76b6e45fa6 Mon Sep 17 00:00:00 2001
From: Daira Hopwood <david-sarah@jacaranda.org>
Date: Thu, 18 Apr 2013 22:45:04 +0100
Subject: [PATCH] Cleanup to declare not_import_versionable and ignorable
 packages in _auto_deps.py

Signed-off-by: Daira Hopwood <david-sarah@jacaranda.org>
---
 src/allmydata/__init__.py   | 6 ++++--
 src/allmydata/_auto_deps.py | 7 +++++++
 2 files changed, 11 insertions(+), 2 deletions(-)

diff --git a/src/allmydata/__init__.py b/src/allmydata/__init__.py
index 694b6e6c..d17abfaa 100644
--- a/src/allmydata/__init__.py
+++ b/src/allmydata/__init__.py
@@ -280,10 +280,12 @@ def cross_check_pkg_resources_versus_import():
 def cross_check(pkg_resources_vers_and_locs, imported_vers_and_locs_list):
     """This function returns a list of errors due to any failed cross-checks."""
 
+    from _auto_deps import not_import_versionable_packages, ignorable_packages
+
     errors = []
     not_pkg_resourceable = set(['python', 'platform', __appname__.lower()])
-    not_import_versionable = set(['zope.interface', 'mock', 'pyasn1', 'python-gflags'])
-    ignorable = set(['argparse', 'pyutil', 'zbase32', 'distribute', 'twisted-web', 'twisted-core', 'twisted-conch', 'six'])
+    not_import_versionable = set(not_import_versionable_packages)
+    ignorable = set(ignorable_packages)
 
     for name, (imp_ver, imp_loc, imp_comment) in imported_vers_and_locs_list:
         name = name.lower()
diff --git a/src/allmydata/_auto_deps.py b/src/allmydata/_auto_deps.py
index 63bf7576..527253b1 100644
--- a/src/allmydata/_auto_deps.py
+++ b/src/allmydata/_auto_deps.py
@@ -99,6 +99,13 @@ package_imports = [
     ('python-gflags',   'gflags'),
 ]
 
+# Packages we cannot find a version number for by importing.
+not_import_versionable_packages = ('zope.interface', 'mock', 'pyasn1', 'python-gflags')
+
+# Packages that pkg_resources might report, but we don't care about checking their version.
+ignorable_packages = ('argparse', 'pyutil', 'zbase32', 'distribute', 'twisted-web', 'twisted-core', 'twisted-conch', 'six')
+
+
 def require_more():
     import sys
 
-- 
2.45.2