> -----Original Message----- > From: Joe Swatosh [mailto:joe.swat...@gmail.com] > Sent: vrijdag 30 juli 2010 16:33 > To: Bert Huijben > Cc: dev@subversion.apache.org; julian.f...@wandisco.com; Hyrum K. > Wright > Subject: Re: svn commit: r959954 - > /subversion/trunk/subversion/libsvn_wc/wc.h > > On Fri, Jul 30, 2010 at 1:21 AM, Bert Huijben <b...@qqmail.nl> wrote: > > > > > >> -----Original Message----- > >> From: Joe Swatosh [mailto:joe.swat...@gmail.com] > >> Sent: vrijdag 30 juli 2010 3:50 > > > > > Julian is looking at the issue now. > > > > There is one more issue in the Ruby tests and you might be able to > give me a > > hint on where I can fix this. > > > > Shortly before switching to in-db properties I made it an error to > update > > properties on nodes that shouldn't have editable properties. (E.g. > deleted > > nodes) > > > > The Ruby tests trigger this error when they try to delete some > mergeinfo, > > but this error is a few levels deep in some lambda processing so I > find it > > hard to find out what the expected status should be here. > > > > Thanks to you and Julian for fixing this! > > Those merge test failures are the other thing I was looking at when I > confused myself about which revision started causing which failures. > The merge tests in the Ruby bindings drive me nuts. They are > complicated and fragile, and I don't like that they mess about with > the mergeinfo property directly; mergeinfo _feels_ like it should be > an implementation detail. So, while I was reasonably sure that the > bindings weren't the problem in what you tested and Julian fixed, I'm > going to have to spend some time with these to figure out what is > what. > > What you describe above is true, but my initial tracing (just doing a > svn st just before the delete) shows that the node already has > property modifications (svn:mergeinfo is the only property), but isn't > marked for deletion.
I think r981319 is the proper fix for this issue. Bert