On 12/03/2010 06:51 AM, Julian Foad wrote:
> Noorul Islam K M wrote:
>> Julian Foad <julian.f...@wandisco.com> writes:

[...]

>> Index: subversion/svn/info-cmd.c
>> ===================================================================
>> --- subversion/svn/info-cmd.c        (revision 1041293)
>> +++ subversion/svn/info-cmd.c        (working copy)
>> @@ -584,12 +584,8 @@
>>              }
>>            else if (err->apr_err == SVN_ERR_RA_ILLEGAL_URL)
>>              {
>> -              SVN_ERR(svn_cmdline_fprintf
>> -                      (stderr, subpool,
>> -                       _("%s:  (Not a valid URL)\n\n"),
>> -                       svn_path_is_url(truepath)
>> -                         ? truepath
>> -                         : svn_dirent_local_style(truepath, pool)));
>> +                SVN_ERR(svn_cmdline_fprintf (stderr, subpool, 
>> +                                                 _("%s\n\n"), 
>> err->message));
> 
> Unfortunately we cannot assume that err->message is a good user-friendly
> description of the problem.

I've tweaked the documentation for svn_error_t in r1041891 to hopefully
point folks away from directly accessing err->message (among other things).

-- 
C. Michael Pilato <cmpil...@collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to