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

Re: svn:auto-props on project root (was Re: svn commit: r1617096 - in /subversion/branches/svn-auth-x509/subversion: include/svn_x509.h libsvn_subr/x509.h libsvn_subr/x509info.c libsvn_subr/x509parse.c tests/libsvn_subr/x509-test.c)

From: Ivan Zhakov <ivan_at_visualsvn.com>
Date: Tue, 2 Sep 2014 14:56:52 +0400

On 2 September 2014 14:26, Branko Čibej <brane_at_wandisco.com> wrote:
> On 02.09.2014 10:50, Ivan Zhakov wrote:
>> On 18 August 2014 12:39, Branko Čibej <brane_at_wandisco.com> wrote:
>>> On 18.08.2014 09:21, Julian Foad wrote:
>>>
>>> Ivan Zhakov wrote:
>>>
>>> * subversion/include/svn_x509.h
>>> * subversion/libsvn_subr/x509.h
>>> * subversion/libsvn_subr/x509info.c
>>> * subversion/libsvn_subr/x509parse.c
>>> * subversion/tests/libsvn_subr/x509-test.c
>>> (): Fix line endings and set svn:eol-style to native.
>>>
>>> May be it's already discussed, but do someone have objections on
>>> settings 'svn:auto-props' on project root with content like this:
>>> [[[
>>> svn:auto-props
>>> *.py = svn:eol-style=native
>>> *.c = svn:eol-style=native
>>> *.h = svn:eol-style=native
>>> ]]]
>>>
>>> Sounds good to me. +1.
>>>
>>>
>>> There are a number of other patterns we want to add to that list: *.hpp,
>>> *.cpp, *.py, *.rb, *.pl, *.txt, README, BRANCH-README, etc. ad nauseam. I
>>> don't expect to find all of them on the first try, but we can sure try. :)
>>>
>> Done in r1621946. Thanks for your feedback.
>
> We'll need the same on ^/subversion/branches, too, since we do add new
> files on branches. And I wonder if it wouldn't be better to just add the
> property on ^/subversion and maintain it in one place.
>
I considered this approach, but decided to set svn:auto-props on trunk
only for the same reasons that Julian given: new branches will get
this property automatically, existing feature branches will this
property on next catch-up merge.

-- 
Ivan Zhakov
Received on 2014-09-02 12:57:41 CEST

This is an archived mail posted to the Subversion Dev mailing list.