Ticket #1047 (closed enhancement: fixed)

Opened 2 years ago

Last modified 1 year ago

Regression: Media library loading breaks stream output

Reported by: courmisch Assigned to:
Priority: high Milestone: 0.9.0-test1
Component: Playlist Version: master
Severity: major Keywords:
Cc: funman Platform(s): all
Difficulty: hard Work status: Not started

Description (Last modified by funman)

When streaming output is enabled, VLC tries to use stream out the saved XSP playlist file at startup (instead of loading into the VLC playlist). Essentially, this renders the --sout parameter completely unusable.

Not only is it silly to stream the playlist, but it actually does prevent stream output from working afterwards.

Fix is trivial and detailed in the ticket comments

Change History

10/02/07 19:38:26 changed by courmisch

  • description changed.

13/03/07 18:50:54 changed by courmisch

Currently, ml.xsp support is disabled. Whoever fix the problem before you reintroduce it. Thanks.

05/04/07 16:13:38 changed by courmisch

  • description changed.

05/04/07 17:42:56 changed by zorglub

  • difficulty changed from hard to easy.

Simply clear the sout (and maybe other potentially not side-effect-free) options for the ML item in playlist_MLLoad

05/04/07 19:08:50 changed by funman

  • status changed from new to closed.
  • resolution set to fixed.

(In [19698]) Streaming output works again. Closes #1047

05/04/07 20:29:39 changed by courmisch

  • status changed from closed to reopened.
  • resolution deleted.

05/04/07 20:30:07 changed by courmisch

The problem is completely unchanged. XSP is still being streamed which is completely idiotic.

15/04/07 14:29:06 changed by courmisch

  • difficulty changed from easy to hard.

Considering the large number of options in Init (src/input/input.c), this is not a solution. Worst yet, it would break every time someone adds a new sout option.

15/04/07 15:11:47 changed by zorglub

Not at all. sout options are only considered if --sout is not empty. My guess is that there are maybe 2 or 3 input options to clear

15/04/07 15:15:38 changed by zorglub

OK, there are 5

  • sub-file
  • sout
  • input-repeat
  • input-slave
  • sout

maybe demux and access, I don't remember if access/demux://url overrides --demux and --access

15/04/07 15:16:47 changed by zorglub

Alternatively, a ":meta-file" option that takes care of overriding the dangerous options directly in input, so that we ensure this is kept up to date.

Because this bug also applies to skin loading

15/04/07 15:22:03 changed by courmisch

That's what I keep saying. This whole solution is too brittle.

Anyway, I am perfectly happy with the current state (that XSP is commented out), and I have wasted enough time on this. I won't take care of this.

15/04/07 15:24:33 changed by courmisch

  • severity changed from blocker to major.
  • summary changed from XSP breaks streaming output to Playlist autoload disabled to preserve stream output.
  • priority changed from highest to normal.
  • milestone changed from 0.9.0 bugs to Features paradize.
  • type changed from defect to enhancement.
  • description changed.

15/04/07 15:24:45 changed by courmisch

  • status changed from reopened to new.
  • owner deleted.

15/04/07 15:26:00 changed by zorglub

  • description changed.

14/08/07 01:13:52 changed by funman

  • priority changed from normal to high.
  • cc set to funman.
  • milestone changed from Features paradize to 0.9.0 bugs.

updating as media library is a new function used in QT4 interface.

30/08/07 02:59:13 changed by funman

(In [21574]) Reenables media library loading Uses --no-media-library if you don't want to use it (see #1047)

30/08/07 06:45:04 changed by funman

  • summary changed from Playlist autoload disabled to preserve stream output to Media library loading breaks stream output.

12/09/07 21:15:13 changed by funman

  • description changed.

22/09/07 19:58:40 changed by courmisch

  • summary changed from Media library loading breaks stream output to Regression: Media library loading breaks stream output.

Why has this been re-enabled?? This is still unfixed as far as I can tell.

22/09/07 20:17:52 changed by courmisch

  • milestone changed from 0.9.0 bugs to 0.9.0-test1.

28/09/07 18:08:56 changed by funman

  • status changed from new to closed.
  • resolution set to fixed.

(In [22294]) Adds an input option "meta-file" to be used by input that aren't real inputs (like playlists). Re-enables media library. Fixes #1047 (nth attempt).