<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Dec 5, 2016, at 5:20 AM, Jussi Kukkonen <<a href="mailto:jussi.kukkonen@intel.com" class="">jussi.kukkonen@intel.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><br class="Apple-interchange-newline"><br style="font-family: Times-Roman; font-size: 14px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><div class="gmail_quote" style="font-family: Times-Roman; font-size: 14px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;">On 1 December 2016 at 10:37, Khem Raj<span class="Apple-converted-space"> </span><span dir="ltr" class=""><<a href="mailto:raj.khem@gmail.com" target="_blank" class="">raj.khem@gmail.com</a>></span><span class="Apple-converted-space"> </span>wrote:<br class=""><blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex; border-left-width: 1px; border-left-style: solid; border-left-color: rgb(204, 204, 204); padding-left: 1ex;">When configure pokes for wayland-protocols isntallations it ended up<br class="">using the ones from host, which is because it did not account for sysroot<br class="">prefix<br class=""><br class="">Signed-off-by: Khem Raj <<a href="mailto:raj.khem@gmail.com" class="">raj.khem@gmail.com</a>><br class="">---<br class=""> .../gstreamer/gstreamer1.0-<wbr class="">plugins-bad.inc | 2 +-<br class=""> ...G_CONFIG_SYSROOT_DIR-to-<wbr class="">pkg-config-output.patch | 37 ++++++++++++++++++++++<br class=""> .../gstreamer/<a href="http://gstreamer1.0-plugins-bad_1.10.1.bb/" rel="noreferrer" target="_blank" class="">gstreamer1.0-<wbr class="">plugins-bad_1.10.1.bb</a> | 1 +<br class=""> 3 files changed, 39 insertions(+), 1 deletion(-)<br class=""> create mode 100644 meta/recipes-multimedia/<wbr class="">gstreamer/gstreamer1.0-<wbr class="">plugins-bad/0001-Prepend-PKG_<wbr class="">CONFIG_SYSROOT_DIR-to-pkg-<wbr class="">config-output.patch<br class=""><br class="">diff --git a/meta/recipes-multimedia/<wbr class="">gstreamer/gstreamer1.0-<wbr class="">plugins-bad.inc b/meta/recipes-multimedia/<wbr class="">gstreamer/gstreamer1.0-<wbr class="">plugins-bad.inc<br class="">index d26a6a9..d3c5326 100644<br class="">--- a/meta/recipes-multimedia/<wbr class="">gstreamer/gstreamer1.0-<wbr class="">plugins-bad.inc<br class="">+++ b/meta/recipes-multimedia/<wbr class="">gstreamer/gstreamer1.0-<wbr class="">plugins-bad.inc<br class="">@@ -64,7 +64,7 @@ PACKAGECONFIG[srtp] = "--enable-srtp,--disable-srtp,<wbr class="">libsrtp"<br class=""> PACKAGECONFIG[uvch264] = "--enable-uvch264,--disable-<wbr class="">uvch264,libusb1 libgudev"<br class=""> PACKAGECONFIG[voaacenc] = "--enable-voaacenc,--disable-<wbr class="">voaacenc,vo-aacenc"<br class=""> PACKAGECONFIG[voamrwbenc] = "--enable-voamrwbenc,--<wbr class="">disable-voamrwbenc,vo-<wbr class="">amrwbenc"<br class="">-PACKAGECONFIG[wayland] = "--enable-wayland,--disable-<wbr class="">wayland,wayland-native wayland"<br class="">+PACKAGECONFIG[wayland] = "--enable-wayland,--disable-<wbr class="">wayland,wayland-native wayland wayland-protocols"<br class=""> PACKAGECONFIG[webp] = "--enable-webp,--disable-webp,<wbr class="">libwebp"<br class=""><br class=""> # these plugins have not been ported to 1.0 (yet):<br class="">diff --git a/meta/recipes-multimedia/<wbr class="">gstreamer/gstreamer1.0-<wbr class="">plugins-bad/0001-Prepend-PKG_<wbr class="">CONFIG_SYSROOT_DIR-to-pkg-<wbr class="">config-output.patch b/meta/recipes-multimedia/<wbr class="">gstreamer/gstreamer1.0-<wbr class="">plugins-bad/0001-Prepend-PKG_<wbr class="">CONFIG_SYSROOT_DIR-to-pkg-<wbr class="">config-output.patch<br class="">new file mode 100644<br class="">index 0000000..eb789df<br class="">--- /dev/null<br class="">+++ b/meta/recipes-multimedia/<wbr class="">gstreamer/gstreamer1.0-<wbr class="">plugins-bad/0001-Prepend-PKG_<wbr class="">CONFIG_SYSROOT_DIR-to-pkg-<wbr class="">config-output.patch<br class="">@@ -0,0 +1,37 @@<br class="">+From c271503d7e233428ac0323c51d6517<wbr class="">113e26bef7 Mon Sep 17 00:00:00 2001<br class="">+From: Khem Raj <<a href="mailto:raj.khem@gmail.com" class="">raj.khem@gmail.com</a>><br class="">+Date: Thu, 1 Dec 2016 00:27:13 -0800<br class="">+Subject: [PATCH] Prepend PKG_CONFIG_SYSROOT_DIR to pkg-config output<br class="">+<br class="">+In cross environment we have to prepend the sysroot to the path found by<br class="">+pkgconfig since the path returned from pkgconfig does not have sysroot prefixed<br class="">+it ends up using the files from host system. If build host has wayland installed<br class="">+the build will succeed but if you dont have wayland-protocols installed on build host then<br class="">+it wont find the files on build host<br class="">+<br class="">+This should work ok with non sysrooted builds too since in those cases PKG_CONFIG_SYSROOT_DIR<br class="">+will be empty<br class="">+<br class="">+Upstream-Status: Pending<br class="">+<br class="">+Signed-off-by: Khem Raj <<a href="mailto:raj.khem@gmail.com" class="">raj.khem@gmail.com</a>><br class="">+---<br class="">+<span class="Apple-converted-space"> </span><a href="http://configure.ac/" rel="noreferrer" target="_blank" class="">configure.ac</a><span class="Apple-converted-space"> </span>| 2 +-<br class="">+ 1 file changed, 1 insertion(+), 1 deletion(-)<br class="">+<br class="">+diff --git a/<a href="http://configure.ac/" rel="noreferrer" target="_blank" class="">configure.ac</a><span class="Apple-converted-space"> </span>b/<a href="http://configure.ac/" rel="noreferrer" target="_blank" class="">configure.ac</a><br class="">+index f8ac96b..dc87b08 100644<br class="">+--- a/<a href="http://configure.ac/" rel="noreferrer" target="_blank" class="">configure.ac</a><br class="">++++ b/<a href="http://configure.ac/" rel="noreferrer" target="_blank" class="">configure.ac</a><br class="">+@@ -2233,7 +2233,7 @@ AG_GST_CHECK_FEATURE(WAYLAND, [wayland sink], wayland , [<br class="">+ PKG_CHECK_MODULES(WAYLAND_<wbr class="">PROTOCOLS, wayland-protocols >= 1.4, [<br class="">+ if test "x$wayland_scanner" != "x"; then<br class="">+ HAVE_WAYLAND="yes"<br class="">+- AC_SUBST(WAYLAND_PROTOCOLS_<wbr class="">DATADIR, `$PKG_CONFIG --variable=pkgdatadir wayland-protocols`)<br class="">++ AC_SUBST(WAYLAND_PROTOCOLS_<wbr class="">DATADIR, ${PKG_CONFIG_SYSROOT_DIR}`$<wbr class="">PKG_CONFIG --variable=pkgdatadir wayland-protocols`)<br class=""></blockquote><div class=""><br class=""></div><div class="">I believe this breaks multilib because wayland-protocols is allarch so PKG_CONFIG_SYSROOT_DIR may then be incorrect.</div></div></div></blockquote><div><br class=""></div><div>Realized my last reply is too terse. expanding on it a bit more.</div><div>this is not really the case. Since PKG_CONFIG_SYSROOT_DIR is set to STAGING_DIR_HOST which is then set to ${STAGING_DIR}/${MACHINE} so effectively its just</div><div>reusing an existing var compared to your case where a new redundant variable is introduced. </div><div><br class=""></div><div>having said that, there could be a multilib issue if pkgdatadir thats in .pc file from wayland-protocols package is using variables like ${libdir} which then are</div><div>multilib dependent, if this is the case then we need to fix .pc file. however when I look at the <target-sysroot>/usr/share/pkgconfig/wayland-protocols.pc it has</div><div><br class=""></div><div><div>prefix=/usr</div><div>datarootdir=${prefix}/share</div><div>pkgdatadir=/usr/share/wayland-protocols</div><div><br class=""></div><div>seems to be free of multilib deps.</div></div><br class=""><blockquote type="cite" class=""><div class=""><div class="gmail_quote" style="font-family: Times-Roman; font-size: 14px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;"><div class=""><br class=""></div><div class="">I've set "WAYLAND_PROTOCOLS_SYSROOT_DIR=${STAGING_DIR}/${MACHINE}" in EXTRA_OECONF in other recipes and used that variable in the patches. It's not pretty but seems to work.<br class=""></div><div class=""><br class=""></div><div class="">Jussi</div><div class=""><br class=""></div><blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex; border-left-width: 1px; border-left-style: solid; border-left-color: rgb(204, 204, 204); padding-left: 1ex;">+ else<br class="">+ AC_MSG_RESULT([wayland-scanner is required to build the wayland plugin])<br class="">+ HAVE_WAYLAND="no"<br class="">+--<br class="">+2.10.2<br class="">+<br class="">diff --git a/meta/recipes-multimedia/<wbr class="">gstreamer/<a href="http://gstreamer1.0-plugins-bad_1.10.1.bb/" rel="noreferrer" target="_blank" class="">gstreamer1.0-<wbr class="">plugins-bad_1.10.1.bb</a><span class="Apple-converted-space"> </span>b/meta/recipes-multimedia/<wbr class="">gstreamer/<a href="http://gstreamer1.0-plugins-bad_1.10.1.bb/" rel="noreferrer" target="_blank" class="">gstreamer1.0-<wbr class="">plugins-bad_1.10.1.bb</a><br class="">index 9cd892e..6c6f011 100644<br class="">--- a/meta/recipes-multimedia/<wbr class="">gstreamer/<a href="http://gstreamer1.0-plugins-bad_1.10.1.bb/" rel="noreferrer" target="_blank" class="">gstreamer1.0-<wbr class="">plugins-bad_1.10.1.bb</a><br class="">+++ b/meta/recipes-multimedia/<wbr class="">gstreamer/<a href="http://gstreamer1.0-plugins-bad_1.10.1.bb/" rel="noreferrer" target="_blank" class="">gstreamer1.0-<wbr class="">plugins-bad_1.10.1.bb</a><br class="">@@ -14,6 +14,7 @@ SRC_URI = " \<br class=""> <a href="file://0001-gstreamer-gl.pc" class="">file://0001-gstreamer-gl.pc</a>.<wbr class="">in-don-t-append-GL_CFLAGS-to-<wbr class="">CFLAGS.patch \<br class=""> <a href="file:///0009-glimagesink-" class="">file://0009-glimagesink-</a><wbr class="">Downrank-to-marginal.patch \<br class=""> <a href="file:///0001-introspection.m4-" class="">file://0001-introspection.m4-</a><wbr class="">prefix-pkgconfig-paths-with-<wbr class="">PKG_CON.patch \<br class="">+ <a href="file:///0001-Prepend-PKG_" class="">file://0001-Prepend-PKG_</a><wbr class="">CONFIG_SYSROOT_DIR-to-pkg-<wbr class="">config-output.patch \<br class=""> "<br class=""> SRC_URI[md5sum] = "<wbr class="">491d2d5aab55ffc60c66e714d3d664<wbr class="">ea"<br class=""> SRC_URI[sha256sum] = "<wbr class="">133e0ed9fe21011b15d3898e3d3a9d<wbr class="">17ab74eed31996da2e353353e688ca<wbr class="">921d"<br class=""><span class="gmail-HOEnZb"><font color="#888888" class="">--<br class="">2.10.2<br class=""><br class="">--<br class="">______________________________<wbr class="">_________________<br class="">Openembedded-core mailing list<br class=""><a href="mailto:Openembedded-core@lists.openembedded.org" class="">Openembedded-core@lists.<wbr class="">openembedded.org</a><br class=""><a href="http://lists.openembedded.org/mailman/listinfo/openembedded-core" rel="noreferrer" target="_blank" class="">http://lists.openembedded.org/<wbr class="">mailman/listinfo/openembedded-<wbr class="">core</a></font></span></blockquote></div></div></blockquote></div><br class=""></body></html>