Merge lp:~manishsinha/totem/enable-zg-dp-building into lp:~ubuntu-desktop/totem/ubuntu

Status: Merged
Merged at revision: 121
Proposed branch: lp:~manishsinha/totem/enable-zg-dp-building
Merge into: lp:~ubuntu-desktop/totem/ubuntu
Diff against target: 82 lines (+17/-3)
5 files modified
debian/changelog (+6/-0)
debian/control (+4/-1)
debian/control.in (+4/-1)
debian/rules (+1/-1)
debian/totem-plugins.install (+2/-0)
To merge this branch: bzr merge lp:~manishsinha/totem/enable-zg-dp-building
Reviewer Review Type Date Requested Status
Jeremy Bícha Approve
Review via email: mp+80492@code.launchpad.net

Commit message

* debian/rules: Enabled vala support
* debian/control: Added valac, libzeitgeist-dev as build-dep and libzeitgeist-1.0-1 as dependency of totem-plugins
* debian/totem-plugins.intall: Added another entry for zeitgeist-dp

Description of the change

Enabled Zeitgeist datasource

Changed made
* Enabled the vala support in debian/rules
* Added an entry in totem-plugins.install
* Updated the respective dependencies in debian/control

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

Please make the same changes to debian/control.in that you do to debian/control.

The plugin appears to work for me. I activated it, played a video, and the video showed up in GNOME Activity Journal. I do get this error on activation but it didn't seem to be too serious.

GLib-GObject-CRITICAL **: Property 'object' on class 'ZeitgeistDpPlugin' has type 'TotemObject' which is different from the type 'GObject', of the property on interface 'PeasActivatable'

122. By Manish Sinha (मनीष सिन्हा)

Made the necessary changes to debian/control.in too

Revision history for this message
Manish Sinha (मनीष सिन्हा) (manishsinha) wrote :

I don't think that is much to be worried about. I keep on getting such messages even when running gaj from command line (though not the same)

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

Thank you!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/changelog'
--- debian/changelog 2011-10-15 21:13:41 +0000
+++ debian/changelog 2011-10-26 20:23:51 +0000
@@ -1,3 +1,9 @@
1totem (3.0.1-0ubuntu9) oneiric; urgency=low
2
3 * Enable zeitgeist-dp vala plugin for totem (LP: #879702)
4
5 -- Manish Sinha <manishsinha@ubuntu.com> Wed, 26 Oct 2011 06:01:03 +0530
6
1totem (3.0.1-0ubuntu8) precise; urgency=low7totem (3.0.1-0ubuntu8) precise; urgency=low
28
3 * debian/patches/04_port_to_gobject.patch:9 * debian/patches/04_port_to_gobject.patch:
410
=== modified file 'debian/control'
--- debian/control 2011-10-11 00:58:24 +0000
+++ debian/control 2011-10-26 20:23:51 +0000
@@ -60,6 +60,8 @@
60 hardening-wrapper,60 hardening-wrapper,
61 dh-autoreconf,61 dh-autoreconf,
62 gnome-common,62 gnome-common,
63 valac,
64 libzeitgeist-dev,
63Standards-Version: 3.8.465Standards-Version: 3.8.4
64Homepage: http://www.gnome.org/projects/totem/66Homepage: http://www.gnome.org/projects/totem/
65Vcs-Bzr: https://code.launchpad.net/~ubuntu-desktop/totem/ubuntu67Vcs-Bzr: https://code.launchpad.net/~ubuntu-desktop/totem/ubuntu
@@ -178,7 +180,8 @@
178 gir1.2-peas-1.0,180 gir1.2-peas-1.0,
179 python-gobject (>= 2.27),181 python-gobject (>= 2.27),
180 python-xdg,182 python-xdg,
181 python-httplib2183 python-httplib2,
184 libzeitgeist-1.0-1
182Recommends: gnome-settings-daemon185Recommends: gnome-settings-daemon
183Suggests: gromit186Suggests: gromit
184Description: Plugins for the Totem media player187Description: Plugins for the Totem media player
185188
=== modified file 'debian/control.in'
--- debian/control.in 2011-10-11 00:58:24 +0000
+++ debian/control.in 2011-10-26 20:23:51 +0000
@@ -55,6 +55,8 @@
55 hardening-wrapper,55 hardening-wrapper,
56 dh-autoreconf,56 dh-autoreconf,
57 gnome-common,57 gnome-common,
58 valac,
59 libzeitgeist-dev,
58Standards-Version: 3.8.460Standards-Version: 3.8.4
59Homepage: http://www.gnome.org/projects/totem/61Homepage: http://www.gnome.org/projects/totem/
60Vcs-Bzr: https://code.launchpad.net/~ubuntu-desktop/totem/ubuntu62Vcs-Bzr: https://code.launchpad.net/~ubuntu-desktop/totem/ubuntu
@@ -173,7 +175,8 @@
173 gir1.2-peas-1.0,175 gir1.2-peas-1.0,
174 python-gobject (>= 2.27),176 python-gobject (>= 2.27),
175 python-xdg,177 python-xdg,
176 python-httplib2178 python-httplib2,
179 libzeitgeist-1.0-1
177Recommends: gnome-settings-daemon180Recommends: gnome-settings-daemon
178Suggests: gromit181Suggests: gromit
179Description: Plugins for the Totem media player182Description: Plugins for the Totem media player
180183
=== modified file 'debian/rules'
--- debian/rules 2011-08-10 04:50:38 +0000
+++ debian/rules 2011-10-26 20:23:51 +0000
@@ -12,7 +12,7 @@
12include /usr/share/gnome-pkg-tools/1/rules/gnome-get-source.mk12include /usr/share/gnome-pkg-tools/1/rules/gnome-get-source.mk
1313
14DEB_CONFIGURE_EXTRA_FLAGS += \14DEB_CONFIGURE_EXTRA_FLAGS += \
15 --enable-vala=no \15 --enable-vala=yes \
16 --disable-run-in-source-tree \16 --disable-run-in-source-tree \
17 --disable-scrollkeeper17 --disable-scrollkeeper
1818
1919
=== modified file 'debian/totem-plugins.install'
--- debian/totem-plugins.install 2011-05-20 14:37:43 +0000
+++ debian/totem-plugins.install 2011-10-26 20:23:51 +0000
@@ -1,3 +1,5 @@
1debian/tmp/usr/lib/totem/plugins/[!gci]*1debian/tmp/usr/lib/totem/plugins/[!gci]*
2debian/tmp/usr/lib/totem/plugins/chapters2debian/tmp/usr/lib/totem/plugins/chapters
3debian/tmp/usr/lib/totem/plugins/im-status3debian/tmp/usr/lib/totem/plugins/im-status
4debian/tmp/usr/lib/totem/plugins/zeitgeist-dp
5

Subscribers

People subscribed via source and target branches