You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Daniel Shahaf <d....@daniel.shahaf.name> on 2009/10/05 19:41:48 UTC

Re: svn commit: r39780 - trunk/subversion/libsvn_fs_fs

Julian Foad wrote on Mon, 5 Oct 2009 at 06:45 -0700:
> Author: julianfoad
> Date: Mon Oct  5 06:45:56 2009
> New Revision: 39780
> 
> Log:
> Write "the 'current' file" instead of "the current file" for clarity in error
> messages and comments when referring to the FSFS file otherwise known as the
> txn-current file and named 'db/current'.

The commit looks fine, but the log message implies that 'txn-current'
and 'current' are the same thing.  And, as far as I can tell,

    [[[
    .../mirror/svn/db% cat current
    39705
    .../mirror/svn/db% cat txn-current
    ut7
    .../mirror/svn/db% grep current ../../../trunk/subversion/libsvn_fs_fs/structure | head -n 2
      txn-current         File containing the next transaction key
      current             File specifying current revision and next node/copy id
    .../mirror/svn/db% 
    ]]]

they aren't the same :)

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2403866

Re: svn commit: r39780 - trunk/subversion/libsvn_fs_fs

Posted by Julian Foad <ju...@btopenworld.com>.
Daniel Shahaf wrote:
> The commit looks fine, but the log message implies that 'txn-current'
> and 'current' are the same thing.  And, as far as I can tell,
[...]
> they aren't the same :)

D'oh, thanks. I have edited the log message.

- Julian

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2403910