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

Re: [PATCH] possible fix for #714, finally ;-)

From: Benjamin Pflugmann <benjamin-svn-dev_at_pflugmann.de>
Date: 2002-09-21 22:50:30 CEST

Hello.

On Sat 2002-09-21 at 22:27:08 +0200, To dev@subversion.tigris.org wrote:
[...]
> Please advise if the chosen method for returning the error and the
> suggested update syntax is acceptable. I am also not sure if the
> quotes may be written this way (regarding i18n). Or is there a quoting
> function available? Anyhow, the filename should be quoted, because it
> may contain special characters and the line is intended to work with
> copy&paste.

Sorry... forgot to include an example of the new output:

nora:~> svn revert -R /usr/local/src/phi/test/one/bla
subversion/libsvn_wc/adm_ops.c:972: (apr_err=200007, src_err=0)
svn: Trying to use an unsupported feature
svn: revert: error using subcommand for `/usr/local/src/phi/test/one/bla'
subversion/libsvn_wc/adm_ops.c:1009: (apr_err=200007, src_err=0)
svn: try instead: svn update '/usr/local/src/phi/test/one/bla'
Exit 1

The message lacks an explanation that the error only happens because
/usr/local/src/phi/test/one/bla does not exist. But I did not find a
way to include that without creating bloat. As you probably noticed
meanwhile, I wouldn't call English one of my strengths. So any
suggestion are welcome, of course.

Ciao,

        Benjamin.

PS: Forgot to mention: of course, passed "make check" and some
    hand-made tests.

PPS: Btw, I did not include any test cases for any of my patches,
     because, after looking for half an hour, I was still not sure
     were to begin with. Is there some doc that I missed?

  • application/pgp-signature attachment: stored
Received on Sat Sep 21 22:51:12 2002

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