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

Re: [PATCH] Fix for issue 2533: status -u doesn't show remote propchanges on the root folder

From: Daniel Rall <dlr_at_collab.net>
Date: 2006-09-28 20:37:19 CEST

On Wed, 27 Sep 2006, Lieven Govaerts wrote:

> Daniel Rall wrote:
> >On Tue, 26 Sep 2006, Daniel Rall wrote:
> >
> >
> >>On Tue, 26 Sep 2006, Lieven Govaerts wrote:
> >>...
> >>
> >>>We need some more status regression tests, do you mind writing one
> >>>for this scenario? I'll probably add some more later.
> >>>
> >>I modified the test for issue #2533 (status test #24) to also use
> >>'-vN', which is a good regression test variation in the first place.
> >>
> >...
> >
> >I committed the test case as XFail, so it'll be there available when
> >you start testing tomorrow, Paul.
> >
> I've made a new version of the patch as committed in r21638 which
> doesn't store the root folder in the status array, but solves it locally
> in the close_directory call.
>
> It seems like this patch keeps the fix for issue 2533, without the bad
> side-effects of the previous one.
>
> Note: this patch is based on r21637, so to test it you first have to
> revert r21638 (at least the changes in status.c).
...

This fixed #2533 in a much simpler fashion, so I committed a variation
of it yesterday with some tweaks, and removed the XFail from our test
case.

Lieven indicated he'd work on a final test case variation to get us
complete regression coverage.

  • application/pgp-signature attachment: stored
Received on Thu Sep 28 20:36:45 2006

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.