Re: svn commit: r24884 - trunk/subversion/svn

On Wed, May 02, 2007 at 09:24:34AM -0700, Daniel Rall wrote:
> > - if (err && APR_STATUS_IS_EACCES(err->apr_err))
> > + if (err && (APR_STATUS_IS_EACCES(err->apr_err) || err->apr_err == EROFS))
> > {
> > const char *temp_dir_apr;
>
> Is EROFS defined on all operating systems? If not, we might want to
> #ifdef that check.
>

It's defined by POSIX, so I hope so (unlike ESTALE, which is a NFS
invention). It looks like Darwin is happy with it, from the buildbot.

Unfortunately I can't check Windows because there aren't any Windows
buildbot slaves running at the moment, but I'd say if it works for
Windows, we're okay.

Regards,
Malcolm