From: Peter Seiderer <ps.report@gmx.net>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH v4 2/8] gst-plugins-${base, good, bad, ugly}: bump to 1.14.0
Date: Tue, 10 Apr 2018 18:28:13 +0200 [thread overview]
Message-ID: <20180410182813.691be3ff@gmx.net> (raw)
In-Reply-To: <20180410100736.2e6f8f7b@windsurf>
Hello Thomas, Adam,
On Tue, 10 Apr 2018 10:07:36 +0200, Thomas Petazzoni <thomas.petazzoni@bootlin.com> wrote:
> Hello Peter,
>
> I talked with Adam about this gstreamer1 patch series, and apparently
> your comment below isn't clear to Adam, and this explains why he hasn't
> resent the patch series.
>
> On Thu, 22 Mar 2018 23:55:37 +0100, Peter Seiderer wrote:
>
> > > Add the following to gst1-plugins-base and remove them from plugins-bad:
> > > - opengl
> > > - gles2
> > > - glx
> > > - egl
> > > - x11
> > > - wayland
> >
> > This still contradicts your own statement [1]:
> >
> > Not entirely! There is --enable-wayland now in both base AND bad!
>
> Indeed, the gst1-plugins-bad package still has a
> --enable-wayland/--disable-wayland option:
>
> output/build/gst1-plugins-bad-1.14.0 $ ./configure --help | grep wayland
> --disable-wayland disable wayland sink: wayland
This is the 'old' one in gstreamer and buildroot and should be kept (NOT moved from bad to base)....
>
> And in base:
>
> output/build/gst1-plugins-base-1.14.0 $ ./configure --help | grep wayland
> --enable-wayland Enable Wayland support (requires EGL) [default=auto]
This is the 'new' one in gstreamer, yet without handling in buildroot, enables
wayland support in libgstgl (which is used by some plugins, but I believe not
by the waylandsink plugin from bad)...
>
> > > Changes v2 -> v3:
> > > - Removed wayland references from plugins-bad (Peter)
> >
> > No, see suggestion from [2]:
> >
> > I think there is a difference, the bad/wayland option enables extra gstreamer
> > plugins/elements, the base/wayland opton enables no extra plugins/elements but
> > enables wayland support in the qt/gtk plugins (did not look further if it
> > results only in wayland support in some elements or extra elements in the same
> > plugin library), in case it enables only wayland support, I think no
> > user visible option is needed (if a user visible option is needed there is
> > definitely a change in option name and help text needed)
>
> It's not entirely clear to me what are the two options doing.
>
> In gst1-plugins-base, the --enable-wayland option requires EGL support,
> and will enable some wayland related code in gst-libs/gl/. Peter, do
> you understand what this does, compared to the wayland sink provided in
> gst1-plugins-bad ?
Yes, your short analysis is right (and sorry, I can add no more details what
is enabled/disabled in the resulting libgstgl library)....
>
> In gst1-plugins-bad, the --enable-wayland option enables the build of a
> wayland sink.
Yes, see [1]...
Regards,
Peter
[1] https://gstreamer.freedesktop.org/data/doc/gstreamer/head/gst-plugins-bad/html/gst-plugins-bad-plugins-waylandsink.html
>
> Best regards,
>
> Thomas
next prev parent reply other threads:[~2018-04-10 16:28 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-03-22 17:32 [Buildroot] [PATCH v4 1/8] gstreamer1: bump to 1.14.0 Adam Duskett
2018-03-22 17:32 ` [Buildroot] [PATCH v4 2/8] gst-plugins-${base, good, bad, ugly}: " Adam Duskett
2018-03-22 22:55 ` Peter Seiderer
2018-03-28 9:07 ` Martin Kepplinger
2018-04-10 8:07 ` Thomas Petazzoni
2018-04-10 16:28 ` Peter Seiderer [this message]
2018-04-10 21:40 ` Thomas Petazzoni
2018-04-11 6:56 ` Peter Seiderer
2018-04-11 9:35 ` Thomas Petazzoni
2018-04-12 13:41 ` Adam Duskett
2018-04-12 19:21 ` Peter Seiderer
2018-04-10 8:08 ` Thomas Petazzoni
2018-03-22 17:32 ` [Buildroot] [PATCH v4 3/8] gst-omx: " Adam Duskett
2018-03-22 17:32 ` [Buildroot] [PATCH v4 4/8] gst1-rtsp-server: " Adam Duskett
2018-03-22 17:32 ` [Buildroot] [PATCH v4 5/8] gst1-vaapi: " Adam Duskett
2018-03-22 17:32 ` [Buildroot] [PATCH v4 6/8] gst1-libav: " Adam Duskett
2018-03-22 17:32 ` [Buildroot] [PATCH v4 7/8] gst1-validate: " Adam Duskett
2018-03-22 17:32 ` [Buildroot] [PATCH v4 8/8] gst1-imx: add hash for license file Adam Duskett
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20180410182813.691be3ff@gmx.net \
--to=ps.report@gmx.net \
--cc=buildroot@busybox.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox