Re: Ruby bindings failing on buildbots

Joe Swatosh <joe.swatosh_at_gmail.com> writes:

> Before r1293375, the node that represented 'diff1.txt' in this walk
> had the "prop_mod" member set true. After r1293375, the "prop_mod"
> member for this node is false. Since the implementation of
> Svn::Info#get_diff_recurse checks "prop_mod" member before attempting
> to collect the property changes on the node, the test started failing.

I don't understand why prop_mod is no longer being set.

> My second patch removes the (optimization?) check for the "prop_mod"
> member and thus always get the properties from the prior revision and
> compares them with the current version. Making the test pass again for
> me.

That also looks dubious to me.

> Is the prop_mod not being set in this scenario for this node the
> desired behavior?

In the past a property delete caused prop_mod to be set and now it does
not. The Subversion core is still reporting the property delete to the
Ruby bindings layer at ChangedEditor.change_file_prop. I don't
understand the Ruby code so I don't know why prop_mod is no longer set
but it seems like a bug to me.

-- 
uberSVN: Apache Subversion Made Easy
http://www.uberSVN.com