<div dir="ltr"><div>Hi Mauro,</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Aug 31, 2023 at 2:28 AM Mauro Carvalho Chehab <<a href="mailto:mauro.chehab@linux.intel.com">mauro.chehab@linux.intel.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">The testlist files are used for two purposes:<br>
<br>
1. for documentation build;<br>
2. for CI automation to run the tests.<br>
<br>
While right now Intel CI doesn't depend on the produced testlists,<br>
this may change in the future. So, instead of making the testlist<br>
build dependent of testplan, the better seems to verify if the<br>
build is not a cross-compilation, e. g. with a patch similar to<br>
the enclosed one.<br></blockquote><div><br></div><div>In my experience it is better to separate build phases from phases that use</div><div>these binaries and execute them. It makes steps more modular and clearer</div><div>where any failures come from. Having test lists sounds like a good idea</div><div>and beneficial to other architectures.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Could you please check if this will address the issue? I don't have<br>
a cross-compilation environment handy to test it, so it would be<br>
easier if you could test it for me.<br></blockquote><div><br></div><div>The enclosed patch "[PATCH i-g-t] tests/meson.build: only generate testlists </div><div>on native builds" works for our build system. If you want to stick with it,</div><div>could you please land it in the upstream tree? Otherwise I could try a different</div><div>patch as well.</div><div><br></div><div>Thank you very much,</div><div><br></div><div>Ilja.</div></div></div>