Re: cvs commit: apr/test testfileinfo.c

Ben Collins-Sussman wrote:

>Branko Čibej <brane@xbc.nu> writes:
>
>
>
>>Oh, by the way: lying about the modification time of a directory is
>>generally not a good idea, and it's not something "make" is interested
>>in. Let's think carefully about what we want to achieve here before
>>starting to change directory mtimes.
>>
>>
>
>Bad idea to set mtimes?
>
>At the moment, I have 'svn export' setting mtimes on files. I figured
>that the export editor's close_directory() could do the same. I mean,
>directories are versioned objects in svn. What's the danger in
>setting their mtime to last-commit-time?
>
I don't know if there's a danger, but someone (you? :-) has to think
about whether it's consistent. Can files in the directory have mtimes
newer than the directory itself? If so, is that reasonable, on all
systems? Does a propchange on a directory affect its mtime? Etc. ad nauseam.

It's simple with files, not so simple with directories -- as we've found
to our detriment (remember all the ghurson problems).

-- 
Brane Čibej   <brane_at_xbc.nu>   http://www.xbc.nu/brane/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org