Replace simplejson with json

Bug #1029094 reported by Barry Warsaw on 2012-07-25
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Ubuntu One Client
Status tracked in Trunk
Stable-4-2
Undecided
Unassigned
Trunk
Undecided
dobey
Ubuntu One Control Panel
Status tracked in Trunk
Stable-4-2
Undecided
Unassigned
Trunk
Undecided
Barry Warsaw
ubuntuone-client (Ubuntu)
Medium
dobey
Raring
Medium
dobey
ubuntuone-control-panel (Ubuntu)
Undecided
Unassigned
Raring
Undecided
Unassigned

Bug Description

In preparation for porting to Python 3, ubuntuone-control-panel should use the standard library `json` module instead of the third party `simplejson` module. The latter is not ported to Python 3, but it is also unnecessary now that there is a stdlib version. This change is appropriate for Python 2.6 and 2.7 as well.

I will attach a branch momentarily.

Related branches

Barry Warsaw (barry) wrote :

I'll also note that this package is the last one that uses simplejson on the Ubuntu desktop images. By making this change it will allow us to drop simplejson as a Python 3 porting dependency.

Barry Warsaw (barry) wrote :

Merge proposed.

dobey (dobey) on 2012-11-08
Changed in ubuntuone-control-panel (Ubuntu Raring):
status: New → Triaged
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubuntuone-control-panel - 4.1.0-0ubuntu1

---------------
ubuntuone-control-panel (4.1.0-0ubuntu1) raring; urgency=low

  * New upstream release.
    - Replace simplejson usage with json. (LP: #1029094)
    - Clear search when clicking (X) icon in search entry. (LP: #1070917)
    - Raise existing window if process already running. (LP: #1063927)
    - Don't auto-publish when selecting a search result. (LP: #1065194)
    - Add a new .desktop file for the GNOME-only remix of Ubuntu.
  * debian/control:
    - Switch pylint build dependency to pyflakes.
    - Remove python-simplejson from dependencies.
  * debian/ubuntuone-control-panel-qt.install:
    - Add new .desktop file for GNOME remix.
    - Remove the messaging menu integration file.
  * debian/watch:
    - Update to use stable-4-2 series for Ubuntu 13.04 releases.
 -- Rodney Dawes <email address hidden> Mon, 10 Dec 2012 09:56:24 -0500

Changed in ubuntuone-control-panel (Ubuntu Raring):
status: Triaged → Fix Released
dobey (dobey) on 2013-01-11
Changed in ubuntuone-client (Ubuntu Raring):
assignee: nobody → Rodney Dawes (dobey)
importance: Undecided → Medium
status: New → Triaged
Julien Funk (jaboing) on 2013-01-25
tags: added: u1-notrack
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubuntuone-client - 4.1.90-0ubuntu1

---------------
ubuntuone-client (4.1.90-0ubuntu1) raring; urgency=low

  * New upstream release.
    - Fix support for searching local files in cloud folders. (LP: #1101271)
    - Various fixes to the test suite.
    - Fix tritcask DataFile.write to flush with new libc6. (LP: #1093111)
    - Remove most of the static glib bindings usage. (LP: #1108036)
    - Avoid g_type_init with new versions of glib. (LP: #1100926)
    - Replace simplejson usage with json. (LP: #1029094)
    - Use a standard LC_CTYPE locale when the LANG is turkish. (LP: #997326)
  * debian/control:
    - Require newer version of ubuntu-sso-client for LANG fix.
    - Update dependencies to drop fallbacks for static bindings.
 -- Rodney Dawes <email address hidden> Tue, 29 Jan 2013 15:22:08 -0500

Changed in ubuntuone-client (Ubuntu Raring):
status: Triaged → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers