Re: [PATCH] new parameter option '--full-paths' for 'svnlook tree'

Mathias.Weinert@gfa-net.de writes:

> I now made a new patch against revision 13968 and attached it to this
> mail.

Please include a log message when you submit patches.

> I hope that it is in the right format and that you can read and use it.

A mime-type of text/plain is better than application/octet.

> I also tried to do some python programming. I have no idea if it is o.k.
> and
> if it is what you expected. I added two 'svnlook tree' calls in the first
> test
> in svnlook_tests.py. They both check if 'svnlook tree --full-paths' fits
> the
> normal 'svnlook tree' results but don't test the output of 'svnlook tree'
> itself.

Looks OK, but I had to change lstrip(' ') to lstrip() and rstrip('\n')
to rstrip() to avoid errors like:

TypeError: strip() takes exactly 0 arguments (1 given)

I'm using python 2.1.3, perhaps you have something different?

Thanks for the patch, I committed it in r13976.

-- 
Philip Martin
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org