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

Re: tsvn:logwidth in svn repository (Re: svn commit: r12696 - trunk/subversion/po)

From: SteveKing <steveking_at_gmx.ch>
Date: 2005-01-14 17:34:40 CET

Branko Čibej wrote:
> steveking wrote:
>
>> Ben Collins-Sussman wrote:
>>>
>>> Well, users have been asking for server-settings that can be
>>> broadcast to clients for quite a while now.. I guess we've ignored
>>> the request long enough to the point that other projects have started
>>> improvising workaround solutions. :-/
>>
>> That's all I can do. It seems I'm just not able to explain why a
>> feature is needed so the svn devs usually agree to not do it.
>
> That.s BS. We know very well why this feature would be useful, it simply
> hasn't floated to the top of anyones priority list yet.

I didn't get that impression from previous discussions. And the comments
in the issue aren't so that I got the impression that Subversion will
ever have _custom_ repo configs.

>> After a while (usually as soon as someone else explains it better than
>> me) such features sometimes get implemented in Subversion after all.
>> But I can't always wait that long. So don't feel bad - it's my fault.
>
> There's one thing I don't understand: Why didn't you implement inherited
> props in the SVN code instead of mucking with workarounds in TSVN? It's
> not as if we reject good patches.

I've already sent several patches to this list. Every one of them got
rejected for different reasons. Put aside those that got rejected
because they wouldn't fit into Subversions concept, here two examples:

1. A three line patch making authentication work on Win98/Me systems
because there the get_UID() function doesn't work:
http://subversion.tigris.org/servlets/ReadMsg?list=dev&msgNo=37397
Got rejected because it would belong into apr. But now, the
authentication does exactly this - a little different of course because
the whole auth stuff has been rewritten since then.

2. A bigger patch to make Subversion build as a dll on windows to avoid
static linking:
http://subversion.tigris.org/servlets/ReadMsg?list=dev&msgNo=33674
Got rejected because (even though the patch uses the exact same approach
as apr does) it wasn't the way the devs wanted it done. A generator for
def files was the choice, but Subversion doesn't build as a dll even
almost two years later...

So you might see now why I don't write patches for Subversion anymore. I
would just spend a lot of time which most likely would be for nothing -
I'd rather spend that time on TSVN where I know it will do some good.

Stefan

-- 
        ___
   oo  // \\      "De Chelonian Mobile"
  (_,\/ \_/ \     TortoiseSVN
    \ \_/_\_/>    The coolest Interface to (Sub)Version Control
    /_/   \_\     http://tortoisesvn.tigris.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Jan 14 17:36:00 2005

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.