Re: [evolution-patches] patch for #70035 (calendar)

[JP Rosevear <jpr novell com>]

On Fri, 2005-04-01 at 11:09 +0200, Rodrigo Moya wrote:
> On Fri, 2005-04-01 at 10:45 +0200, Rodrigo Moya wrote:
> > On Fri, 2005-04-01 at 00:29 -0500, JP Rosevear wrote:
> > > 
> > > > Attached patch makes the importer do the same. About other parts, what
> > > > else should be using it?
> > > 
> > > This is my worry doing it this way - we won't find all the bits :-).
> > > 
> > well, the bug was just about the HTTP backend, I haven't seen any other
> > similar bug for other parts, so if they ever appear, they can be easily
> > fixed by getting the new function used.
> 
> also, changing it in icalparser_parse_string, which is used everywhere,
> might add some bugs that might affect other parts, so hence another
> reason to not change it there, at least for 2.2.x

Risky either way I suppose.  I see a couple of solutions:

1) save this work for 2.2.3 and move it lower in the stack
2) Use your current patch for 2.2.2 and fix it lower in 2.3

I think DnD/CCP might need to use the function as well.

-JP
-- 
JP Rosevear <jpr novell com>
Novell, Inc.