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

Re: svn commit: r1185746 - Fix up some erroneous "Could not frob some targets because..." warnings.

From: Stefan Sperling <stsp_at_apache.org>
Date: Wed, 19 Oct 2011 11:48:50 +0200

On Wed, Oct 19, 2011 at 10:13:48AM +0100, Julian Foad wrote:
> --- On Tue, 18/10/11, stsp_at_apache.org wrote:
> > Modified: subversion/trunk/subversion/svn/add-cmd.c
> > ==============================================================
> > @@ -80,21 +79,34 @@ svn_cl__add(apr_getopt_t *os,
> > SVN_ERR_ENTRY_EXISTS, SVN_ERR_WC_PATH_NOT_FOUND, SVN_NO_ERROR));
> [...]
> > +          else if (status == SVN_ERR_UNVERSIONED_RESOURCE)
> > +            err = svn_error_quick_wrap(err,
> > +              _("Could not add all targets because "
> > +                "some targets are already versioned"));
>
> That error status doesn't look right; should be _ENTRY_EXISTS?
>

Yes. And the one above it is also wrong. r1186059, thanks!
Received on 2011-10-19 11:49:32 CEST

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