Port to GStreamer 1.0
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
| Alarm Clock |
Fix Released
|
Low
|
Unassigned |
Bug Description
GStreamer 0.10 has been obsoleted and superseeded by GStreamer 1.0 branch.
Porting alarm-clock-applet to GStreamer seems fairly simple (as not many APIs are being used):
- replace gstreamer-0.10 in configure.ac with gstreamer-1.0 (detect the new version)
- Replace playbin2 in src/player.c with playbin (the element was renamed).
Related branches
DimStar (dominique-launchpad-net) wrote : | #1 |
Johannes H. Jensen (joh) wrote : | #2 |
DimStar (dominique-launchpad-net) wrote : | #3 |
I think it's questionable to support both: since GNOME 3.6, gstreamer 1.0 is a requirement (so, 6 month ago) and as such you can safely assume that all major distros will have support for it shortly.
There is no longer any maintenance done on the GStreamer 0.10 branch.
Johannes H. Jensen (joh) wrote : | #4 |
Ah in that case you have me convinced! I tried your patch but there seems to be a problem:
** (alarm-
Program received signal SIGSEGV, Segmentation fault.
0x0000000000409e0e in media_player_start (player=0x0) at player.c:236
236 bus = gst_pipeline_
The segfault is probably because we previously failed to create the player (player=NULL).
Perhaps there are some more API changes with GStreamer 1.0 which were missed in the patch?
DimStar (dominique-launchpad-net) wrote : | #5 |
Ups.. yes, of course.. sorry for that.
gconfaudiosink and gconfvideosink no longer exist... (the segfault is a consequence of how the code is written though.. that's an error on itself and not really due to the patch).
Just reworking the patch and resubmitting...
do you have a good test plan for what all should work?
DimStar (dominique-launchpad-net) wrote : | #6 |
A reworked patch... I created an alarm entry (timered) and the sound worked without crashes... also, testing sounds works fine (switched from gcong*sink to auto*sink, which represents the same in the end)
Johannes H. Jensen (joh) wrote : | #7 |
Thank you, the reworked patch works perfectly. Committed to trunk at revision 226.
I'll look into the (separate) segfault case.
Changed in alarm-clock: | |
importance: | Undecided → Low |
status: | New → Fix Committed |
Sebastian Dröge (slomo) wrote : | #8 |
It would be good to get a release out with this, there are already distros that don't ship GStreamer 0.10 anymore.
Changed in alarm-clock: | |
status: | Fix Committed → Fix Released |
Patch looks good. I'm not sure we want to drop support for GStreamer 0.10 though, as it seems to be prevalent in a lot of distros still. How about supporting both?