> -----Original Message-----
> From: kot...@apache.org [mailto:kot...@apache.org]
> Sent: maandag 20 november 2017 13:44
> To: comm...@subversion.apache.org
> Subject: svn commit: r1815799 - in /subversion/trunk/subversion:
> libsvn_ra_serf/commit.c tests/libsvn_ra/ra-test.c
>
> Author: kotkov
> Date: Mon Nov 20 12:43:33 2017
> New Revision: 1815799
>
> URL: http://svn.apache.org/viewvc?rev=1815799&view=rev
> Log:
> ra_serf: Properly process lock tokens for empty relative paths ("")
> within the commit editor.
I'm not sure if we should really allow this.
The delta editor explicitly describes that you are opening a directory and then
edit the nodes inside. Only changing properties on the root is allowed and
other operations are all on nodes within. Allowing to open the node itself
again may cause all kinds of problems as there are now multiple handles
pointing to the same thing. How will this be expressed in the
filesystem/transaction?
I'm surprised that all the other filesystems allow this, so perhaps this is a
safe change... but the documentation in svn_delta.h doesn't describe this as a
safe extension. (Which would theoretically allow this as a safe extension in
later versions... but we must make sure that we are not opening new issues this
way)
Currently I would guess that making the ra layers provide a proper error for
this case would not be a bad thing... All our drivers explicitly open an
existing directory when they want to edit a file...
Bert
