Merge lp:~manishsinha/gedit/enable-zeitgeist-datasource-plugin into lp:~ubuntu-desktop/gedit/ubuntu

Status: Merged
Merged at revision: 85
Proposed branch: lp:~manishsinha/gedit/enable-zeitgeist-datasource-plugin
Merge into: lp:~ubuntu-desktop/gedit/ubuntu
Diff against target: 62 lines (+15/-4)
3 files modified
debian/changelog (+7/-0)
debian/control (+4/-2)
debian/control.in (+4/-2)
To merge this branch: bzr merge lp:~manishsinha/gedit/enable-zeitgeist-datasource-plugin
Reviewer Review Type Date Requested Status
Jeremy Bícha Approve
Review via email: mp+80561@code.launchpad.net

Commit message

Append libzeitgeist-dev as build-depends and libzeitgeist-1.0-1 as gedit dependency to enable zeitgeist-dataprovider plugin

Description of the change

Append libzeitgeist-dev as build-depends and libzeitgeist-1.0-1 as gedit dependency to enable zeitgeist-dataprovider plugin

To post a comment you must log in.
Revision history for this message
Jeremy Bícha (jbicha) wrote :

Thank you! I've merged this into our packaging branch but I think we should figure out whether to enable this by default before we push a package update.

review: Approve
Revision history for this message
Didier Roche-Tolomelli (didrocks) wrote :

Some additional comment on the merge req (same for the totem one that has been merged as well):
+ libzeitgeist-1.0-1
-> this isn't needed and in addition, should be avoided at soon as possible. You are forcing a dependency on a versionned libzeitgeist soname, which means that if we have a new libzeitgeist, we need to change the gedit packaging (and all other packages depending on libzeitgeist, which will be a lot). Basically we just rebuild them.
libzeitgeist-1.0-1 (>= 0.3.2) is automatically added by dh_shlibdeps which expands ${shlibs:Depends}, and replace with library dependencies.

I fixed it directly in gedit upload and going to fix totem as well. If you want more info on that, please use man dh_shlibdeps ;)

I activated the plugin by default as well.
Thanks for your work there!

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 2011-10-17 15:18:05 +0000
3+++ debian/changelog 2011-10-27 13:00:29 +0000
4@@ -1,3 +1,10 @@
5+gedit (3.2.1-0ubuntu2) oneiric; urgency=low
6+
7+ * Append libzeitgeist-dev as build-depends and libzeitgeist-1.0-1 as
8+ gedit dependency to enable zeitgeist-dataprovider plugin
9+
10+ -- Manish Sinha <manishsinha@ubuntu.com> Thu, 27 Oct 2011 18:23:34 +0530
11+
12 gedit (3.2.1-0ubuntu1) oneiric-proposed; urgency=low
13
14 * New upstream version (LP: #876592)
15
16=== modified file 'debian/control'
17--- debian/control 2011-10-17 14:45:56 +0000
18+++ debian/control 2011-10-27 13:00:29 +0000
19@@ -32,7 +32,8 @@
20 gobject-introspection,
21 libgirepository1.0-dev (>= 0.9.3),
22 scrollkeeper,
23- liblaunchpad-integration-3.0-dev
24+ liblaunchpad-integration-3.0-dev,
25+ libzeitgeist-dev
26 Build-Depends-Indep: libglib2.0-doc,
27 libgtk-3-doc,
28 libgtksourceview-3.0-doc
29@@ -52,7 +53,8 @@
30 gir1.2-glib-2.0,
31 gir1.2-peas-1.0,
32 gir1.2-gtk-3.0,
33- iso-codes
34+ iso-codes,
35+ libzeitgeist-1.0-1
36 Conflicts: gir1.2-gedit-3.0
37 Breaks: seahorse (<< 2.24),
38 seahorse-plugins (<< 2.26),
39
40=== modified file 'debian/control.in'
41--- debian/control.in 2011-10-17 14:45:56 +0000
42+++ debian/control.in 2011-10-27 13:00:29 +0000
43@@ -27,7 +27,8 @@
44 gobject-introspection,
45 libgirepository1.0-dev (>= 0.9.3),
46 scrollkeeper,
47- liblaunchpad-integration-3.0-dev
48+ liblaunchpad-integration-3.0-dev,
49+ libzeitgeist-dev
50 Build-Depends-Indep: libglib2.0-doc,
51 libgtk-3-doc,
52 libgtksourceview-3.0-doc
53@@ -47,7 +48,8 @@
54 gir1.2-glib-2.0,
55 gir1.2-peas-1.0,
56 gir1.2-gtk-3.0,
57- iso-codes
58+ iso-codes,
59+ libzeitgeist-1.0-1
60 Conflicts: gir1.2-gedit-3.0
61 Breaks: seahorse (<< 2.24),
62 seahorse-plugins (<< 2.26),

Subscribers

People subscribed via source and target branches