Subversion needs to change the files' modification times. The appended patch adds apr_file_mtime_set, which does so. It also adds apr_time_ansi_get, which is used to convert an apr_time_t to a time_t.
Most of the manpages I read say that we should use utimes() not utime(). It seems it'd be good to get away from using the utimbuf structure if at all possible. I'm not sure about the general availability of utimes() though (autoconf test?).
I would think it would be best to have apr_time_ansi_get as a macro rather than a function. (And that macro should just use the apr_time_as_sec macro with an appropriate cast.)
I would also suggest reviewing the style guidelines. You have tabs and no braces around if statements. -- justin
