<div dir="ltr"><div dir="ltr"><div dir="ltr">On Fri, May 3, 2019 at 5:32 PM Petr Štetiar <<a href="mailto:ynezz@true.cz" target="_blank">ynezz@true.cz</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Jeffery To <<a href="mailto:jeffery.to@gmail.com" target="_blank">jeffery.to@gmail.com</a>> [2019-05-03 00:47:06]:<br>
<br>
> The default zlib.pc file generated by cmake contains absolute paths.<br>
> This patches the file to use relative paths (relative to ${prefix} and<br>
> ${exec_prefix}).<br>
<br>
I can see that from the diff already, but I miss the information, why do you<br>
need this change. Could this perhaps be upstreamed first as well?<br>
<br>
-- ynezz<br></blockquote><div><br></div><div>The buildroot pkg-config (in staging_dir/host/bin) overrides the prefix and exec_prefix variables in *.pc files, to supply the correct (buildroot) paths for callers. If other variables are not defined relative to prefix and exec_prefix, then the returned values will be incorrect.<br></div><div><br></div><div>A similar change was merged for xz[1] in the packages repo.</div><div><br></div><div>I'm not sure upstream will want this change, since what is there currently isn't "wrong"; it only affects our cross-compilation.<br></div><div><br></div><div><br></div><div>1. <a href="https://github.com/openwrt/packages/pull/8880" target="_blank">https://github.com/openwrt/packages/pull/8880</a></div><div><br></div></div></div></div>