After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 362202 - [patch] A replacement of examples/vorbisplay.py for 0.10
[patch] A replacement of examples/vorbisplay.py for 0.10
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-python
git master
Other Linux
: Normal normal
: 0.10.6
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2006-10-14 18:50 UTC by Jason Gerard DeRose
Modified: 2006-10-20 09:48 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch replaces examples/vorbisplay.py with alternative that runs under 0.10 (6.53 KB, patch)
2006-10-14 18:52 UTC, Jason Gerard DeRose
none Details | Review
vorbisplay.py, bare file for easier review (3.02 KB, text/plain)
2006-10-14 18:53 UTC, Jason Gerard DeRose
  Details
Patch replaces examples/vorbisplay.py with alternative that runs under 0.10 (6.55 KB, patch)
2006-10-16 16:03 UTC, Jason Gerard DeRose
none Details | Review
vorbisplay.py, bare file for easier review (3.05 KB, text/plain)
2006-10-16 16:04 UTC, Jason Gerard DeRose
  Details

Description Jason Gerard DeRose 2006-10-14 18:50:45 UTC
examples/vorbisplay.py should be removed, ported, or replaced as it doesn't run under gstreamer0.10.  Below I have a proposed replacement.

Aside from the file name and that they both play an ogg/vorbis file, my replacement doesn't have anything in common with the original, so I put my name in the copyright, as I see it as a replacement rather than a derivative.  If anyone has a problem with this, I'd be happy to change this.

I've submitted it as a patch in case that is easiest to commit, and also as a bare file as to be easier to review.

Considering the number of great examples already included, I'm not certain this is a terribly important example anyway, so simply removing examples/vorbisplay.py is fine with me.
Comment 1 Jason Gerard DeRose 2006-10-14 18:52:29 UTC
Created attachment 74701 [details] [review]
Patch replaces examples/vorbisplay.py with alternative that runs under 0.10
Comment 2 Jason Gerard DeRose 2006-10-14 18:53:15 UTC
Created attachment 74702 [details]
vorbisplay.py, bare file for easier review
Comment 3 Jason Gerard DeRose 2006-10-16 16:03:57 UTC
Created attachment 74821 [details] [review]
Patch replaces examples/vorbisplay.py with alternative that runs under 0.10

adds missing gobject.threads_init()
Comment 4 Jason Gerard DeRose 2006-10-16 16:04:43 UTC
Created attachment 74822 [details]
vorbisplay.py, bare file for easier review

adds missing gobject.threads_init()
Comment 5 Edward Hervey 2006-10-20 09:01:26 UTC
99% of that is the same code as in decodebin.py (bug #362183)
Comment 6 Edward Hervey 2006-10-20 09:48:11 UTC
2006-10-20 Jason Gerard DeRose <jderose@jasonderose.org>
	
	reviewed by: Edward Hervey  <edward@fluendo.com>

	* examples/decodebin.py:
	* examples/gst123:
	* examples/vorbisplay.py:
	Removed gst123 and vorbisplay examples which weren't working with
	0.10, and replaced them with decodebin.py
	Closes #362183 and #362202