<div dir="ltr">Hey,<br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, May 22, 2017 at 4:22 PM, Rene Engelhard <span dir="ltr"><<a href="mailto:rene@debian.org" target="_blank">rene@debian.org</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"><span class="gmail-">On Mon, May 22, 2017 at 12:55:56PM +0200, Khaled Hosny wrote:<br>
> On Mon, May 22, 2017 at 12:36:26PM +0200, Rene Engelhard wrote:<br>
> > Hi,<br>
> ><br>
> > On Mon, May 22, 2017 at 11:38:43AM +0200, Khaled Hosny wrote:<br>
</span>> > > The test (naďvely?) assumes DejaVu Sans is always present since it is<br>
<span class="gmail-">> > > one of the bundled fonts, if this is not true since the test needs to be<br>
> > > conditionally enabled somehow.<br>
> ><br>
> > Maybe check it like the other fonts? If --without-fonts is specified that is.<br>
> > See <a href="https://cgit.freedesktop.org/libreoffice/core/tree/configure.ac#n11251" rel="noreferrer" target="_blank">https://cgit.freedesktop.org/<wbr>libreoffice/core/tree/<wbr>configure.ac#n11251</a> ff.<br>
><br>
> That would do it I think.<br>
<br>
</span>This patch should do it. (Just tested configure for now)<br>
<br>
Regards,<br></blockquote></div><br><br></div><div class="gmail_extra">All tests in master should pass a --with-fonts=no build. See e.g. <a href="https://ci.libreoffice.org/job/lo_tb_random_config_linux/521/consoleFull">https://ci.libreoffice.org/job/lo_tb_random_config_linux/521/consoleFull</a><br><br></div><div class="gmail_extra">I think I disabled all tests that depend on the additional fonts for the random config CI build.<br><br></div><div class="gmail_extra">Regards,<br></div><div class="gmail_extra">Markus<br></div></div>