output is not reproducible about gpxpy

[tkrajina]

Yes, I'm aware of this problem.

I just pushed a quickfix for this here: 75381db

It will sort the attributes each time if there is more than one attribute.

Now, this has a small performance penalty and it works only with python2. I'll try to think for a better solution. Of course, ideas are welcome.

[wrohdewald]

how about using OrderedDict? It exists since Python 2.7

--- a/gpxpy/gpx.py
+++ b/gpxpy/gpx.py
@@ -2582,11 +2582,11 @@ class GPX:
self.creator = 'gpx.py -- https://github.com/tkrajina/gpxpy'

     v = version.replace('.', '/')

•    xml_attributes = {
  

•            'xmlns:xsi': 'http://www.w3.org/2001/XMLSchema-instance',
  

•            'xmlns': 'http://www.topografix.com/GPX/%s' % v,
  

•            'xsi:schemaLocation': 'http://www.topografix.com/GPX/%s http://www.topografix.com/GPX/%s/gpx.xsd' % (v, v)
  

•    }
  

•    xml_attributes = mod_collections.OrderedDict([
  

•            ('xmlns:xsi', 'http://www.w3.org/2001/XMLSchema-instance'),
  

•            ('xmlns', 'http://www.topografix.com/GPX/%s' % v),
  

•            ('xsi:schemaLocation', 'http://www.topografix.com/GPX/%s http://www.topografix.com/GPX/%s/gpx.xsd' % (v, v))
  

•    ])
  
     content = mod_gpxfield.gpx_fields_to_xml(self, 'gpx', version, custom_attributes=xml_attributes)
  

[tkrajina]

Yes, it makes sense. Although, now that I look at it, the custom_attributes named parameter is used only once, so even the dict.sort() wouldn't be such a problem.  But, on the other side, there is no sort() for {}.items() in py3. And if OrderedDict is the only reason to break py2.6, I'll just not sort but expect the attrs to be a list:

8abebca

...and it will work everywhere.

Thanks for your help!