<div dir="ltr"><div><div><div>Hi Oliver,<br><br></div>Yes, both patches should work nicely together indeed. <br><br></div>Cheers,<br></div>Tomek<br></div><br><div class="gmail_quote"><div dir="ltr">On Thu, Jul 6, 2017 at 1:31 PM Olivier Blin <<a href="mailto:olivier.blin@softathome.com">olivier.blin@softathome.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Le 06/07/2017 à 11:02, Tomek Bury a écrit :<br>
> Hi,<br>
><br>
> I've attached a trivial patch we discussed yesterday on #wayland<br>
> channel. The patch adds a sysroot prefix to the "pkgdatadir" variable.<br>
> The pkg-config automatically adds sysroot to paths returned by<br>
> --cflags and --libs but *NOT* to --variable.<br>
><br>
> The "pkgdatadir" in wayland-protocols.pc is actually a path pointing<br>
> to the protocol installation directory on the target system but in a<br>
> cross-compilation scenario we need pkg-config to return a temporary<br>
> location in the staging area on the build system just as it does for<br>
> --cflags and --libs.<br>
<br>
Hi,<br>
<br>
I have just sent a complimentary patch for weston, which simplifies the<br>
<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a> and allows to override pkgdatadir from environment.<br>
See [PATCH weston] <a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a>: use PKG_CHECK_VAR for wayland-protocols<br>
<br>
This would not be needed with your solution, but I think it's still a<br>
cleanup that could be merged.<br>
<br>
Thanks<br>
<br>
--<br>
Olivier Blin - SoftAtHome<br>
<br>
</blockquote></div>