Re: svn commit: r1663738 - in /subversion/trunk/subversion: libsvn_fs/fs-loader.c tests/cmdline/svnlook_tests.py tests/libsvn_fs/fs-test.c

Bert Huijben <bert_at_qqmail.nl> writes:

>> Don't expose internal transaction properties like 'svn:check-locks' through
>> the public FS API and error out on attempts to change them.
>
> I wish it would be possible to put these in their own property namespace,
> to avoid those list of properties to check against...
>
> But that is not really possible any more, is it?

I am afraid it is not possible for the reasons Philip has already mentioned.
As long as we use these properties in the on-disk format, we cannot alter
their names without breaking compatibility.

Unfortunately, I am unaware of a solution that would allow us to stop checking
these properties against a predefined list of names. Even with theoretically
possible options like using a special namespace in the newer filesystem formats
or making these filesystems use something else than a set of temporary props
for the transaction flags, we'd still require this check somewhere in order to
be backward compatible.

Regards,
Evgeny Kotkov
Received on 2015-03-04 02:43:45 CET