"tahoe cp": audit use of "mutable" flag #916

Open
opened 2010-01-18 23:30:49 +00:00 by warner · 0 comments
warner commented 2010-01-18 23:30:49 +00:00
Owner

David-Sarah pointed out a probable bug in source:src/allmydata/scripts/tahoe_cp.py , in which the self.mutable flag (set on things like TahoeFileTarget and TahoeDirectorySource) is defaulting to False when the webapi JSON data doesn't provide a value. I suspect that I lazily copy-and-pasted code from the File cases into the Directory cases, and that while older webapi data might omit mutable and should default to False for files, for directories the opposite is true, and we should default to True.

This probably won't have any practical consequence until "tahoe cp" is taught to take advantage of immutable directories (by copying the dircap instead of creating a brand new directory).

David-Sarah pointed out a probable bug in source:src/allmydata/scripts/tahoe_cp.py , in which the `self.mutable` flag (set on things like `TahoeFileTarget` and `TahoeDirectorySource`) is defaulting to False when the webapi JSON data doesn't provide a value. I suspect that I lazily copy-and-pasted code from the File cases into the Directory cases, and that while older webapi data might omit `mutable` and should default to False for **files**, for **directories** the opposite is true, and we should default to True. This probably won't have any practical consequence until "tahoe cp" is taught to take advantage of immutable directories (by copying the dircap instead of creating a brand new directory).
tahoe-lafs added the
code-frontend-cli
minor
defect
1.5.0
labels 2010-01-18 23:30:49 +00:00
tahoe-lafs added this to the undecided milestone 2010-01-18 23:30:49 +00:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: tahoe-lafs/trac-2024-07-25#916
No description provided.