<div dir="ltr">On 5 December 2017 at 13:15, Daniel Stone <span dir="ltr"><<a href="mailto:daniel@fooishbar.org" target="_blank">daniel@fooishbar.org</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Quentin,<br>
I've added Ross and Fabien, who've been looking at this for Yocto.<br>
<br>
On 18 August 2017 at 10:30, Quentin Glidic<br>
<<a href="mailto:sardemff7%2Bwayland@sardemff7.net">sardemff7+wayland@sardemff7.<wbr>net</a>> wrote:<br>
> --- a/wayland-scanner.m4<br>
> +++ b/wayland-scanner.m4<br>
> @@ -1,3 +1,5 @@<br>
> +#serial 2<br>
<br>
I don't quite understand the significance of this.<br></blockquote><div><br></div><div>autoconf macros can be versioned and this is how that is done.  You don't see it much in practise because the one piece of autoconf which should respect the versions doesn't, but I approve of the intention!</div><div><br></div><div>Fixing aclocal is on my long list of Things To Do, but this involves 1) autoconf 2) m4 so would mean more mental strength than I currently have.</div><div> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
> +# WL_PROG_WAYLAND_SCANNER()<br>
> +AC_DEFUN([WL_PROG_WAYLAND_<wbr>SCANNER], [<br>
> +    AC_REQUIRE([AC_CANONICAL_<wbr>BUILD])<br>
> +    AC_REQUIRE([PKG_PROG_PKG_<wbr>CONFIG])<br>
> +    wl_cv_native_pkg_config=<br>
> +    AS_IF([test x${cross_compiling} = xyes], [<br>
> +        wl_cv_native_pkg_config=${<wbr>build}-pkg-config<br>
> +        AC_PATH_PROGS([wl_cv_native_<wbr>pkg_config], [${build}-pkg-config])<br>
<br>
Ah, no thanks. I would prefer to have this passed in as<br>
$HOST_PKG_CONFIG (following the $HOST_CC precedent), falling back to<br>
just plain 'pkg-config'. Guessing triplets is a bit too much like<br>
magic, especially when you can be 'cross-compiling' to the same<br>
triplet.<br></blockquote><div><br></div><div>I'm not aware of a precedent for "this is the host pkgconfig" but I endorse HOST_PKG_CONFIG.  Clear semantics and we could export it globally.</div><div><br></div><div>It would be great if someone pushed that into pkg.m4...</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
The rest looks good to me, so assuming we can test it in Yocto and it<br>
works fine there, then it should be good to merge.<br></blockquote><div><br></div><div>I'll definitely throw a v2 at our autobuilder cluster to see what happens.  This builds gtk+, xserver-xorg and gstreamer with wayland support, which isn't everything but should be enough to find any gotchas.</div><div><br></div><div>Ross </div></div></div></div>