[Bug 620826] Review Request: python-icalendar - Parser/generator of iCalendar files following the RFC 2445

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

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=620826

--- Comment #4 from Stanislav Ochotnicky <sochotni@xxxxxxxxxx> 2010-08-05 02:57:24 EDT ---
> Huh? How is it failing? Maybe just a missing BR?
> It works here with rpmbuild:
> + /usr/bin/python test.py
> test.py:80: DeprecationWarning: the sets module is deprecated
>   from sets import Set
> Imported 1 modules in 0.278s
> 
> ----------------------------------------------------------------------
> Ran 36 tests in 0.124s
> 
> OK
> 
> -> scratch build F15:
> http://koji.fedoraproject.org/koji/taskinfo?taskID=2379894
> 
> Failed example:
>     g
> Expected:
>     (37.386012999999998, -122.082932)
> Got:
>     (37.386013, -122.082932)
> 
> There it's failing because python 2.7 seems to round a bit different, than
> python 2.6 did (and the tests were written for python 2.6).
> 
> Maybe the next version will fix the doctests, because the author will switch to
> python 2.7 too... (But here with python-2.6.4-27.fc13.x86_64 it's working for
> me.)    

You are correct, it is failing because of Python 2.7. Few hours after I sent
package for review this message got to the icalendar-dev mailing list:

http://codespeak.net/pipermail/icalendar-dev/2010-August/000156.html

So it is a python 2.7 rounding issue and I will apply that patch, but first I
am going to wait for official review and do it all in one take

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]