[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: cvs commit: apr/test testfileinfo.c

From: Branko Čibej <brane_at_xbc.nu>
Date: 2003-07-09 00:35:35 CEST

Branko Čibej wrote:

>Ben Collins-Sussman wrote:
>
>
>
>>sussman@apache.org writes:
>>
>>
>>
>>
>>
>>>sussman 2003/07/07 15:44:11
>>>
>>> Log:
>>> New apr_file_mtime_set() API, implemented in unix and win32.
>>> Patches from Branko Cibej (brane) and Matt Kraai <kraai@alumni.cmu.edu>.
>>>
>>>
>>>
>>>
>>So can this API be renamed to apr_path_mtime_set()? Can it be made to
>>work on directories as well as files? Certainly the unix utime()
>>implementation works on any kind of inode. How about brane's win32
>>implementation?
>>
>>
>>
>I think the Win32 bit won't work on directories without changes,
>although I think it can be changed so that it does. The Unix
>implementation probably will work on dirs unchanged.
>
>I have no objection to changing the funciton name (or at least the docs)
>and teach it to work on both files and dirs.
>
>
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.

-- 
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
Received on Wed Jul 9 00:38:32 2003

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.