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 661163 - [patch] add PES support, take 2
[patch] add PES support, take 2
Status: RESOLVED WONTFIX
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
git master
Other Linux
: Normal enhancement
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on: 657343
Blocks:
 
 
Reported: 2011-10-07 09:54 UTC by Stas Sergeev
Modified: 2013-08-26 08:12 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
the patch (114.45 KB, patch)
2011-10-07 09:54 UTC, Stas Sergeev
needs-work Details | Review

Description Stas Sergeev 2011-10-07 09:54:43 UTC
Created attachment 198516 [details] [review]
the patch

Hi.

Here's the patch.
I decided to open a new entry for it, leaving the previous
one for the small patches to the existing code, that are
relevant to this.
Comment 1 Sebastian Dröge (slomo) 2013-08-23 11:21:01 UTC
Needs to be updated to 1.0, and also the patch contains some unrelated changes. Are you planning to work on that?
Comment 2 Stas Sergeev 2013-08-23 11:25:20 UTC
What do you mean by an unrelated changes?

In bug 657343 it was said that the different approach
is needed for gst-1.0, and on that I am not planning to
work. If you can get any use of what is currently here,
then I can be in service (like, eg, at least update it
with the latest fixes I did locally).
Comment 3 Sebastian Dröge (slomo) 2013-08-23 11:29:24 UTC
It includes a DVB video sink too
Comment 4 Stas Sergeev 2013-08-23 11:32:59 UTC
PES video sink for DVB is an essential part of
the PES support (at least, as was in my plan).
Comment 5 Edward Hervey 2013-08-23 11:34:15 UTC
The first step would be to add a PES GstMeta which demuxers could output, and which muxers (or sinks) could take. But the caps would remain standard gstreamer caps (ex: video/x-h264).

I'm still not 100% convinced about the usefulness of standalone PES parser/packetizer/depacketizer.
Comment 6 Stas Sergeev 2013-08-24 10:31:39 UTC
(In reply to comment #2)
> In bug 657343 it was said that the different approach
> is needed for gst-1.0, and on that I am not planning to
> work.
Actually, this doesn't even look plausible.
I could spend months of work updating the code,
and then, 2 years of rot in bugzilla, and it
is useless again.
Comment 7 Sebastian Dröge (slomo) 2013-08-26 08:02:27 UTC
Edward, so let's keep this open or not? Is anybody realistically going to work on supporting this properly?
Comment 8 Edward Hervey 2013-08-26 08:12:04 UTC
Let's close it and re-open when/if someone wants this.