Re: [PATCH] Fix typos in error messages

Michael W Thelen wrote:

>Index: subversion/libsvn_client/commit_util.c
>===================================================================
>--- subversion/libsvn_client/commit_util.c (revision 9929)
>+++ subversion/libsvn_client/commit_util.c (working copy)
>@@ -641,7 +641,7 @@
> return svn_error_createf
> (SVN_ERR_ILLEGAL_TARGET, NULL,
> "Entry for '%s' is marked as 'copied' but is not itself scheduled\n"
>- "for addition. Perhaps you're committing a target that this\n"
>+ "for addition. Perhaps you're committing a target that is\n"
> "inside of an unversioned (or not-yet-versioned) directory?",
> target);
>
>
While you're at it, you might've removed that "of" after the "inside"...

>Index: subversion/svnlook/main.c
>===================================================================
>--- subversion/svnlook/main.c (revision 9929)
>+++ subversion/svnlook/main.c (working copy)
>@@ -1893,7 +1893,7 @@
> SVN_INT_ERR (svn_error_create
> (SVN_ERR_CL_MUTUALLY_EXCLUSIVE_ARGS, NULL,
> _("The '--transaction' (-t) and '--revision' (-r) arguments "
>- "can not co-exist")));
>+ "cannot co-exist")));
>
>
Nope. "can not" is perfectly correct spelling, and the emphasis is
correct for this case, too..

>Index: subversion/mod_dav_svn/version.c
>===================================================================
>--- subversion/mod_dav_svn/version.c (revision 9929)
>+++ subversion/mod_dav_svn/version.c (working copy)
>@@ -308,7 +308,7 @@
> {
> return dav_new_error_tag(resource->pool, HTTP_NOT_IMPLEMENTED,
> SVN_ERR_UNSUPPORTED_FEATURE,
>- "CHECKOUT can not create an activity at this "
>+ "CHECKOUT cannot create an activity at this "
>
>
Same here.

By the way: Once we have translations bundled with the release, we
should also bundle an en.po that contains a simple 1:1 mapping between
the text in the source and in the message file. Then we can fix typos in
messages, and distribute the fix, /without/ having to recompile everything.

-- 
Brane Čibej   <brane_at_xbc.nu>   http://www.xbc.nu/brane/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org