Login | Register
My pages Projects Community openCollabNet

Discussions > dev [DISABLED] > Re: svn commit: r37709 - trunk/subversion/libsvn_subr

subversion
Discussion topic

Back to topic list

Re: svn commit: r37709 - trunk/subversion/libsvn_subr

Author arfrever
Full name Arfrever Frehtes Taifersar Arahesis
Date 2009-05-13 03:30:56 PDT
Message 2009-05-12 21:11 Greg Stein <gstein at gmail dot com> napisaƂ(a):
> On Tue, May 12, 2009 at 20:20, Branko Cibej <brane at xbc dot nu> wrote:
>> Arfrever Frehtes Taifersar Arahesis wrote:
>>>> Why is this necessary?
>>>>
>>>> If the error is useful, the full error message can be returned (as wrapped
>>>> error?) in the svn_error_t* returned by this function
>>>>
>>>
>>> The doc string of svn_dso_load() contains:
>>> "If @a libname cannot be loaded set @a dso to NULL and return @c SVN_NO_ERROR."
>>>
>>> The error message might be useful to find bugs such as the one introduced
>>> in r35309+r35385+r36823.
>>>
>>
>> Never mind if it might be useful; we do not print anything from our
>> libraries. It's just Not Done and I don't care about broken libraries
>> that do expect to be able to print anything.
>
> To be fair, the print is within SVN_DEBUG which means "only svn devs"
>
> But yah: rev the API and improve the error return. Consumers can
> always throw out the error.

`svn`, `svnsync` etc. should probably print the error message
retrieved from apr_dso_error(), but they must try to continue to work
when dynamic loading of a library failed. For example, when dynamic
loading of libsvn_ra_neon-1.so.0 fails during accessing a repository
through http:// protocol, libsvn_ra_serf-1.so.0 might be available, so
libsvn_ra-1.so.0 should try to dynamically load libsvn_ra_serf-1.so.0.

svn_dso_load() is called only by Subversion libraries (libsvn_subr,
libsvn_fs, libsvn_subr), not by `svn` and other programs, so I'm not
sure how to implement sending of error message from a library function
(which calls svn_dso_load()) to a program without returning error from
this function.

--
Arfrever Frehtes Taifersar Arahesis

« Previous message in topic | 6 of 9 | Next message in topic »

Messages

Show all messages in topic

RE: svn commit: r37709 - trunk/subversion/libsvn_subr rhuijben Bert Huijben 2009-05-12 08:42:06 PDT
     Re: svn commit: r37709 - trunk/subversion/libsvn_subr arfrever Arfrever Frehtes Taifersar Arahesis 2009-05-12 09:11:42 PDT
         Re: svn commit: r37709 - trunk/subversion/libsvn_subr gstein Greg Stein 2009-05-12 09:18:51 PDT
         Re: svn commit: r37709 - trunk/subversion/libsvn_subr brane Branko Cibej 2009-05-12 11:21:02 PDT
             Re: svn commit: r37709 - trunk/subversion/libsvn_subr gstein Greg Stein 2009-05-12 12:11:42 PDT
                 Re: svn commit: r37709 - trunk/subversion/libsvn_subr arfrever Arfrever Frehtes Taifersar Arahesis 2009-05-13 03:30:56 PDT
                     Re: svn commit: r37709 - trunk/subversion/libsvn_subr brane Branko Cibej 2009-05-13 03:50:54 PDT
                         Re: svn commit: r37709 - trunk/subversion/libsvn_subr gstein Greg Stein 2009-05-13 04:37:58 PDT
                             Re: svn commit: r37709 - trunk/subversion/libsvn_subr brane Branko Cibej 2009-05-13 05:24:56 PDT
Messages per page: