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 330086 - [0.10] non-user friendly error when no dvd support
[0.10] non-user friendly error when no dvd support
Status: RESOLVED FIXED
Product: totem
Classification: Core
Component: GStreamer backend
1.3.x
Other Linux
: Normal normal
: ---
Assigned To: Maintainer alias for GStreamer component of Totem
Maintainer alias for GStreamer component of Totem
Depends on:
Blocks:
 
 
Reported: 2006-02-06 09:20 UTC by Fernando Herrera
Modified: 2007-03-18 23:31 UTC
See Also:
GNOME target: ---
GNOME version: 2.13/2.14


Attachments
Properly check gst-0.10 for dvd playing support (2.09 KB, patch)
2006-02-06 12:17 UTC, Fernando Herrera
rejected Details | Review

Description Fernando Herrera 2006-02-06 09:20:34 UTC
When trying to play a dvd disc with gst-0.10 backend (without dvd support) the users get an error dialog saying:
"Totem could not play 'dvd://'.
Location not found."

The user should know that there is not support for dvd playing on that version of totem, instead that its dvd has not been located.
Comment 1 Bastien Nocera 2006-02-06 09:25:59 UTC
Yup, the GStreamer backend needs to return FALSE to bacon_video_widget_can_play() for MEDIA_TYPE_DVD
Comment 2 Fernando Herrera 2006-02-06 09:40:37 UTC
so for gst-0.10 is safe to do this?:
diff -u -u -r1.48 bacon-video-widget-gst-0.10.c
--- bacon-video-widget-gst-0.10.c       5 Feb 2006 15:46:17 -0000       1.48
+++ bacon-video-widget-gst-0.10.c       6 Feb 2006 09:39:34 -0000
@@ -3200,9 +3200,9 @@
   switch (type) {
     case MEDIA_TYPE_CDDA:
     case MEDIA_TYPE_VCD:
-    case MEDIA_TYPE_DVD:
       res = TRUE;
       break;
+    case MEDIA_TYPE_DVD:
     default:
       res = FALSE;
       break;
Comment 3 Bastien Nocera 2006-02-06 09:49:45 UTC
Yeah, looks good.
Comment 4 Fernando Herrera 2006-02-06 12:17:33 UTC
Created attachment 58801 [details] [review]
Properly check gst-0.10 for dvd playing support
Comment 5 Tim-Philipp Müller 2006-02-06 12:38:05 UTC
You can check for the presence of a certain elements with

 if (gst_default_registry_check_feature_version ("foobar", 0, 10, 0)) {
   ..
 }

but there's more to full DVD support than just having these elements (e.g. the presence of dvdnavsrc doesn't mean that DVD menus will actually work, that requires all elements in the chain to support that). Also, the elements in question are called 'dvdreadsrc' and 'dvdnavsrc' :)

For now, totem should probably just return FALSE.
Comment 6 Tim-Philipp Müller 2006-02-06 15:07:40 UTC
Now totem says "you don't have the necessary plugins installed", which isn't entirely accurate, but still much better than the previous message.


2006-02-06  Tim-Philipp Müller  <tim at centricular dot net>

       * src/backend/bacon-video-widget-gst-0.10.c:
       (bacon_video_widget_can_play), (bacon_video_widget_get_mrls):
         Don't claim to be able to play back DVDs when we aren't
         (for now, at least) (#330086, patch by: Fernando Herrera)

Comment 7 John Thacker 2006-05-05 15:57:37 UTC
So, since dvdreadsrc has been ported to 0.10 now, can totem be changed
to support playing DVDs (without menus) if it's present with
gstreamer-0.10?
Comment 8 Tim-Philipp Müller 2006-05-05 16:12:54 UTC
The DVD stuff isn't robust enough yet for that (seeking is still a bit wonky, no subtitles yet, no menus yet).

It will be enabled again as soon as it's in a state where we wouldn't immediately incur hundreds of bug reports from users.

For the time being you can watch DVD titles in totem by selecting 'Open Location' and then passing a dvd://<title-number> URI to totem.