
<html><body>


<p> </p>

<blockquote>

<div class="message_header"><span>Fr Sep 04 2009 07:41:46 EDT</span> <span>von  "Patrick Ohly" <patrick.ohly@gmx.de> </span> <span class="message_subject">Betreff: Re: [Freeassociation-devel] libical fix recurrence not excluded when it should be</span></div>

<div class="message_content"><tt></tt><tt>I really don't know. What surprises me about Gren's original report</tt><br /> <tt>("icalproperty_recurrence_is_excluded" does not take into account any</tt><br /> <tt>timezone associated with EXDATEs being tested") is that in Evolution,</tt><br /> <tt>EXDATEs with TZID are the rule, not the exception. That has worked,</tt><br /> <tt>whereas I know from real-life problems that EXDATES with UTC were not</tt><br /> <tt>handled.</tt><br /> <tt></tt><br /> <tt>Anyway, the test shows that the current code works. It didn't cover the</tt><br /> <tt>EXDATE in UTC case; I have added that and it still passes (patch</tt><br /> <tt>attached). So this looks good to go into a release.</tt><br /> <tt></tt><br /> <tt>Note that I ran into other issues trying to run "make check". Patches</tt><br /> <tt>also attached.</tt><br /> <tt></tt><br /> <tt>-- </tt><br /> <tt>Bye, Patrick Ohly</tt><br /> <tt>-- </tt><br /> <tt>Patrick.Ohly@gmx.de</tt><br /> <tt><a href="http://www.estamos.de/" target="webcit01">http://www.estamos.de/</a></tt><br /> <tt></tt><br /> <br /></div>

</blockquote>

<p>Thanks a lot! I guess your patches plus removing the now unnedef function will make the next release.</p>

<p>Any other opinions on that? Art? Allen?</p>

<p> </p>

<p> </p>

<p>Wilfried Goesgens</p>

</body></html>