<div dir="ltr"><div><div><div>Sorry for a delay, broke my leg :(<br><br></div>I'll send a patch without the extra "/" using git send-mail as soon as I got that one configured.<br><br></div>Cheers,<br></div>Tomek<br><div><div><div><br></div></div></div></div><br><div class="gmail_quote"><div dir="ltr">On Thu, Jul 20, 2017 at 10:45 AM Quentin Glidic <<a href="mailto:sardemff7%2Bwayland@sardemff7.net">sardemff7+wayland@sardemff7.net</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 7/20/17 11:25 AM, Daniel Stone wrote:<br>
> Hi,<br>
><br>
> On 20 July 2017 at 10:14, Quentin Glidic<br>
> <<a href="mailto:sardemff7%2Bwayland@sardemff7.net" target="_blank">sardemff7+wayland@sardemff7.net</a>> wrote:<br>
>> Using ${pc_sysrootdir} sounds like the correct thing to do. However, your<br>
>> patch introduces a triple / when PKG_CONFIG_SYSROOT_DIR is not set.<br>
>> Having two / is already trouble on some platform (though the one we<br>
>> currently work on should be fine, I think), three / would break even more.<br>
>> Please just use "${pc_sysrootdir}@datadir@".<br>
><br>
> Yeah, this all LGTM, and gets my:<br>
> Reviewed-by: Daniel Stone <<a href="mailto:daniels@collabora.com" target="_blank">daniels@collabora.com</a>><br>
><br>
>> Also, using this variable should be in prefix=, IMO, since all the paths<br>
>> should be fixed.<br>
><br>
> I'm wary of doing that, if only because it establishes a pattern. It's<br>
> fine for wayland-protocols itself, but when doing sysroot builds,<br>
> includedir/libdir/etc already get prefixed by the compiler/linker. So<br>
> I wouldn't want anyone to look at wayland-protocols and blindly copy<br>
> the prepend to prefix. Containing it to just variables we know will be<br>
> used and queried directly seems more safe.<br>
<br>
pkg-config already prepends PKG_CONFIG_SYSROOT_DIR to --cflags and<br>
--libs (and properly handle it if you put it in prefix, i.e. no<br>
dublicate), so it is completely safe to have in all variables.<br>
<br>
--<br>
<br>
Quentin “Sardem FF7” Glidic<br>
</blockquote></div>