You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Greg Stein <gs...@gmail.com> on 2012/03/22 17:25:20 UTC
Re: svn commit: r1303856 - in /subversion/trunk/subversion:
bindings/swig/ruby/svn/fs.rb bindings/swig/ruby/test/test_info.rb
svnlook/main.c tests/cmdline/svnlook_tests.py
On Mar 22, 2012 11:56 AM, <ph...@apache.org> wrote:
>
> Author: philip
> Date: Thu Mar 22 15:56:30 2012
> New Revision: 1303856
>
> URL: http://svn.apache.org/viewvc?rev=1303856&view=rev
> Log:
> Ask for deltas during replay as this allows property deletes that
> remove all properties to be detected. Prior to r1293375 the replay
> driver sent a dummy delete which was used to detect these property
> changes.
>
> This reverts r1302524 as the Ruby tests have been fixed in the same
> way.
Edit svn:log on r1302524?
Re: svn commit: r1303856 - in /subversion/trunk/subversion:
bindings/swig/ruby/svn/fs.rb bindings/swig/ruby/test/test_info.rb
svnlook/main.c tests/cmdline/svnlook_tests.py
Posted by Greg Stein <gs...@gmail.com>.
On Thu, Mar 22, 2012 at 13:35, Philip Martin <ph...@wandisco.com> wrote:
> Greg Stein <gs...@gmail.com> writes:
>> On Mar 22, 2012 11:56 AM, <ph...@apache.org> wrote:
>>>...
>>> This reverts r1302524 as the Ruby tests have been fixed in the same
>>> way.
>>
>> Edit svn:log on r1302524?
>
> OK. I'm more interested in whether people think this is the right way
> to fix the problem. Is asking for deltas during replay the right way to
> ensure that the node editor sets prop_mod?
>From svn_repos_replay2():
* If @a send_deltas is @c TRUE, the text and property deltas for changes
* will be sent, otherwise NULL text deltas and empty prop changes will be
* used.
That seems like it was abusing the delta editor right from the start.
I might suggest that we kill the parameter altogether.
If generating and delivering text deltas and property diffs is "too
much work", then I believe functionality which requires the
minimal-work use a new interface (not Ev1 or Ev2).
Cheers,
-g
Re: svn commit: r1303856 - in /subversion/trunk/subversion: bindings/swig/ruby/svn/fs.rb bindings/swig/ruby/test/test_info.rb svnlook/main.c tests/cmdline/svnlook_tests.py
Posted by Philip Martin <ph...@wandisco.com>.
Greg Stein <gs...@gmail.com> writes:
> On Mar 22, 2012 11:56 AM, <ph...@apache.org> wrote:
>>
>> Author: philip
>> Date: Thu Mar 22 15:56:30 2012
>> New Revision: 1303856
>>
>> URL: http://svn.apache.org/viewvc?rev=1303856&view=rev
>> Log:
>> Ask for deltas during replay as this allows property deletes that
>> remove all properties to be detected. Prior to r1293375 the replay
>> driver sent a dummy delete which was used to detect these property
>> changes.
>>
>> This reverts r1302524 as the Ruby tests have been fixed in the same
>> way.
>
> Edit svn:log on r1302524?
OK. I'm more interested in whether people think this is the right way
to fix the problem. Is asking for deltas during replay the right way to
ensure that the node editor sets prop_mod?
--
uberSVN: Apache Subversion Made Easy
http://www.uberSVN.com