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

Re: svn commit: r33288 - in trunk/subversion: svn tests/cmdline tests/cmdline/svntest

From: Blair Zajac <blair_at_orcaware.com>
Date: Thu, 25 Sep 2008 18:56:12 -0700

It sounds like the only two people who said anything about it agreed on the
change but we're waiting for feedback. So maybe after N days with no feedback
the change is made?

Anyway, this bikeshed is very easy, what character do you want in the 'svn
status' output for a new column for a tree conflict?

Blair

Greg Stein wrote:
> Yup... it *is* a bikeshed, but there does need to be a way to resolve
> it. So a random throw at "consensus" might not do it. Maybe something
> a little more proactive? I'm coming from the peanut gallery, so I
> don't really know the options. I'd love to provide some input (heh,
> "bikeshed input"), but it is unclear what is really going on here. I
> would guess that some others are similarly unaware. Kinda the nature
> where/how people pay attention.
>
> Thanks,
> -g
>
> On Thu, Sep 25, 2008 at 10:10 AM, Julian Foad
> <julianfoad_at_btopenworld.com> wrote:
>> On Thu, 2008-09-25 at 09:34 -0700, Blair Zajac wrote:
>>> julianfoad_at_tigris.org wrote:
>>>> Author: julianfoad
>>>> Date: Wed Sep 24 23:32:16 2008
>>>> New Revision: 33288
>>>>
>>>> Log:
>>>> In "svn status", report tree conflict status on each victim individually.
>>>> (For the time being, we still also report the parent as having tree-conflicted
>>>> children.) In the test suite, adjust a test that looks at this status output
>>>> directly. Provide support for this new status character to be tested using an
>>>> attribute of StateItem, which will be the normal way to test it, but make the
>>>> testing of this attribute optional and do not make any tests request it yet.
>>>>
>>>> * subversion/svn/status.c
>>>> (print_status): Print "T" in the tree status column if this is a tree
>>>> conflict victim.
>>> How about printing a "C" in that column instead. I think people are pretty used
>>> to seeing a "C" to mean conflict. And "T" can appear in the sixth lock column
>>> which will make it harder to distinguish if I have a tree conflict or a stolen
>>> lock without counting columns.
>> How about it? Sure, it's a fair option. It might well be better.
>> Suggestion noted, and I think I agree. I'll be happy to change it when
>> there's a consensus. Until then it's a bike-shed that requires changes
>> in a few places in the test suite and a test-build each time we change
>> it, so I'll avoid hasty action.
>>
>> Thanks.
>> - Julian

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-09-26 03:57:16 CEST

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.