[tahoe-lafs-trac-stream] [Tahoe-LAFS] #2047: refactor *Source classes in tahoe_cp.py to have a basename() method rather than passing around (name, source) pairs
Tahoe-LAFS
trac at tahoe-lafs.org
Tue Jan 20 20:48:30 UTC 2015
#2047: refactor *Source classes in tahoe_cp.py to have a basename() method rather
than passing around (name, source) pairs
-----------------------------------+------------------------------
Reporter: daira | Owner: daira
Type: defect | Status: assigned
Priority: normal | Milestone: undecided
Component: code-frontend-cli | Version: 1.10.0
Resolution: | Keywords: cleanup tahoe-cp
Launchpad Bug: |
-----------------------------------+------------------------------
Comment (by warner):
The source object might be a pure filecap or dircap, and thus have no
name. I think I wrote it this way (with tuples) as a reminder of that
feature/property. I wouldn't object to moving this value into an property
or method of the `TahoeFileSource` and `LocalFileSource` objects, but I'm
not sure that `basename()` is the best name for it, if that's likely to
imply that it always exists.
I think this assumption is the root cause for the #2329 regression.
--
Ticket URL: <https://tahoe-lafs.org/trac/tahoe-lafs/ticket/2047#comment:3>
Tahoe-LAFS <https://Tahoe-LAFS.org>
secure decentralized storage
More information about the tahoe-lafs-trac-stream
mailing list