Merge lp:~laney/friends/eds-3.8 into lp:friends

Proposed by Iain Lane
Status: Merged
Approved by: Ken VanDine
Approved revision: 205
Merged at revision: 204
Proposed branch: lp:~laney/friends/eds-3.8
Merge into: lp:friends
Diff against target: 122 lines (+30/-14)
4 files modified
debian/changelog (+14/-0)
debian/control (+7/-5)
friends/utils/base.py (+8/-8)
service/src/Makefile.am (+1/-1)
To merge this branch: bzr merge lp:~laney/friends/eds-3.8
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Ken VanDine Approve
Review via email: mp+166326@code.launchpad.net

Commit message

Port to EDS 3.8 - the contacts functionality was moved into an EBookContacts library. Add Build-Deps and Deps on gir1.2-ebookcontacts-1.2 accordingly. Version the rest.

Description of the change

Port to EDS 3.8. This isn't in the archive yet so the auto testing will fail; we can re-run it once it is in. In the meantime, I'd appreciate it if you could grab the 3.8 packages from

  https://launchpad.net/~laney/+archive/evolution-3.8

and review this for pre-approval. I'll upload this into there too so we can see if it builds (for some reason the testsuite fails badly in sbuild here).

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:204
http://jenkins.qa.ubuntu.com/job/friends-ci/44/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/friends-saucy-amd64-ci/1/console

Click here to trigger a rebuild:
http://s-jenkins:8080/job/friends-ci/44/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Ken VanDine (ken-vandine) wrote :

This looks good, but I am not going to top approve it just yet. The build of gir1.2-dee-1.0 in the daily-build PPA appears to be broken, so the tests will fail. I have confirmed the tests all pass on saucy.

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Autolanding.
More details in the following jenkins job:
http://jenkins.qa.ubuntu.com/job/friends-autolanding/25/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/friends-saucy-amd64-autolanding/1/console

review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Autolanding.
More details in the following jenkins job:
http://jenkins.qa.ubuntu.com/job/friends-autolanding/26/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/friends-saucy-amd64-autolanding/2/console

review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:205
http://jenkins.qa.ubuntu.com/job/friends-ci/45/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/friends-saucy-amd64-ci/2/console

Click here to trigger a rebuild:
http://s-jenkins:8080/job/friends-ci/45/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Autolanding.
More details in the following jenkins job:
http://jenkins.qa.ubuntu.com/job/friends-autolanding/27/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/friends-saucy-amd64-autolanding/3/console

