Merge lp:~darkxst/ubuntu/saucy/evolution-data-server/lp1184234 into lp:~ubuntu-desktop/evolution-data-server/ubuntu

Proposed by Tim Lunn
Status: Merged
Merge reported by: Jeremy Bícha
Merged at revision: not available
Proposed branch: lp:~darkxst/ubuntu/saucy/evolution-data-server/lp1184234
Merge into: lp:~ubuntu-desktop/evolution-data-server/ubuntu
Diff against target: 37 lines (+19/-0)
3 files modified
debian/changelog (+8/-0)
debian/patches/series (+1/-0)
debian/patches/uoa_eds_desktop_entry.patch (+10/-0)
To merge this branch: bzr merge lp:~darkxst/ubuntu/saucy/evolution-data-server/lp1184234
Reviewer Review Type Date Requested Status
Iain Lane Approve
Review via email: mp+168340@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Iain Lane (laney) wrote :

Thanks - don't suppose you could forward it upstream too could you?

Revision history for this message
Iain Lane (laney) :
review: Approve
Revision history for this message
Iain Lane (laney) wrote :

Although, hmm, what happens when evolution isn't installed? Perhaps it'd be better for eds to have its own maybe?

Revision history for this message
Tim Lunn (darkxst) wrote :

Yes will forward it upstream.

If evolution is not installed, then it won't find the desktop file and will end up with the previous behaviour of emitting a warning and not display in UOA.

Revision history for this message
Tim Lunn (darkxst) wrote :

actually maybe e-d-s should install its own desktop file, since the UOA integration should work across the gnome stack, not just evolution?

Revision history for this message
Iain Lane (laney) wrote :

On Mon, Jun 10, 2013 at 10:33:26PM -0000, Tim wrote:
> actually maybe e-d-s should install its own desktop file, since the UOA integration should work across the gnome stack, not just evolution?

I think so. I'm not sure anything else actually does use it currently,
but it could in the future.

--
Iain Lane [ <email address hidden> ]
Debian Developer [ <email address hidden> ]
Ubuntu Developer [ <email address hidden> ]

Revision history for this message
Tim Lunn (darkxst) wrote :

gnome-contacts seems to work, documents has not been implemented.

Revision history for this message
Iain Lane (laney) wrote :

On Tue, Jun 11, 2013 at 08:27:28AM -0000, Tim wrote:
> gnome-contacts seems to work, documents has not been implemented.

So yes, doing this in eds seems to make sense to me. If you propose this
directly upstream (ask mardy to review, maybe?) then we can easily
cherry-pick or you can ask for it to be pushed into the 3.8 branch and
we'll get it normally.

Thanks for your work!

--
Iain Lane [ <email address hidden> ]
Debian Developer [ <email address hidden> ]
Ubuntu Developer [ <email address hidden> ]

Revision history for this message
Jeremy Bícha (jbicha) wrote :

We merged Laney's latest version of this patch.

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-06-07 15:59:38 +0000
3+++ debian/changelog 2013-06-10 02:17:30 +0000
4@@ -1,3 +1,11 @@
5+evolution-data-server (3.8.2-1ubuntu3) UNRELEASED; urgency=low
6+
7+ * debian/patches/uoa_eds_desktop_entry.patch
8+ - UOA: Add desktop_entry to e-d-s application provider so that e-d-s
9+ integration can be toggled on/off from UOA. (LP: #1184234)
10+
11+ -- Tim Lunn <tim@feathertop.org> Mon, 10 Jun 2013 12:10:22 +1000
12+
13 evolution-data-server (3.8.2-1ubuntu2) saucy; urgency=low
14
15 * debian/patches/uoa_google_responsetype.patch
16
17=== modified file 'debian/patches/series'
18--- debian/patches/series 2013-06-07 15:59:38 +0000
19+++ debian/patches/series 2013-06-10 02:17:30 +0000
20@@ -1,1 +1,2 @@
21 uoa_google_responsetype.patch
22+uoa_eds_desktop_entry.patch
23
24=== added file 'debian/patches/uoa_eds_desktop_entry.patch'
25--- debian/patches/uoa_eds_desktop_entry.patch 1970-01-01 00:00:00 +0000
26+++ debian/patches/uoa_eds_desktop_entry.patch 2013-06-10 02:17:30 +0000
27@@ -0,0 +1,10 @@
28+--- a/modules/ubuntu-online-accounts/evolution-data-server.application.in.in
29++++ b/modules/ubuntu-online-accounts/evolution-data-server.application.in.in
30+@@ -1,6 +1,7 @@
31+ <?xml version="1.0" encoding="UTF-8"?>
32+ <application id="evolution-data-server">
33+ <description>Evolution-Data-Server</description>
34++ <desktop-entry>evolution.desktop</desktop-entry>
35+ <translations>@GETTEXT_PACKAGE@</translations>
36+ <service-types>
37+ <service-type id="mail"/>

Subscribers

People subscribed via source and target branches

to all changes: