[Pulp-list] f17 and simplejson

[Nick Coghlan]

On 06/09/2012 07:21 AM, Jeff Ortel wrote:
> Fedora 17 GOLD includes python-simplejson-2.3.0-2.fc17.  In simplejson
> 2.2, support for namedtuples was added.  In doing so, a bug was
> introduced that causes JSON serialization errors in gofer <= 0.68.  A
> description and trace can be found in bugzilla 829767.
> 
> This issue has been addressed in gofer 0.69.  The basic change is
> re-parenting (messaging) Envelope & Options classes to 'object' rather
> than 'dict'.  Then, translating the envelope (as needed) to 'dict'
> during serialization.  This is a better approach anyway since overriding
> __getattr__ & __setattr__ on a subclass of dict opens the door for name
> collisions with native dict attributes.
> 
> This issue has *also* been addressed in simplejson 2.5.2 which is
> available in F17 (updates).  I intend to update (pulp) rpm/gofer to 0.69
> on master to be included in the today's QE build (dev/testing). However,
> since it appears that doing a 'yum update python-simplejson' is a viable
> workaround, I don't plan to update rpm/gofer on pulp_v1 (v1/testing) at
> this time.  I'd like to get some miles on gofer 0.69 running with v2 first.
> 
> So, on F17, if you're seeing this issue, do
> 
> # yum update python-simplejson
> 
> to be sure you have the latest (python-simplejson-2.5.2-1.fc17) and you
> should be good-to-go.  Remember to bounce goferd and httpd.  If you
> continue to see any issues, please let me know.

Have you considered just dropping the simplejson dependency on platforms
that provide Python 2.6+? Or are there specific simplejson features that
Pulp uses that weren't included when simplejson was added to the stdlib
as the json module in 2.6?

Cheers,
Nick.

-- 
Nick Coghlan
Red Hat Infrastructure Engineering & Development, Brisbane