[PATCH] Adding an apr_utime() function

2002-03-14 Thread Robert Simonson
This patch adds apr_utime() to the APR. I did this against the unix directories. I don't know how this affects (if at all) other platforms. Thanks. Rob Simonson [EMAIL PROTECTED] apr/file_io/unix/filestat.c === --- filestat.c.old

Re: [PATCH] Adding an apr_utime() function

2002-03-14 Thread Jeff Trawick
"Robert Simonson" <[EMAIL PROTECTED]> writes: > This patch adds apr_utime() to the APR. I did this against the unix > directories. A missing piece for unix is detecting the presence of utime.h and setting the utimeh variable. I think you just need to add utime.h to the APR_FLAG_HEADERS invocati

Re: [PATCH] Adding an apr_utime() function

2002-03-14 Thread William A. Rowe, Jr.
Before anyone even _considers_ polluting the API [which would raise an instant veto from me] we have to finally address the create time issue on non-Unix. Then we can get such a patch committed to fit this resolution of this issue.. Unix has ctime, mtime and atime. How often will we change all th

Re: [PATCH] Adding an apr_utime() function

2002-03-14 Thread William A. Rowe, Jr.
At 02:29 PM 3/14/2002, I wrote: Shall we add crtime to track the file creation timestamp, and mark ctime as not present on all the non-unix platforms, while marking crtime as invalid on win32? s/win32/unix/. Unix is the one that doesn't support crtime. Most others don't support ctime, IIUC. That

Re: [PATCH] Adding an apr_utime() function

2002-03-14 Thread Jeff Trawick
"William A. Rowe, Jr." <[EMAIL PROTECTED]> writes: > Before anyone even _considers_ polluting the API [which would > raise an instant veto from me] we have to finally address the create > time issue on non-Unix. Then we can get such a patch committed > to fit this resolution of this issue.. > >

Re: [PATCH] Adding an apr_utime() function

2002-03-21 Thread Robert Simonson
Finally got to this. Sorry for the delay... My case is a proxy cache implementation that we have here. We want the ability to reset the mtime and atime when doing a cache maintenance so that when the proxy code is run, it has an accurate mtime/atime. We don't care about ctime. In fact, we can'

Re: [PATCH] Adding an apr_utime() function

2002-03-21 Thread William A. Rowe, Jr.
When I suggested this would polute the API --- I'm just suggesting that we deal with atime, mtime, ctime and createtime all at once, and make sure the fn names work. I think a call like apr_file_time_set() or something that passes which metric and it's new value would allow us to have an extensible

[PATCH] apr_file_time_set() (was: Re: [PATCH] Adding an apr_utime() function)

2002-03-22 Thread Robert Simonson
I've taken a shot at redoing this. Is this more what you're thinking? Thanks. Rob Simonson [EMAIL PROTECTED] apr/file_io/unix/filestat.c === --- filestat.c.old Thu Mar 14 11:20:04 2002 +++ filestat.cFri Mar 22 09:49:34

Re: How 'bout this (was: [PATCH] Adding an apr_utime() function)

2002-03-14 Thread Robert Simonson
Thanks for your input Jeff. I've revised this patch some. Here it is... Thanks. Rob Simonson [EMAIL PROTECTED] apr/file_io/unix/filestat.c === --- filestat.c.old Thu Mar 14 11:20:04 2002 +++ filestat.cThu Mar 14 13:19:

Re: How 'bout this (was: [PATCH] Adding an apr_utime() function)

2002-03-14 Thread Jeff Trawick
"Robert Simonson" <[EMAIL PROTECTED]> writes: > Thanks for your input Jeff. I've revised this patch some. Here it is... I'll commit in a day or so if nobody has any concerns. Feel free to remind me Monday if I haven't done so. There are some tweaks which I will make, as mentioned below... >

Re: [PATCH] apr_file_time_set() (was: Re: [PATCH] Adding an apr_utime() function)

2002-03-22 Thread William A. Rowe, Jr.
At 10:00 AM 3/22/2002, you wrote: To: "William A. Rowe, Jr." <[EMAIL PROTECTED]> Cc: dev@apr.apache.org, Jeff Trawick <[EMAIL PROTECTED]> I've taken a shot at redoing this. Is this more what you're thinking? Thanks. Rob Simonson [EMAIL PROTECTED] Terrific! I'll commit as soon as I have a free min