<div dir="ltr">The ability to disabling everything except the library itself seems like a good idea.<div><br></div><div>Make sure "make test" fails if tests are disabled.</div><div><br></div><div>I do think it would be nice to actually patch the tests so they compile on these platforms.</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jan 20, 2016 at 1:13 AM, Thomas Petazzoni <span dir="ltr"><<a href="mailto:thomas.petazzoni@free-electrons.com" target="_blank">thomas.petazzoni@free-electrons.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello,<br>
<span class=""><br>
On Wed, 20 Jan 2016 09:27:46 +0200, Siarhei Siamashka wrote:<br>
<br>
> Thanks for this patch. Though if building (and using) pixman on<br>
> such platforms is wanted, then a much better solution would be to<br>
> update the problematic tests and make them compile. Skipping some<br>
> sub-tests is better than having no tests at all. I also remember<br>
> your patch for FE_DIVBYZERO from a few months ago:<br>
><br>
>     <a href="http://lists.freedesktop.org/archives/pixman/2015-September/004019.html" rel="noreferrer" target="_blank">http://lists.freedesktop.org/archives/pixman/2015-September/004019.html</a><br>
><br>
> Is it still the same Microblaze or Nios2 architecture that is causing<br>
> problems for you?<br>
<br>
</span>Yes, it is. Other architectures might be affected because the <fenv.h><br>
implementation in uClibc is not complete for all architectures.<br>
<span class=""><br>
> While adding new configure options just adds functionality and<br>
> preserves the existing behavior, I don't feel very happy about<br>
> the fact that this provides an easy way to ignore problems instead<br>
> of fixing them. It would be really great is somebody tried to run<br>
> the pixman test suite ("make check") on these architectures at<br>
> least once.<br>
><br>
> Encountering compiler bugs is unfortunately a regular occurrence<br>
> for pixman. For example, not so long ago, GCC 4.9 miscompiled<br>
> pixman on ARM (fortunately, the broken code was in the test suite<br>
> itself and not in the pixman library):<br>
><br>
>     <a href="https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64172" rel="noreferrer" target="_blank">https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64172</a><br>
><br>
> And even just a few days ago, pixman was one of the victims during<br>
> a GCC 6 snapshot test (an easy to notice ICE during a distro test<br>
> rebuild):<br>
><br>
>     <a href="https://gcc.gnu.org/ml/gcc/2016-01/msg00101.html" rel="noreferrer" target="_blank">https://gcc.gnu.org/ml/gcc/2016-01/msg00101.html</a><br>
>     <a href="https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66856" rel="noreferrer" target="_blank">https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66856</a><br>
><br>
> What I'm trying to say is that there had been many compiler bugs<br>
> affecting pixman during the last few years. Now you are dealing<br>
> with uncommon architectures, and the compilers there are probably<br>
> even less mature than GCC on x86 / arm / powerpc.<br>
<br>
</span>I agree, but those options also allow to skip building things that<br>
won't be used, even if they actually build properly. On ARM, x86,<br>
PowerPC and other "mainstream" architectures, the demos and tests build<br>
fine, but they are not used at all by Buildroot, so it's just spending<br>
time building things that aren't necessary.<br>
<br>
So even if those tests and demos were building for all architectures,<br>
it would still be useful to have a way to *not* build them.<br>
<br>
But I'll have a look at re-enabling the building of the pixman tests in<br>
Buildroot. Now that the FE_* things are disabled in the tests when not<br>
available (after commit 4297e9058d252cac653723fe0b1bee559fbac3a4).<br>
<br>
Thanks!<br>
<span class="HOEnZb"><font color="#888888"><br>
Thomas<br>
--<br>
Thomas Petazzoni, CTO, Free Electrons<br>
Embedded Linux, Kernel and Android engineering<br>
<a href="http://free-electrons.com" rel="noreferrer" target="_blank">http://free-electrons.com</a><br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
Pixman mailing list<br>
<a href="mailto:Pixman@lists.freedesktop.org">Pixman@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/pixman" rel="noreferrer" target="_blank">http://lists.freedesktop.org/mailman/listinfo/pixman</a><br>
</div></div></blockquote></div><br></div></div>