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

Re: [PATCH] Add "copied" subcommand to svnlook

From: Peter N. Lundblad <peter_at_famlundblad.se>
Date: 2005-09-23 20:31:03 CEST

On Fri, 23 Sep 2005, C. Michael Pilato wrote:

> Christopher Ness <chris@nesser.org> writes:
>
> > What you have above seems to be the standard, minus the added
> > information about the "copy from" and the revision. Is that information
> > necessary in the output?
> >
> > > Seems like a judgement call from the big-picture guys is needed.
>
> My take:
>
> Don't add a new subcommand, just add a new flag --show-copy-info to

+1.

> 'svnlook changed'. When in this mode, simply tack " (from
> /copy/from/path:copyfromrev)" to the ends of lines where copies
> occured.
>
> So:
>
> $ svnlook changed /path/to/repos
> U trunk/build.conf
> D trunk/foo.c
> A trunk/bar.c
>
> becomes:
>
> $ svnlook changed /path/to/repos --show-copy-info
> U trunk/build.conf
> D trunk/foo.c
> A trunk/bar.c (from trunk/foo.c:14)
>
> This way, your output is *very* similar to the format used by 'svn
> log', and you don't break existing parsers of 'svnlook changed'.
>
Except that this isn't safe against spaces in pathnames. I like the
solution with two lines better.

Thanks,
//Peter

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Sep 23 20:32:27 2005

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.