Repo dictated config (was Re: svn commit: r1781441 - /ofbiz/trunk/)

[Jacques Le Roux]

Hi Committers,

With r1781445+1781446+1781447, I have now removed the "svn:auto-props" at the trunk level and put all at the OFBiz svn repo root level

Now, every committer must be sure to use a svn 1.8 client. Because older clients don't support the svn:auto-props in repo

Committers will still need to use the svn config file for other stuff. But if they don't, at least the "svn:eol-style=native" aspect will not be forgotten

Note though that this only works on new checkout/working copies done with a svn 1.8 client...  Documentation:

http://subversion.apache.org/docs/release-notes/1.8.html#repos-dictated-config
http://blogs.collab.net/subversion/the-road-to-repository-dictated-configuration-day-2-autoprops

So, if you still use an old working copy, either with a svn 1.8, or prior version, client, please be sure to have the svn config correctly put in the
right place, see

https://cwiki.apache.org/confluence/display/OFBADMIN/OFBiz+Committers+Roles+and+Responsibilities#OFBizCommittersRolesandResponsibilities-CommittingChanges

Previously we used the sentence "Committers should set up their SVN client to use the official OFBIZ Subversion client configuration file"
<http://ofbiz.apache.org/svn/config>

It should have really been

"Committers MUST set up their SVN client to use the official OFBIZ Subversion client configuration file" <http://ofbiz.apache.org/svn/config>

But it will no longer be a problem for new committers as long as they use a svn 1.8 client. You, committers, still need to be sure you are doing the
same with new working copies


Thanks

Jacques


Le 02/02/2017 à 18:56, [hidden email] a écrit :

> Author: jleroux
> Date: Thu Feb  2 17:56:14 2017
> New Revision: 1781441
>
> URL: http://svn.apache.org/viewvc?rev=1781441&view=rev
> Log:
> No functional change. Tries to set the svn:auto-props on OFBiz svn repo root
>
> Modified:
>      ofbiz/trunk/   (props changed)
>
> Propchange: ofbiz/trunk/
> ------------------------------------------------------------------------------
> --- svn:auto-props (added)
> +++ svn:auto-props Thu Feb  2 17:56:14 2017
> @@ -0,0 +1,38 @@
> +*.java       = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
> +*.bsh        = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
> +*.groovy     = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
> +*.jsp        = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
> +*.tld        = svn:mime-type=text/xml;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
> +*.ftl        = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
> +*.js         = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
> +*.sql        = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
> +*.c          = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
> +*.cpp        = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
> +*.h          = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
> +*.txt        = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
> +*.sgml       = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
> +*.properties = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
> +*.xml        = svn:mime-type=text/xml;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
> +*.classpath  = svn:mime-type=text/xml;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
> +*.project    = svn:mime-type=text/xml;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
> +*.sld        = svn:mime-type=text/xml;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
> +*.gml        = svn:mime-type=text/xml;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
> +*.xsl        = svn:mime-type=text/xsl;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
> +*.xsd        = svn:mime-type=text/xsd;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
> +*.html       = svn:mime-type=text/html;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
> +*.htm        = svn:mime-type=text/html;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
> +*.css        = svn:mime-type=text/css;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
> +*.gradle     = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
> +*.md         = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
> +Makefile     = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
> +README       = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
> +NOTICE       = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
> +LICENSE      = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
> +ant          = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
> +ij.ofbiz     = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
> +rc.ofbiz.*   = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
> +*.less       = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
> +*.bat        = svn:mime-type=text/plain;svn:eol-style=native;svn:executable;svn:keywords="Date Rev Author URL Id"
> +*.sh         = svn:mime-type=text/plain;svn:eol-style=native;svn:executable;svn:keywords="Date Rev Author URL Id"
> +*.dsp        = svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
> +*.dsw        = svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
>
>
>

[Jacques Le Roux]

Maybe some wonder why it was a problem. It's was a 2 phases issue.

When some committers were not putting the svn config file in the right place, after editing a file on another platform you got mixed EOLs in the file,
so the impossibility to commit the file w/o changing all EOLs.
This can be tedious when you edit a bunch of files. Fortunately simply setting svn:native on the file fixes both issues. But then you get another
issue: you loose the annotations (last history changes by line) for concerned files.

Now, even if a committer forget to set the svn config file, the svn:native property will be automatically set when a file will be committed the 1st
time which prevents those issues.

There are other properties which are not [auto-props] so putting the svn config file in the right place is still needed but in case it's forgotten the
impact will be less important (please don't take that as an excuse to not set it ;))

Jacques


Le 02/02/2017 à 19:41, Jacques Le Roux a écrit :

> Hi Committers,
>
> With r1781445+1781446+1781447, I have now removed the "svn:auto-props" at the trunk level and put all at the OFBiz svn repo root level
>
> Now, every committer must be sure to use a svn 1.8 client. Because older clients don't support the svn:auto-props in repo
>
> Committers will still need to use the svn config file for other stuff. But if they don't, at least the "svn:eol-style=native" aspect will not be
> forgotten
>
> Note though that this only works on new checkout/working copies done with a svn 1.8 client...  Documentation:
>
> http://subversion.apache.org/docs/release-notes/1.8.html#repos-dictated-config
> http://blogs.collab.net/subversion/the-road-to-repository-dictated-configuration-day-2-autoprops
>
> So, if you still use an old working copy, either with a svn 1.8, or prior version, client, please be sure to have the svn config correctly put in
> the right place, see
>
> https://cwiki.apache.org/confluence/display/OFBADMIN/OFBiz+Committers+Roles+and+Responsibilities#OFBizCommittersRolesandResponsibilities-CommittingChanges 
>
>
> Previously we used the sentence "Committers should set up their SVN client to use the official OFBIZ Subversion client configuration file"
> <http://ofbiz.apache.org/svn/config>
>
> It should have really been
>
> "Committers MUST set up their SVN client to use the official OFBIZ Subversion client configuration file" <http://ofbiz.apache.org/svn/config>
>
> But it will no longer be a problem for new committers as long as they use a svn 1.8 client. You, committers, still need to be sure you are doing the
> same with new working copies
>
>
> Thanks
>
> Jacques
>
>
> Le 02/02/2017 à 18:56, [hidden email] a écrit :
>> Author: jleroux
>> Date: Thu Feb  2 17:56:14 2017
>> New Revision: 1781441
>>
>> URL: http://svn.apache.org/viewvc?rev=1781441&view=rev
>> Log:
>> No functional change. Tries to set the svn:auto-props on OFBiz svn repo root
>>
>> Modified:
>>      ofbiz/trunk/   (props changed)
>>
>> Propchange: ofbiz/trunk/
>> ------------------------------------------------------------------------------
>> --- svn:auto-props (added)
>> +++ svn:auto-props Thu Feb  2 17:56:14 2017
>> @@ -0,0 +1,38 @@
>> +*.java       = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
>> +*.bsh        = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
>> +*.groovy     = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
>> +*.jsp        = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
>> +*.tld        = svn:mime-type=text/xml;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
>> +*.ftl        = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
>> +*.js         = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
>> +*.sql        = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
>> +*.c          = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
>> +*.cpp        = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
>> +*.h          = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
>> +*.txt        = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
>> +*.sgml       = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
>> +*.properties = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
>> +*.xml        = svn:mime-type=text/xml;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
>> +*.classpath  = svn:mime-type=text/xml;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
>> +*.project    = svn:mime-type=text/xml;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
>> +*.sld        = svn:mime-type=text/xml;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
>> +*.gml        = svn:mime-type=text/xml;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
>> +*.xsl        = svn:mime-type=text/xsl;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
>> +*.xsd        = svn:mime-type=text/xsd;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
>> +*.html       = svn:mime-type=text/html;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
>> +*.htm        = svn:mime-type=text/html;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
>> +*.css        = svn:mime-type=text/css;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
>> +*.gradle     = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
>> +*.md         = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
>> +Makefile     = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
>> +README       = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
>> +NOTICE       = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
>> +LICENSE      = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
>> +ant          = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
>> +ij.ofbiz     = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
>> +rc.ofbiz.*   = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
>> +*.less       = svn:mime-type=text/plain;svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
>> +*.bat        = svn:mime-type=text/plain;svn:eol-style=native;svn:executable;svn:keywords="Date Rev Author URL Id"
>> +*.sh         = svn:mime-type=text/plain;svn:eol-style=native;svn:executable;svn:keywords="Date Rev Author URL Id"
>> +*.dsp        = svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
>> +*.dsw        = svn:eol-style=native;svn:keywords="Date Rev Author URL Id"
>>
>>
>>
>
>