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 328830 - [0.10] Funny error when trying to play an AVI over HTTP
[0.10] Funny error when trying to play an AVI over HTTP
Status: RESOLVED FIXED
Product: totem
Classification: Core
Component: GStreamer backend
1.3.x
Other Linux
: Normal minor
: ---
Assigned To: Maintainer alias for GStreamer component of Totem
Maintainer alias for GStreamer component of Totem
Depends on:
Blocks:
 
 
Reported: 2006-01-27 13:06 UTC by Andy Wingo
Modified: 2006-02-03 17:05 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
proposed patch (1.64 KB, patch)
2006-01-27 14:32 UTC, Tim-Philipp Müller
reviewed Details | Review
new patch (1.64 KB, patch)
2006-01-27 15:43 UTC, Tim-Philipp Müller
committed Details | Review

Description Andy Wingo 2006-01-27 13:06:44 UTC
If I try to play an AVI over HTTP, for example http://plastic.fork.pl/195_95_party_version.avi, I get an error from Totem with the primary text:

"Totem could not play 'http://plastic.fork.pl/195_95_party_version.avi'."

and secondary,

"GstAviDemux cannot work in push mode. The operation is not supported with this source element or protocol."

The real bug is in avidemux not supporting push-mode operation, but more helpful would be a message saying something like "Cannot play this file over the network. Try downloading it to disk first" or something. In any case this message shouldn't be shown to the user, it's more appropriate to debug information.
Comment 1 Tim-Philipp Müller 2006-01-27 14:32:53 UTC
Created attachment 58208 [details] [review]
proposed patch

This makes totem display an error dialog with the text you suggested.

Attaching as patch for now until a final phrasing has been agreed on, as string changes have to be announced these days.
Comment 2 Andy Wingo 2006-01-27 15:17:35 UTC
1) The string should end in a period.

2) The test is incorrect; it should check for GstAviDemux I suppose.
Comment 3 Tim-Philipp Müller 2006-01-27 15:43:21 UTC
Created attachment 58218 [details] [review]
new patch


1) Ok.

2) It is in fact the typefind element that emits this message (I did actually
   try the patch :P). Arguably this could be changed on the GStreamer side
   so that each and every demuxer/decoder that only operates in pull-mode
   posts an appropriate error message itself if it can't activate, but I am
   not really sure that is a better solution. As it stands, this patch fixes
   the problem with current Gtreamer. It doesn't prevent us from changing
   things in GStreamer later, does it?
Comment 4 Andy Wingo 2006-01-27 17:27:08 UTC
Ah, I was just speaking from the error message that I saw, which printed GstAviDemux. Never mind me then. :-)
Comment 5 Andy Wingo 2006-02-03 16:48:47 UTC
See also comment #5 of bug #302606:

"the following error alert appears:

   Totem could not play 'fd://0'.

   GstQTDemux cannot work in push mode. The operation is
   not supported with this source element or protocol.
"
So also qtdemux.
Comment 6 Tim-Philipp Müller 2006-02-03 17:05:58 UTC
Same thing, isn't it? Fixed in CVS, and string change announced to the appropriate mailing lists.

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

        * src/backend/bacon-video-widget-gst-0.10.c:
        (bvw_error_from_gst_error):
          Show a more meaningful error message when playback fails
          because the decoder/demuxer can only operate pull-based,
          but the source does not support that (e.g. playing AVI
          videos or FLAC files over the network) (#328830).