Re: cvs2svn: Excluding CVS folders

Thanks Michael.

Making it non-optional is logical, and would appear to simplify the patch
down to a single file.  I was erring on the side of caution at the time,
and didn't want presume anything.


On Thu, Sep 19, 2013 at 12:10 AM, Michael Haggerty <[email protected]>wrote:

> On 04/20/2013 07:29 AM, David Morris wrote:
> > Here you go.  Again, I have not yet tried the command-line option
> > other than to see that it prints out the help.  All of our conversions
> > are options file driven so that's the only testing I've done.
>
> I just, very belatedly, pushed your change to trunk.
>
> I don't see a reason to make the behavior optional.  After all, if there
> were real content in a CVS directory, then it couldn't be checked out of
> CVS anyway.  So I changed the code to *always* exclude CVS directories.
>  If anybody can see a reason that anybody would want to include CVS
> subdirectories in a conversion, please speak up.
>
> I also applied the same logic to CVS subdirectories found within Attic
> directories, as presumably they can also occur there.
>
> Thanks for your contribution!
> Michael
>
> --
> Michael Haggerty
> [email protected]
> http://softwareswirl.blogspot.com/
>

------------------------------------------------------
http://cvs2svn.tigris.org/ds/viewMessage.do?dsForumId=1670&dsMessageId=3064852

To unsubscribe from this discussion, e-mail: [[email protected]].