Re: svn commit: r31535 - in branches/1.5.x: . subversion/po

On Sun, Jun 1, 2008 at 10:04 PM, Karl Fogel <kfogel_at_red-bean.com> wrote:
> Blair Zajac <blair_at_orcaware.com> writes:
>> If we're concerned about people using pre 1.5 clients, then we should
>> update the collab.net start-commit to check for the mergeinfo
>> capability.
>>
>> Unfortunately, we can't check for capabilities in the pre-commit
>> script, so you couldn't check that capability only for paths on
>> branches/, so people could still use < 1.5 on trunk.
>
> Two questions:
>
> 1. In the future, should we add a capabilities check in pre-commit?
> (I haven't thought deeply about what's involved in doing that, but
> hey, we could at least get newer clients and servers doing it
> reliably -- the rest is a compatibility dance).
>
> 2. I don't quite understand your last paragraph. How is whether or
> not we can check a capability related to which paths the user is
> committing on? Could you say it more verbosely, maybe?

We could pass information from start-commit to pre-commit by setting a
txnprop. This is almost certainly a bad idea.

--dave

-- 
David Glasser | glasser@davidglasser.net | http://www.davidglasser.net/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org