[tahoe-dev] [tahoe-lafs] #628: "mtime" and "ctime": I don't think that word means what you think it means.

[tahoe-lafs]

#628: "mtime" and "ctime": I don't think that word means what you think it means.
---------------------------+------------------------------------------------
 Reporter:  zooko          |           Owner:  zooko   
     Type:  defect         |          Status:  assigned
 Priority:  major          |       Milestone:  1.4.0   
Component:  code-dirnodes  |         Version:  1.3.0   
 Keywords:                 |   Launchpad_bug:          
---------------------------+------------------------------------------------

Comment(by zooko):

 Oh, I see where this patch is breaking backupdb, by updating the
 {{{mtime}}} and the {{{ctime}}} even if the caller passed in a metadata
 argument.  The following version of the patch doesn't update the
 {{{mtime}}} and {{{ctime}}} when the caller has passed in a metadata dict
 (just like Tahoe 1.3.0), but it still updates the {{{__sys}}} timestamps
 (which is the semantics that I think I want in the future -- those
 timestamps are describing what changes were made to the tahoe filesystem
 itself and have nothing to do with timestamps that were associated with
 the files in their local filesystem before they were uploaded to a tahoe
 filesystem).

 This finally makes it pass {{{test_dirnode}}} and {{{test_backupdb}}}.
 Now I have to run off to volunteer to teach arithmetic to second-graders.

-- 
Ticket URL: <http://allmydata.org/trac/tahoe/ticket/628#comment:10>
tahoe-lafs <http://allmydata.org>
secure decentralized file storage grid