RE: subversion and code peer reviews

On Fri, 2005-01-14 at 15:04 -0600, Rule, Chris wrote:
> Sorry. Should have included the svn version.
>
> $ svn --version
> svn, version 1.1.2 (r12471)
> compiled Dec 30 2004, 15:47:22
>
> I knew about the --diff-cmd options. And yes, TSVN will handle external diff
> programs. Difficult to do that via email from svn though.

viewcvs (http://viewcvs.sourceforge.net/) has a pretty good graphical
diff view. If you were to check in changes to a private branch, then
it would be quite simple to review the changes.

>
> -----Original Message-----
> From: Ben Collins-Sussman [mailto:sussman@collab.net]
> Sent: Friday, January 14, 2005 3:01 PM
> To: Rule, Chris
> Cc: users@subversion.tigris.org; Geoff Krapf
> Subject: Re: subversion and code peer reviews
>
> On Jan 14, 2005, at 2:48 PM, Rule, Chris wrote:
> >
> > Unified diffs don't seem useful to me. We have gotton used to (spoiled
> > by?)
> > graphical difference programs. The one we actually like the best is
> > gdiff on the SGI. I tend to use tkdiff and others in our group uses
> > windiff.
> > These
> > tools allow you to see the context of the changes.
>
> svn can use 3rd-party GUI diff programs too. See the --diff-cmd and
> --diff3-cmd args. I'm sure TSVN allows this also.... just write a wrapper
> script.
>
> >
> > I did try the "svn status" command where everything was on my local
> > machine.
> > It took 1 minute 45 seconds to execute. This is probably not going to
> > be acceptable. However, the TortiseSVN Check for Modifications menu
> > only took
> > 15 seconds. I haven't tried this across the network yet though I will
> > be.
> >
> > Machine details:
> > 2.8 GHz P4
> > 1 Gbyte RAM
> > Windows XP
> > Using svn:: for repository access.
> >
>
> What version of TSVN? The latest release (1.1.2) is much faster at status.
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: users-unsubscribe@subversion.tigris.org
> For additional commands, e-mail: users-help@subversion.tigris.org

-- 
------------------------------------------------------------
Gary Thomas                 |
MLB Associates              |  Consulting for the
+1 (970) 352-4947           |    Embedded world
http://www.mlbassoc.com/    |
email: <gary@mlbassoc.com>  |
gpg: http://www.chez-thomas.org/gary/gpg_key.asc
------------------------------------------------------------
---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@subversion.tigris.org
For additional commands, e-mail: users-help@subversion.tigris.org