On 24.12.2011 11:37, Stefan Sperling wrote:
> On Sat, Dec 24, 2011 at 09:41:30AM +0100, Stefan Küng wrote:
>> Yes, I'm ranting. But I think I have a reason to. Because I'm pretty
>> sure that the asserts in the svn library will stay - you guys made
>> it clear every time I brought this up that you rather have the app
>> crash than not have a chance to get a useful stack trace from users.
>
> Oh, come on. We are making commits to fix these issues and there
> are and have been corresponding nominations in STATUS to get these
> fixes backported ASAP.
>
> I think you tend to misunderstand people's writing tone sometimes.
> That's understandable, but please try to stop being so defensive.
> You shouldn't assume people in this community are working against you.
I know that you're now trying to improve the situation. That was my
intention after all.
When you guys first started abusing asserts, all reports for those came
to the TSVN mailing list. I had to answer those reports, find out the
reason (which takes a lot of emails back and fourth with those users)
and then come up with a reproduction recipe so I could report it here.
Back then, my complaints about that were completely ignored here.
Now that I've implemented a dialog in TSVN that tells users to report
the issue directly to the svn devs, now you're starting to notice how
much work this causes and how annoying it is.
So for me this is now a moment of "I told you so".
But then I read some comments from you guys that it's still my fault,
that I have to either change back the dialogs (sorry, won't happen: I've
got enough work with TSVN as it is) or that I'm using the malfunction
handler wrong.
So please forgive me if I seem a little upset with this discussion.
> And don't forget that you also have commit access to Subversion itself
> which means your voice counts as much as any other committer's voice and
> you can also help out making happen changes which TSVN needs.
> For instance, there is this nomination in STATUS right now which could use
> your vote:
>
> * r1222521, r1222693
> Turn another wc-db assertion people are reporting into a normal error.
> Justification:
> An informative error message helps people more than an assert (which
> in case of TSVN can mean crashing the Windows Explorer).
> Reported several times on users@, see:
> http://svn.haxx.se/users/archive-2011-11/0527.shtml
> http://svn.haxx.se/users/archive-2011-12/0063.shtml
> http://svn.haxx.se/users/archive-2011-12/0299.shtml
> Votes:
> +1: stsp
Looks good. Voted.
Stefan
--
___
oo // \\ "De Chelonian Mobile"
(_,\/ \_/ \ TortoiseSVN
\ \_/_\_/> The coolest Interface to (Sub)Version Control
/_/ \_\ http://tortoisesvn.net
Received on 2011-12-24 12:24:23 CET