On 2/18/20 6:43 PM, Mittal, Anuj wrote:
On Tue, 2020-02-18 at 18:39 -0800, Khem Raj wrote:
On Tue, Feb 18, 2020 at 6:37 PM Anuj Mittal <[email protected]>
wrote:
Use the correct variable so options actually get used and use the
correct qt option name to prevent errors.

Signed-off-by: Anuj Mittal <[email protected]>
---
  .../gstreamer/gstreamer1.0-plugins-good_1.16.1.bb             | 4
++--
  1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-multimedia/gstreamer/gstreamer1.0-
plugins-good_1.16.1.bb b/meta/recipes-
multimedia/gstreamer/gstreamer1.0-plugins-good_1.16.1.bb
index a6eb9986dd..07a0bf5085 100644
--- a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-
good_1.16.1.bb
+++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-
good_1.16.1.bb
@@ -55,7 +55,7 @@ PACKAGECONFIG[x11]        =
"${X11ENABLEOPTS},${X11DISABLEOPTS},${X11DEPENDS}"
  # This is better done in a separate qt5 layer (which then should
add a "qt5" packageconfig
  # in a gstreamer1.0-plugins-good bbappend).

-EXTRA_OECONF += " \
+EXTRA_OEMESON += " \
      -Daalib=disabled \
      -Ddirectsound=disabled \
      -Ddv=disabled \
@@ -64,7 +64,7 @@ EXTRA_OECONF += " \
      -Doss4=disabled \
      -Dosxaudio=disabled \
      -Dosxvideo=disabled \
-    -Dqt=disabled \
+    -Dqt5=disabled \
this perhaps should be a packageconfig so people using meta-qt5 can
enable it
Yes probably but in a separate change I think. This just ensures that
the right options are used and nothing else is changed.

perhaps no, since it will regress meta-qt5 which already does something like this

https://github.com/meta-qt5/meta-qt5/blob/master/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good_%25.bbappend


this being wrong in OE-core was ok for meta-qt5.


Thanks,

Anuj

      -Dshout2=disabled \
      -Dtwolame=disabled \
      -Dwaveform=disabled \
--
2.24.1

--
_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.openembedded.org/mailman/listinfo/openembedded-core
--
_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Reply via email to