Re: [evolution-patches] patch for #62870 (alarm-dameon)



On Mon, 2004-08-16 at 14:26 +0530, Sivaiah N wrote:
> Hi,
> alarm-dameon is loading groupwise calendar by giving an Esource which
> does not have all the properites reuired for laoding groupwise calendr
> (propertis like "use_ssl", "soap_port" etc). This is because it uses the
> fuinction auth_new_cal_from_uri. The backend uses same info even while
> trying to login from evo calendar view also. It is talking long time to
> time out as we are trying to connect to a port where no body is
> listeneing  Attached patch relaces all auth_new_cal_from_uri calls in
> alarm-dameon code auth_new_cal_from_source.
> 
> This patch also should make alarm-notify work for groupwise calendar
> which was not the case earlier. 
> 

> +       /*** FIXME we should run an idle loop to check for availbility
> of passwords for these sources 
> +           and  add them to the list once they are available */
> 
right, we should really do that. In fact, we used to do something
similar in 1.4, for the exchange connector mainly, which was to keep
retrying the connection when it couldn't connect.

the rest of the patch looks ok
-- 
Rodrigo Moya <rodrigo novell com>




[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]