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

Re: [patch]: cvs2svn not synced with current svn_repos API

From: Karl Fogel <kfogel_at_newton.ch.collab.net>
Date: 2003-02-24 17:04:38 CET

David Kimdon <dwhedon@debian.org> writes:
> This fixes a broken cvs2svn for me. Passing the null config looks okay since
> svn_repos_create will obtain a config otherwise if it needs it. If this looks
> good I will commit. I note that this change is also missing from the
> cvs2svn-mmacek branch. Should I commit to the branch as well?

Sure, thanks. I'll be merging that branch soon, so we can concentrate
dev on the trunk, but it'll help to have the change in there first, to
do independent work with the branch first.

-K

P.S. The change looks good to me; haven't tested, but it makes sense.

> * tools/cvs2svn/cvs2svn.py (pass4) : Update to current with libsvn_repos API.
> Pass a config NULL config argument to repos.svn_repos_create().
>
>
> Index: tools/cvs2svn/cvs2svn.py
> ===================================================================
> --- tools/cvs2svn/cvs2svn.py (revision 5022)
> +++ tools/cvs2svn/cvs2svn.py (working copy)
> @@ -550,7 +550,7 @@
> # create the target repository
> if not ctx.dry_run:
> if ctx.create_repos:
> - t_repos = repos.svn_repos_create(ctx.target, None, None, ctx.pool)
> + t_repos = repos.svn_repos_create(ctx.target, None, None, None, ctx.pool)
> else:
> t_repos = repos.svn_repos_open(ctx.target, ctx.pool)
> t_fs = repos.svn_repos_fs(t_repos)
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
> For additional commands, e-mail: dev-help@subversion.tigris.org

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon Feb 24 17:39:01 2003

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.