You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Justin Erenkrantz <je...@apache.org> on 2002/06/07 20:22:55 UTC

Re: svn commit: rev 2113 - trunk/tools/cvs2svn

On Fri, Jun 07, 2002 at 02:24:55PM -0500, gstein@tigris.org wrote:
> Author: gstein
> Date: 2002-06-07 19:24 GMT
> New Revision: 2113
> 
> Removed:
>    trunk/tools/cvs2svn/CheckedClass.pm
>    trunk/tools/cvs2svn/cvs2svn.pl
>    trunk/tools/cvs2svn/tests/
> Log:
> The Perl stuff is way obsolete. Less functionality, and less
> performance to boot. bye bye...

As a meta comment, should the commit email really diff on deleted
files?  

Since SVN distributes the email script, this is on-topic, I
guess.  -- justin

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

Re: svn commit: rev 2113 - trunk/tools/cvs2svn

Posted by cm...@collab.net.
Greg Stein <gs...@lyra.org> writes:

> On Fri, Jun 07, 2002 at 01:22:55PM -0700, Justin Erenkrantz wrote:
> > On Fri, Jun 07, 2002 at 02:24:55PM -0500, gstein@tigris.org wrote:
> > > Author: gstein
> > > Date: 2002-06-07 19:24 GMT
> > > New Revision: 2113
> > > 
> > > Removed:
> > >    trunk/tools/cvs2svn/CheckedClass.pm
> > >    trunk/tools/cvs2svn/cvs2svn.pl
> > >    trunk/tools/cvs2svn/tests/
> > > Log:
> > > The Perl stuff is way obsolete. Less functionality, and less
> > > performance to boot. bye bye...
> > 
> > As a meta comment, should the commit email really diff on deleted
> > files?  
> > 
> > Since SVN distributes the email script, this is on-topic, I
> > guess.  -- justin
> 
> I'd say "no". Let's blame C.Mike. It's his nasty Perl script to send those
> emails. Let him fix it :-)

This work is not done in the commit-email script, but by `svnlook
diff'.  Regardless, the change would be an easy one to make.  However,
I rather like seeing all those -'s -- they stand out as a really bold
statement that something is being *really* cored out, perhaps even
deleted altogether.  Total personal preference thing, though.

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

Re: svn commit: rev 2113 - trunk/tools/cvs2svn

Posted by Greg Stein <gs...@lyra.org>.
On Fri, Jun 07, 2002 at 01:22:55PM -0700, Justin Erenkrantz wrote:
> On Fri, Jun 07, 2002 at 02:24:55PM -0500, gstein@tigris.org wrote:
> > Author: gstein
> > Date: 2002-06-07 19:24 GMT
> > New Revision: 2113
> > 
> > Removed:
> >    trunk/tools/cvs2svn/CheckedClass.pm
> >    trunk/tools/cvs2svn/cvs2svn.pl
> >    trunk/tools/cvs2svn/tests/
> > Log:
> > The Perl stuff is way obsolete. Less functionality, and less
> > performance to boot. bye bye...
> 
> As a meta comment, should the commit email really diff on deleted
> files?  
> 
> Since SVN distributes the email script, this is on-topic, I
> guess.  -- justin

I'd say "no". Let's blame C.Mike. It's his nasty Perl script to send those
emails. Let him fix it :-)

Cheers,
-g

-- 
Greg Stein, http://www.lyra.org/

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