After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 734017 - python/tests/sync.py assumes Python dictionaries are ordered
python/tests/sync.py assumes Python dictionaries are ordered
Status: RESOLVED FIXED
Product: libxml2
Classification: Platform
Component: general
git master
Other Solaris
: Normal normal
: ---
Assigned To: Daniel Veillard
libxml QA maintainers
Depends on:
Blocks:
 
 
Reported: 2014-07-30 23:10 UTC by John Beck
Modified: 2014-10-06 10:28 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
patch to fix pythons/tests/sync.py (2.88 KB, patch)
2014-07-30 23:10 UTC, John Beck
none Details | Review

Description John Beck 2014-07-30 23:10:38 UTC
Created attachment 282118 [details] [review]
patch to fix pythons/tests/sync.py

Solaris has had libxml2 version 2.9.1 for a while, with Python versions 2.6 and 2.7.  While preparing to also build a module for Python 3.4, we ran into an issue with the test case sync.py failing.  The failure involved parsing a string that included a Python dictionary, then complaining when the order of the parsed result did not match the original order.  But Python dictionaries are unordered by definition; see section 5.5 of https://docs.python.org/2/tutorial/datastructures.html .  For whatever reason, Python 2.6 and 2.7 always happened to report the pair of values back in their original order, but with Python 3.4 the order is random.  The attached patch allows for either order; it also fixes a typo that was repeated several times thanks to the magic of copy & paste.
Comment 1 Daniel Veillard 2014-10-06 10:28:52 UTC
ACK, funny how this kind of implementation dependent bugs can stick in for
such a long time :)
Thanks for reporting this, the patch and the typo fixes, now commited in
git head !

Daniel