Re: Feature enhancement for cvs2svn

Brian W. Fitzpatrick <fitz@red-bean.com>:
> Can you write a log message to go with your patch? See
> http://svn.collab.net/repos/cvs2svn/trunk/HACKING for log message
> guidelines.

I'll do that when I submit version 2 of this patch.
 
> > +# This should probably be local to some class
> > +tag_transforms = []
> > +
>
> Please toss it into the Ctx class along with the rest of the application
> defaults.

That's how I started to do it originally. Then I discovered that Ctx
doesn't seem to be available at the point where I need the
tag-transforms list. Changing that will make this a rather more
intrusive patch...but if that's how you want it, I'll do it that way.

> Hmm... since this transform is an expected behavior, I'd prefer to see
> the warning printed to STDOUT via the cvs2svn log facility...

Done.

Now I need to figure out how to make Ctx visible at the right place.

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