mutable: remove work-around for a flaw in an older version of foolscap
authorZooko O'Whielacronx <zooko@zooko.com>
Tue, 26 Aug 2008 15:50:55 +0000 (08:50 -0700)
committerZooko O'Whielacronx <zooko@zooko.com>
Tue, 26 Aug 2008 15:50:55 +0000 (08:50 -0700)
We now require "foolscap[secure_connections] >= 0.3.0", per [source:_auto_deps.py].

src/allmydata/mutable/publish.py

index 04621559cbcca985de00c182f629e406a7fbfb26..d6706d54e01594fb51460bfa0f342238272b4021 100644 (file)
@@ -576,18 +576,7 @@ class Publish:
 
             else:
                 # add a testv that requires the share not exist
-                #testv = (0, 1, 'eq', "")
-
-                # Unfortunately, foolscap-0.2.5 has a bug in the way inbound
-                # constraints are handled. If the same object is referenced
-                # multiple times inside the arguments, foolscap emits a
-                # 'reference' token instead of a distinct copy of the
-                # argument. The bug is that these 'reference' tokens are not
-                # accepted by the inbound constraint code. To work around
-                # this, we need to prevent python from interning the
-                # (constant) tuple, by creating a new copy of this vector
-                # each time. This bug is fixed in later versions of foolscap.
-                testv = tuple([0, 1, 'eq', ""])
+                testv = (0, 1, 'eq', "")
 
             testvs = [testv]
             # the write vector is simply the share