Re: [PATCH] check name svn special properties

Dear developers,

> Please find attached a patch which checks the name of svn special properties
> and refuses unexpected ones. The check are performed in the client library,
> where it seems to belong. The error can be overridden with --force if
> necessary.
>
> The new feature is tested in the regression stuff. I had to remove 6 lines of
> some tests which were setting invalid svn special properties, which are
> now forbidden by default.

May I gather after all the discussions that the submitted patch is
rejected because of a lack of consensus? ;-)

Just my 0.02 euro-cents on the various views expressed in the discussion:

I think that pre-commit is too late for a small stupid typo in an svn
property name. Moreover it requires an additionnal setup for a feature
that everybody should require, so that should be the default.

I agree that having a nice "per-repository" client configuration would be
a good idea and it would be the better place to put the constraint in to
be exported to the client. But I don't see it coming, and I my typos are
now;-)

I think that the "hard coded" check I submitted could be removed once such
a nicer dynamically configured checking is in place, so the very small
patch I submitted could have been a temporary relief while waiting for the
"grand" feature, some day... ;-)

Thanks for having considered this patch anyway, have a nice day,

-- 
Fabien.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org