Re: CVS update: subversion/subversion/client import-cmd.c

Greg Stein <gstein@lyra.org> writes:
> > Added: subversion/client import-cmd.c
> >...
> > /* Get a repository url. */
> > if (targets->nelts < 1)
> > return svn_error_create
> > (SVN_ERR_UNKNOWN_NODE_KIND, 0, NULL, pool,
> > "repository url required when importing");
> >...
> > else
> > return svn_error_create
> > (SVN_ERR_UNKNOWN_NODE_KIND, 0, NULL, pool,
> > "too many arguments to import command");
>
> I can see using placeholder error values, but please leave a comment that it
> needs to be updated. And maybe use something like APR_EGENERAL. The above
> error is just too whacky for this context :-)

Actually, it's just a cut-and-pasto (thanks for spotting it), not a
deliberate choice or anything as rational as that :-). Will fix.

-K
Received on Sat Oct 21 14:36:26 2006