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

[Sivaiah N <snallagatla novell com>]

Commited ,

Thanks,
Siva
On Mon, 2004-08-16 at 11:41 +0200, Rodrigo Moya wrote:
> 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