Re: svn commit: r1396184 - in /subversion/branches/auto-props-sdc/subversion: include/svn_props.h libsvn_client/add.c libsvn_client/client.h libsvn_client/commit.c tests/cmdline/autoprop_tests.py tests/cmdline/svntest/main.py

On Tue, Oct 9, 2012 at 8:46 PM, Mark Phippard <markphip_at_gmail.com> wrote:
> On Tue, Oct 9, 2012 at 2:41 PM, <pburba_at_apache.org> wrote:
>> Author: pburba
>> Date: Tue Oct 9 18:41:55 2012
>> New Revision: 1396184
>>
>> URL: http://svn.apache.org/viewvc?rev=1396184&view=rev
>> Log:
>> On the auto-props-sdc branch: Rename the reserved property
>> "svn:config:auto-props" to svn:iprop-auto-props".
>>
>> First and foremost this is to address cmpilato's concerns, see
>> http://svn.haxx.se/dev/archive-2012-10/0097.shtml But it is also a bit of
>> bike-shedding; since the old name contained the word "config" it needlessly
>> conflated itself with the config file which is home of the traditional
>> auto-props. I think the new name makes it clearer that these auto-props
>> are outside of the config file.
>
> I liked the use of "config" precisely because this property IS
> somewhat connected to what is in that file. I am not against dropping
> config, but if we do, then I would also drop iprop and just make the
> property be svn:auto-props

+1.

I don't like the term "iprop" (assuming that stands for "inheritable
property"), because as I understood it all properties are inheritable
(and it's up to the "user" of that property to decide to use it in an
inheritable manner or not).

-- 
Johan