Re: [Evolution-hackers] SyncEvolution + EClient API + EXDATE regression (Bug #655253)

[Milan Crha <mcrha redhat com>]

On Tue, 2011-09-13 at 18:44 +0200, Patrick Ohly wrote:
> On Di, 2011-09-13 at 17:59 +0200, Patrick Ohly wrote:
> > The root cause of the bug is that the detached recurrences, if created
> > with current Evolution master, do not get the same UID as the original
> > recurring event. I've verified that by looking at the resulting .ics
> > file.
> > 
> > It works correctly in 2.32.4.
> > 
> > Does anyone know where in the 3.x cycle this broke?
> 
> Darn, I can no longer reproduce the problem. I was able to reproduce
> with the source from current master (thus my statement above about the
> UIDs not matching), but after removing the EXDATE code it all works
> fine: same UID, no duplicates shown even after restart. Even after
> restoring the original source it still works (same UID, EXDATE added).
> 
> Milan, do you agree that the e_cal_util_remove_instances() call can and
> should be removed? Either way, it only works around whatever is causing
> the UID issue. Adding the EXDATE just pampers over that problem and
> still doesn't fix things like "remove all recurrences".

	Hi,
yes, I agree, it wasn't the right change, I'm sorry for that. Please
revert the commit and reopen the bug. Thanks for the pointer on this.
	Bye,
	Milan