Merge ~cjwatson/launchpad:six-cpickle into launchpad:master

Proposed by Colin Watson
Status: Merged
Approved by: Colin Watson
Approved revision: 085d0eeea1f0ae7b4bcf88311febd10294d7ddff
Merge reported by: Otto Co-Pilot
Merged at revision: not available
Proposed branch: ~cjwatson/launchpad:six-cpickle
Merge into: launchpad:master
Diff against target: 95 lines (+8/-7)
4 files modified
lib/lp/bugs/scripts/bugimport.py (+4/-4)
lib/lp/services/testing/profiled.py (+2/-1)
lib/lp/services/utils.py (+1/-1)
lib/lp/services/webapp/pgsession.py (+1/-1)
Reviewer Review Type Date Requested Status
Thiago F. Pappacena (community) Approve
Review via email: mp+379469@code.launchpad.net

Commit message

Import cPickle from six.moves

Description of the change

This part of the standard library was rearranged in Python 3.

To post a comment you must log in.
Revision history for this message
Thiago F. Pappacena (pappacena) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/lib/lp/bugs/scripts/bugimport.py b/lib/lp/bugs/scripts/bugimport.py
2index ed3d8c5..1d872de 100644
3--- a/lib/lp/bugs/scripts/bugimport.py
4+++ b/lib/lp/bugs/scripts/bugimport.py
5@@ -14,7 +14,6 @@ __all__ = [
6 'BugImporter',
7 ]
8
9-import cPickle
10 from cStringIO import StringIO
11 import datetime
12 import logging
13@@ -23,6 +22,7 @@ import time
14
15 from defusedxml import cElementTree
16 import pytz
17+from six.moves import cPickle as pickle
18 from storm.store import Store
19 from zope.component import getUtility
20 from zope.contenttype import guess_content_type
21@@ -215,15 +215,15 @@ class BugImporter:
22 self.bug_id_map = {}
23 self.pending_duplicates = {}
24 else:
25- self.bug_id_map, self.pending_duplicates = cPickle.load(
26+ self.bug_id_map, self.pending_duplicates = pickle.load(
27 open(self.cache_filename, 'rb'))
28
29 def saveCache(self):
30 """Save the bug ID mapping and pending duplicates list to cache."""
31 tmpfilename = '%s.tmp' % self.cache_filename
32 fp = open(tmpfilename, 'wb')
33- cPickle.dump((self.bug_id_map, self.pending_duplicates),
34- fp, protocol=2)
35+ pickle.dump((self.bug_id_map, self.pending_duplicates),
36+ fp, protocol=2)
37 fp.close()
38 os.rename(tmpfilename, self.cache_filename)
39
40diff --git a/lib/lp/services/testing/profiled.py b/lib/lp/services/testing/profiled.py
41index f0560e4..620d2fa 100644
42--- a/lib/lp/services/testing/profiled.py
43+++ b/lib/lp/services/testing/profiled.py
44@@ -7,11 +7,12 @@ __metaclass__ = type
45 __all__ = ['profiled', 'setup_profiling']
46
47 import atexit
48-import cPickle as pickle
49 import os
50 import tempfile
51 import time
52
53+from six.moves import cPickle as pickle
54+
55
56 _profile_stats_filename = os.environ.get('lp_layer_profile_filename', None)
57 _profiling_setup_time = None
58diff --git a/lib/lp/services/utils.py b/lib/lp/services/utils.py
59index 04e24aa..beda118 100644
60--- a/lib/lp/services/utils.py
61+++ b/lib/lp/services/utils.py
62@@ -33,7 +33,6 @@ __all__ = [
63 ]
64
65 import bz2
66-import cPickle as pickle
67 from datetime import datetime
68 from itertools import (
69 islice,
70@@ -53,6 +52,7 @@ from fixtures import (
71 )
72 from lazr.enum import BaseItem
73 import pytz
74+from six.moves import cPickle as pickle
75 from twisted.python.util import mergeFunctionMetadata
76 from zope.security.proxy import isinstance as zope_isinstance
77
78diff --git a/lib/lp/services/webapp/pgsession.py b/lib/lp/services/webapp/pgsession.py
79index edbb0b0..a11fedb 100644
80--- a/lib/lp/services/webapp/pgsession.py
81+++ b/lib/lp/services/webapp/pgsession.py
82@@ -5,12 +5,12 @@
83
84 __metaclass__ = type
85
86-import cPickle as pickle
87 import hashlib
88 import time
89 from UserDict import DictMixin
90
91 from lazr.restful.utils import get_current_browser_request
92+from six.moves import cPickle as pickle
93 from storm.zope.interfaces import IZStorm
94 from zope.authentication.interfaces import IUnauthenticatedPrincipal
95 from zope.component import getUtility

Subscribers

People subscribed via source and target branches

to status/vote changes: