<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Jul 21, 2014 at 6:00 PM, Jacob Kroon <span dir="ltr"><<a href="mailto:jacob.kroon@gmail.com" target="_blank">jacob.kroon@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">Hi Ross,<div class=""><br><br>On Mon, Jul 21, 2014 at 2:49 PM, Burton, Ross <span dir="ltr"><<a href="mailto:ross.burton@intel.com" target="_blank">ross.burton@intel.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div>On 21 July 2014 13:39, Jacob Kroon <<a href="mailto:jacob.kroon@mikrodidakt.se" target="_blank">jacob.kroon@mikrodidakt.se</a>> wrote:<br>
> gdk-pixbuf-native uses png and jpeg loaders. When updating the<br>
> loader-cache, these needs to be already populated in the sysroot.<br>
><br>
> Although png was present, the jpeg dependency was missing, so add it.<br>
<br>
</div>But those dependencies are dependent on a PACKAGECONFIG setting, so<br>
you'll potentially be forcing a build of libjpeg purely to satisfy a<br>
fetch from sstate of a JPEG-disabled gdk-pixbuf. Also, libtiff and<br>
libjasper are missing.<br>
<br>
If we can't change the sysroot creation order so that the order is<br>
zlib, libpng, libjpeg, and then gdk-pixbuf then I think this needs to<br>
be pulled out to a function that can do neater logic, such as iterate<br>
through PACKAGECONFIG and add the required dependencies (for each<br>
enabled option, turn the build depends into<br>
%s-native:do_populate_sysroot_setscene).<br>
<span><font color="#888888"><br></font></span></blockquote><br></div></div><div class="gmail_quote">What about something like the patch below. It looks at the package's DEPENDS,<br>and for native packages just adds them to <br>
<br>do_populate_sysroot[depends]<br>do_populate_sysroot_setscene[depends]<br><br></div></div></div></blockquote><div><br></div><div>Thinking some more about it I don't see this working either, since it only guarantees 1-level-of-dependencies to be populated, right ?<br>
</div><div>Wouldn't the right thing here be to somehow postpone updating the loader-caches until the entire sysroot is populated ?<br></div></div></div></div>