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

Re: handling of reparse points

From: Philip Martin <philip_at_codematters.co.uk>
Date: Tue, 05 Jun 2018 22:35:38 +0100

Stefan Küng <tortoisesvn_at_gmail.com> writes:

> Index: subversion/libsvn_subr/io.c
> ===================================================================
> --- subversion/libsvn_subr/io.c (revision 1831874)
> +++ subversion/libsvn_subr/io.c (working copy)
> @@ -342,8 +342,11 @@
> /* Not using svn_io_stat() here because we want to check the
> apr_err return explicitly. */
> SVN_ERR(cstring_from_utf8(&path_apr, path, pool));
> -
> +#ifdef WIN32
> + flags = APR_FINFO_MIN;
> +#else
> flags = resolve_symlinks ? APR_FINFO_MIN : (APR_FINFO_MIN | APR_FINFO_LINK);
> +#endif
> apr_err = apr_stat(&finfo, path_apr, flags, pool);
> if (APR_STATUS_IS_ENOENT(apr_err))

This needs a comment to explain why Windows needs to do something

I think we would have to back this change out if we ever attempted to
add svn:special support on Windows, but I suppose any such support is
unlikely in the near future; as Branko pointed out CreateSymbolicLink
doesn't have the semantics we want.

Making .svn a symbolic link on Linux came up in another thread recently:


At present there is an assumption that .svn lives on the same filesystem
as the working copy and that files can be atomically moved from .svn/tmp
into the working copy (see workqueue.c calling svn_io_file_rename2). If
.svn becomes a symlink that points to a different filesystem then these
moves fail and Subversion doesn't work. We might be able to work around
this by replacing svn_io_file_rename2 with svn_io_file_move.

I don't know if Windows has the same problem.

Received on 2018-06-05 23:36:01 CEST

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