Re: [evolution-patches] e-url fix
- From: Not Zed <notzed ximian com>
- To: Rodrigo Moya <rodrigo novell com>
- Cc: Sarfraaz Ahmed <asarfraaz novell com>, asdf <evolution-patches lists ximian com>
- Subject: Re: [evolution-patches] e-url fix
- Date: Tue, 18 Jan 2005 09:07:07 +0800
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]