Merge lp:~matt-goodall/canonical-identity-provider/no-more-bson-madness-other-than-bson-itself into lp:canonical-identity-provider/release

Proposed by Matt Goodall
Status: Merged
Approved by: Matt Goodall
Approved revision: no longer in the source branch.
Merged at revision: 1367
Proposed branch: lp:~matt-goodall/canonical-identity-provider/no-more-bson-madness-other-than-bson-itself
Merge into: lp:canonical-identity-provider/release
Diff against target: 36 lines (+2/-4)
2 files modified
requirements.txt (+1/-3)
src/identityprovider/tests/test_wsgi.py (+1/-1)
To merge this branch: bzr merge lp:~matt-goodall/canonical-identity-provider/no-more-bson-madness-other-than-bson-itself
Reviewer Review Type Date Requested Status
Matias Bordese (community) Approve
Review via email: mp+278162@code.launchpad.net

Commit message

Upgrade oops-amqp to simplify bson dependency.

Description of the change

Fix bson madness:

- upgrade oops-amqp to version with no pymongo dependency
- remove pymongo dependency
- remove pip bson as it's already a deb dependency

To post a comment you must log in.
Revision history for this message
Matias Bordese (matiasb) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'requirements.txt'
--- requirements.txt 2015-11-18 16:24:16 +0000
+++ requirements.txt 2015-11-20 15:37:10 +0000
@@ -1,4 +1,3 @@
1bson==0.3.3
2canonical-raven==0.0.31canonical-raven==0.0.3
3convoy==0.4.12convoy==0.4.1
4dj-static==0.0.6.dev13dj-static==0.0.6.dev1
@@ -17,13 +16,12 @@
17oath==1.4.016oath==1.4.0
18oauthlib==0.7.217oauthlib==0.7.2
19oops==0.0.1318oops==0.0.13
20oops-amqp==0.0.719oops-amqp==0.0.8b1
21oops-datedir-repo==0.0.2220oops-datedir-repo==0.0.22
22oops-dictconfig==0.0.621oops-dictconfig==0.0.6
23oops-timeline==0.0.222oops-timeline==0.0.2
24oops-wsgi==0.0.1123oops-wsgi==0.0.11
25paste==2.0.124paste==2.0.1
26pymongo==3.0.1
27raven==5.6.025raven==5.6.0
28requests-oauthlib==0.4.226requests-oauthlib==0.4.2
29static3==0.5.1.dev127static3==0.5.1.dev1
3028
=== modified file 'src/identityprovider/tests/test_wsgi.py'
--- src/identityprovider/tests/test_wsgi.py 2014-12-01 11:10:45 +0000
+++ src/identityprovider/tests/test_wsgi.py 2015-11-20 15:37:10 +0000
@@ -52,7 +52,7 @@
52 oops_path = self.oops_dir + '/' + todays_oops_path + '/' + oops_list[0]52 oops_path = self.oops_dir + '/' + todays_oops_path + '/' + oops_list[0]
53 with open(oops_path, "r") as oops_file:53 with open(oops_path, "r") as oops_file:
54 oops_file_contents = oops_file.read()54 oops_file_contents = oops_file.read()
55 return bson.decode_all(oops_file_contents)[0]55 return bson.loads(oops_file_contents)
5656
57 def request_wsgi_response(self, environ):57 def request_wsgi_response(self, environ):
58 app = make_app()58 app = make_app()