You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Branko Čibej <br...@wandisco.com> on 2014/09/17 15:40:44 UTC

Re: svn commit: r1623660 - /subversion/trunk/tools/client-side/svn-vendor.py

On 09.09.2014 07:57, stilor@apache.org wrote:
> Author: stilor
> Date: Tue Sep  9 05:57:36 2014
> New Revision: 1623660
>
> URL: http://svn.apache.org/r1623660
> Log:
> * tools/client-side/svn-vendor.py
>   New script, an alternative to svn_load_dirs.pl that can auto-detect
>   renames and copies.

[...]

> +#########################################################################################

Please reformat the script so that lines aren't longer than about 78
columns.

> +# svn-import.py
The doc mentions "svn-import.py" in several places; should be renamed?

-- Brane


Re: svn commit: r1623660 - /subversion/trunk/tools/client-side/svn-vendor.py

Posted by Alexey Neyman <st...@att.net>.
On Wednesday, September 17, 2014 06:40:44 am Branko Čibej wrote:
> On 09.09.2014 07:57, stilor@apache.org wrote:
> > Author: stilor
> > Date: Tue Sep  9 05:57:36 2014
> > New Revision: 1623660
> > 
> > URL: http://svn.apache.org/r1623660
> > Log:
> > * tools/client-side/svn-vendor.py
> > 
> >   New script, an alternative to svn_load_dirs.pl that can auto-detect
> >   renames and copies.
> 
> [...]
> 
> > +########################################################################
> > #################
> 
> Please reformat the script so that lines aren't longer than about 78
> columns.
> 
> > +# svn-import.py
> 
> The doc mentions "svn-import.py" in several places; should be renamed?
> 
> -- Brane

Committed revision 1627828.

Regards,
Alexey.