review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2013-05-07 17:53:19 +0000
3+++ debian/changelog 2013-06-06 18:27:26 +0000
4@@ -1,3 +1,17 @@
5+friends (0.2.0daily13.05.07.1-0ubuntu3) UNRELEASED; urgency=low
6+
7+ * Updated for libaccounts-glib >= 1.10
8+
9+ -- Ken VanDine <ken.vandine@canonical.com> Thu, 06 Jun 2013 14:02:48 -0400
10+
11+friends (0.2.0daily13.05.07.1-0ubuntu2) saucy; urgency=low
12+
13+ * Port to EDS 3.8 - the contacts functionality was moved into an
14+ EBookContacts library. Add Build-Deps and Deps on gir1.2-ebookcontacts-1.2
15+ accordingly. Version the rest.
16+
17+ -- Iain Lane <iain.lane@canonical.com> Wed, 29 May 2013 18:31:49 +0100
18+
19 friends (0.2.0daily13.05.07.1-0ubuntu1) saucy; urgency=low
20
21 [ Robert Bruce Park ]
22
23=== modified file 'debian/control'
24--- debian/control 2013-04-16 15:40:42 +0000
25+++ debian/control 2013-06-06 18:27:26 +0000
26@@ -7,8 +7,9 @@
27 evolution-data-server,
28 gir1.2-dbusmenu-glib-0.4,
29 gir1.2-dee-1.0,
30- gir1.2-ebook-1.2,
31- gir1.2-edataserver-1.2,
32+ gir1.2-ebook-1.2 (>= 3.8),
33+ gir1.2-ebookcontacts-1.2 (>= 3.8),
34+ gir1.2-edataserver-1.2 (>= 3.8),
35 gir1.2-gdkpixbuf-2.0,
36 gir1.2-glib-2.0,
37 gir1.2-messagingmenu-1.0,
38@@ -17,7 +18,7 @@
39 gir1.2-signon-1.0,
40 gir1.2-soup-2.4,
41 gir1.2-unity-5.0,
42- libaccounts-glib-dev,
43+ libaccounts-glib-dev (>= 1.10),
44 libsignon-glib1,
45 python3,
46 python3-dbus,
47@@ -51,8 +52,9 @@
48 Breaks: friends (<< 0.1.1bzr13.02.12)
49 Conflicts: friends (<< 0.1.1bzr13.02.12)
50 Depends: gir1.2-dee-1.0,
51- gir1.2-ebook-1.2,
52- gir1.2-edataserver-1.2,
53+ gir1.2-ebook-1.2 (>= 3.8),
54+ gir1.2-ebookcontacts-1.2 (>= 3.8),
55+ gir1.2-edataserver-1.2 (>= 3.8),
56 gir1.2-gdkpixbuf-2.0,
57 gir1.2-glib-2.0,
58 gir1.2-networkmanager-1.0,
59
60=== modified file 'friends/utils/base.py'
61--- friends/utils/base.py 2013-04-17 06:13:49 +0000
62+++ friends/utils/base.py 2013-06-06 18:27:26 +0000
63@@ -31,7 +31,7 @@
64 from datetime import datetime, timedelta
65 from oauthlib.oauth1 import Client
66
67-from gi.repository import GLib, GObject, EDataServer, EBook
68+from gi.repository import GLib, GObject, EDataServer, EBook, EBookContacts
69
70 from friends.errors import FriendsError, ContactsError
71 from friends.utils.authentication import Authentication
72@@ -586,8 +586,8 @@
73
74 def _previously_stored_contact(self, source, field, search_term):
75 client = self._new_book_client(source)
76- query = EBook.book_query_vcard_field_test(
77- field, EBook.BookQueryTest(0), search_term)
78+ query = EBookContacts.BookQuery.vcard_field_test(
79+ field, EBookContacts.BookQueryTest.IS, search_term)
80 success, result = client.get_contacts_sync(query.to_string(), None)
81 if not success:
82 raise ContactsError('Search failed on field {}'.format(field))
83@@ -610,22 +610,22 @@
84 def _create_contact(self, user_fullname, user_nickname,
85 social_network_attrs):
86 """Build a VCard based on a dict representation of a contact."""
87- vcard = EBook.VCard.new()
88+ vcard = EBookContacts.VCard.new()
89 info = social_network_attrs
90
91 for i in info:
92- attr = EBook.VCardAttribute.new('social-networking-attributes', i)
93+ attr = EBookContacts.VCardAttribute.new('social-networking-attributes', i)
94 if type(info[i]) == type(dict()):
95 for j in info[i]:
96- param = EBook.VCardAttributeParam.new(j)
97+ param = EBookContacts.VCardAttributeParam.new(j)
98 param.add_value(info[i][j])
99 attr.add_param(param);
100 else:
101 attr.add_value(info[i])
102 vcard.add_attribute(attr)
103
104- contact = EBook.Contact.new_from_vcard(
105- vcard.to_string(EBook.VCardFormat(1)))
106+ contact = EBookContacts.Contact.new_from_vcard(
107+ vcard.to_string(EBookContacts.VCardFormat(1)))
108 contact.set_property('full-name', user_fullname)
109 if user_nickname is not None:
110 contact.set_property('nickname', user_nickname)
111
112=== modified file 'service/src/Makefile.am'
113--- service/src/Makefile.am 2013-03-05 00:40:28 +0000
114+++ service/src/Makefile.am 2013-06-06 18:27:26 +0000
115@@ -5,7 +5,7 @@
116 $(BASE_CFLAGS)
117
118 VALAFLAGS = \
119- --pkg accounts \
120+ --pkg libaccounts-glib \
121 --pkg dee-1.0 \
122 --pkg gio-2.0
123

Subscribers

People subscribed via source and target branches

to all changes: