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

Re: svn commit: r18764 - trunk/www

From: <kfogel_at_collab.net>
Date: 2006-03-08 06:26:40 CET

offby1@tigris.org writes:
> Log:
> * www/faq.html (terse-diff): Give two recipes: one for the general
> case where we're interested in a range of revisions, and (the
> original) one for the special case of two adjacent revisions.
>
>
> Modified: trunk/www/faq.html
> Url: http://svn.collab.net/viewcvs/svn/trunk/www/faq.html?rev=18764&p1=trunk/www/faq.html&p2=trunk/www/faq.html&r1=18763&r2=18764
> ==============================================================================
> --- trunk/www/faq.html (original)
> +++ trunk/www/faq.html Tue Mar 7 22:46:29 2006
> @@ -2124,8 +2124,18 @@
> <div class="h3" id="terse-diff" title="terse-diff"><h3>How can I
> make <tt>svn diff</tt> show me just the names of the changed
> files, not their contents?</h3>
> -<tt>svn diff</tt> doesn't have an option to do this, but <tt>svn log
> - -vq</tt> does exactly what you want.
> +<tt>svn diff</tt> doesn't have an option to do this, but
> +<ul>
> +<li>
> + If you only are interested in the diffs between two adjacent versions,
> + <pre>svn log -vq -r10:11</pre> does exactly what you want;
> +</li>

Did you test this? :-)

If you pass a revision range (in this case, the two revisions 10:11),
you will see both revisions in the output. What you really want is
just "-r11", with the surrounding explanatory text adjusted as
necessary of course.

> +<li>
> + otherwise, if you're fortunate enough to be using Unix, this works
> + for any range of versions:
> + <pre>
> + svn log -vq -r123:456 working-copy | egrep '^ +M' | awk '{print $2}' | sort | uniq </pre> </li>
> +</ul>
> </div>

I saw you added 'A' and 'D' to the regexp in r18766, but don't forget
'R'eplace.

You might also want to make the regexp tighter. Why allow any number
of spaces at the start of the line, when we know there are always 3?
Why not follow the code letter with a space too? We might as well
avoid false matches as much as possible.

Best,
-Karl

-- 
www.collab.net  <>  CollabNet  |  Distributed Development On Demand
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Mar 8 08:13:55 2006

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