Peter N. Lundblad wrote:
>
> I want to get rid of the prop-time field of the entries in the
> .svn/entries file in the new WC format. Since property modification state
> is stored instead, it becomes unnecessary. The interesting thing is that
> we have a test case (stat_test 13) that depend on this field (it looks at
> the svn info output). As I see prop-time as an implementation detail, I
> think this test case is broken.
Yes - don't worry about the test case, just adjust it.
> But, maybe I'm wrong and we must keep this prop-time just because svn info
> outputs it? BTW, it is not always available (sachedule ad comes to mind),
> so people can't just blindly rely on its existence.
I suspect "svn info" arose as a debugging aid, and not all of what it prints
would have been designed into such a feature intended for end-users.
Personally I'd be happy to not have the "Props Last Updated" field. However, I
think it would be a pretty bad compatibility decision to remove it before
version 2. Therefore I think it needs to stay. (If you don't combine the
various props files into fewer, you could perhaps use their on-disk
last-modified time stamps instead.)
> I think it would be good to get rid of it both for code cleanliness and
> because it saves some space in the entries file.
Maybe simplify the code by not using it, but keep it just for "svn info".
The space it takes in the entries file is something around 10%, a noticeable
proportion but not important, and certainly not a significant proportion of
Subversion's overall space overhead.
- Julian
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon Nov 7 18:05:21 2005