On Thu, May 28, 2020 at 8:33 AM Daniel Shahaf <d...@daniel.shahaf.name> wrote:
>
> svn-r...@apache.org wrote on Thu, 28 May 2020 04:00 -0000:
> > +++ subversion/branches/1.14.x/subversion/libsvn_client/mtcc.c Thu May 28 
> > 04:00:12 2020
> > @@ -453,7 +453,8 @@ mtcc_verify_create(svn_client__mtcc_t *m
> >
> >        if (op)
> >          return svn_error_createf(SVN_ERR_FS_ALREADY_EXISTS, NULL,
> > -                                 _("Path '%s' already exists"),
> > +                                 _("Path '%s' already exists, or was 
> > created "
> > +                                   "by an earlier operation"),
> >                                   new_relpath);
>
> Is "was created" in the correct tense?  This error message is triggered
> before the RA session is opened, so it shouldn't imply that the earlier
> operation completed successfully.

Hmm, I completely "read over" that. But you're probably right: with
"casual user glasses" on, this might be misunderstood. Though I'm not
sure how to rephrase it to make it clear enough, yet remain succinct.

-- 
Johan

Reply via email to