Re: [evolution-patches] e-url fix




There's no reason for mail/ to use e-url at all, we just use camelurl whenever we need a url parsed.

Note that em-filter-source-element.c does actually use camelurl anyway, the e-url include should be removed.  It doesn't appear that mail-config.c uses the e-url it includes either.  Probably cruft left over from old code when someone unfamiliar with the codebase wrote something.

Michael

On Mon, 2005-01-17 at 14:44 +0100, Rodrigo Moya wrote:
On Mon, 2005-01-17 at 18:35 +0530, Sarfraaz Ahmed wrote:
> Rodrigo Moya wrote:
> 
> >On Mon, 2005-01-17 at 14:18 +0530, Sarfraaz Ahmed wrote:
> >  
> >
> >>Hi,
> >>
> >>The uri parsing in libedataserver is different from that in 
> >>evolution/e-util. The diff is attached in the patch file. This fixes a 
> >>crash in connector startup.
> >>
> >>    
> >>
> >hmm, why do we still have e-url.[ch] in evolution tree? I think those
> >files should be removed, right?
> >  
> >
> True, but i am not sure about why they are present yet. I dont know what 
> the plan is about removing those files. Till these are removed, this 
> patch would fix the connector crash. I might currently not be able to 
> give the patch for removing this file though.
> 
right, due to some dependency, connector is using evo's e-url. Attached
patch removes e-url from the build and changes mail/ to use
libedataserver's instead. Calendar is already changed.



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