Merge lp:~kiwinote/software-center/bug893247 into lp:software-center

Proposed by Kiwinote
Status: Merged
Merged at revision: 2566
Proposed branch: lp:~kiwinote/software-center/bug893247
Merge into: lp:software-center
Diff against target: 41 lines (+14/-2)
2 files modified
debian/changelog (+10/-0)
softwarecenter/ui/gtk3/widgets/videoplayer.py (+4/-2)
To merge this branch: bzr merge lp:~kiwinote/software-center/bug893247
Reviewer Review Type Date Requested Status
Gary Lasker (community) Approve
Review via email: mp+82933@code.launchpad.net

Description of the change

In videoplayer.py we import Gst, but the current version of the code doesn't use it. Importing Gst requires a dependency on gir1.2-gstreamer-0.10, but this isn't installed by default. This results in s-c hanging on startup.

To post a comment you must log in.
Revision history for this message
Gary Lasker (gary-lasker) wrote :

D'oh! Good catch, kiwinote, thanks!

review: Approve

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-11-21 13:03:59 +0000
3+++ debian/changelog 2011-11-21 20:40:28 +0000
4@@ -1,3 +1,13 @@
5+software-center (5.1.2.1) UNRELEASED; urgency=low
6+
7+ [ Kiwinote ]
8+ * sc/ui/gtk3/widgets/videoplayer.py:
9+ - don't import Gst - we don't use it at the moment
10+ this unbreaks startup for those who don't have gir1.2-gstreamer-0.10
11+ installed (LP: #893247)
12+
13+ -- Kiwinote <kiwinote@gmail.com> Mon, 21 Nov 2011 20:29:35 +0000
14+
15 software-center (5.1.2) precise; urgency=low
16
17 [ Michael Vogt ]
18
19=== modified file 'softwarecenter/ui/gtk3/widgets/videoplayer.py'
20--- softwarecenter/ui/gtk3/widgets/videoplayer.py 2011-11-11 03:20:16 +0000
21+++ softwarecenter/ui/gtk3/widgets/videoplayer.py 2011-11-21 20:40:28 +0000
22@@ -21,7 +21,9 @@
23
24 from gettext import gettext as _
25 from gi.repository import Gdk
26-from gi.repository import Gst
27+# FIXME: uncomment this import and add a dependency on gir1.2-gstreamer-0.10
28+# if we start using VideoPlayerGtk3
29+# from gi.repository import Gst
30 from gi.repository import Gtk
31 from gi.repository import WebKit
32
33@@ -144,7 +146,7 @@
34 if __name__ == "__main__":
35 logging.basicConfig()
36 Gdk.threads_init()
37- Gst.init(sys.argv)
38+ # Gst.init(sys.argv)
39
40 win = get_test_videoplayer_window()
41 Gtk.main()

Subscribers

People subscribed via source and target branches