use foolscap's new app_versions API, require foolscap-0.3.1

diff --git a/_auto_deps.py b/_auto_deps.py
index 0712ec0262810fcc6c23f6ed29454234e475312d..46d455ca32c55bf81f1ffdaf672ff5f03fd8be34 100644 (file)
--- a/_auto_deps.py
+++ b/_auto_deps.py
@@ -1,5 +1,5 @@
 install_requires=["zfec >= 1.1.0",
-                  "foolscap[secure_connections] >= 0.3.0",
+                  "foolscap[secure_connections] >= 0.3.1",
                   "simplejson >= 1.4",
 
                   # pycryptopp < 0.5 had a bug which, using a Microsoft

diff --git a/src/allmydata/node.py b/src/allmydata/node.py
index 8ad627060ed787df52fc51643fdfe9dd56798cb6..f7c74872dcd9d7f293599e4ed004efa96dd953aa 100644 (file)
--- a/src/allmydata/node.py
+++ b/src/allmydata/node.py
@@ -15,13 +15,13 @@ from allmydata.util.assertutil import precondition
 # Just to get their versions:
 import allmydata, pycryptopp, zfec
 
-from foolscap.logging.publish import LogPublisher
+from foolscap.logging import app_versions
 # Add our application versions to the data that Foolscap's LogPublisher
 # reports. Our __version__ attributes are actually instances of a "Version"
 # class, so convert them into strings first.
-LogPublisher.versions['allmydata'] = str(allmydata.__version__)
-LogPublisher.versions['zfec'] = str(zfec.__version__)
-LogPublisher.versions['pycryptopp'] = str(pycryptopp.__version__)
+app_versions.add_version('allmydata', str(allmydata.__version__))
+app_versions.add_version('zfec', str(zfec.__version__))
+app_versions.add_version('pycryptopp', str(pycryptopp.__version__))
 
 # group 1 will be addr (dotted quad string), group 3 if any will be portnum (string)
 ADDR_RE=re.compile("^([1-9][0-9]*\.[1-9][0-9]*\.[1-9][0-9]*\.[1-9][0-9]*)(:([1-9][0-9]*))?$")
@@ -90,6 +90,10 @@ class Node(service.MultiService):
         self.log("Node constructed. " + get_package_versions_string())
         iputil.increase_rlimits()
 
+    def get_app_versions(self):
+        # TODO: merge this with allmydata.get_package_versions
+        return dict(app_versions.versions)
+
     def get_config(self, name, required=False):
         """Get the (string) contents of a config file, or None if the file
         did not exist. If required=True, raise an exception rather than