<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">2017-05-22 11:38 GMT+02:00 Khaled Hosny <span dir="ltr"><<a href="mailto:khaledhosny@eglug.org" target="_blank">khaledhosny@eglug.org</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>On Mon, May 22, 2017 at 11:28:20AM +0200, Tomáš Chvátal wrote:<br>
> 2017-05-22 10:53 GMT+02:00 Khaled Hosny <<a href="mailto:khaledhosny@eglug.org" target="_blank">khaledhosny@eglug.org</a>>:<br>
><br>
> > On Mon, May 22, 2017 at 10:28:03AM +0200, Miklos Vajna wrote:<br>
> > > Hi Tom,<br>
> > ><br>
> > > On Sat, May 20, 2017 at 11:29:03AM +0200, Tomáš Chvátal <<br>
> > <a href="mailto:tomas.chvatal@gmail.com" target="_blank">tomas.chvatal@gmail.com</a>> wrote:<br>
> > > > I really have no clue what the test is doing or how to fix it. The<br>
> > commits<br>
> > > > introducing the tests are:<br>
> > > > <a href="https://cgit.freedesktop.org/libreoffice/core/commit/vcl/" rel="noreferrer" target="_blank">https://cgit.freedesktop.org/l<wbr>ibreoffice/core/commit/vcl/</a><br>
> > qa/cppunit/complextext.cxx?id=<wbr>62592cf1ebcc8ef381fe5385568434<wbr>2d963e6138<br>
> > > > <a href="https://cgit.freedesktop.org/libreoffice/core/commit/vcl/" rel="noreferrer" target="_blank">https://cgit.freedesktop.org/l<wbr>ibreoffice/core/commit/vcl/</a><br>
> > qa/cppunit/complextext.cxx?id=<wbr>9af45ef67824f3ec6e95c16e7deb00<wbr>5a73532d65<br>
> > ><br>
> > > It can be a good idea to CC Khaled, then. :-)<br>
> ><br>
> > The test is pretty obvious, it lays out the text and the checks the<br>
> > returned character widths.<br>
> ><br>
> > If the test is failing then either 1) your version of HarfBuzz is<br>
> > returning different glyph advances which would almost certainly be a bug<br>
> > in HarfBuzz, or 2) your version of DejaVu Sans have different glyph<br>
> > advances but that is unlikely, or 3) VCL ended up selecting a different<br>
> > font other than the one we requested for whatever reason, or 4)<br>
> > something else fishy is going on in VCL.<br>
> ><br>
><br>
> Oky Is there way to dump what dejavu sans it found?<br>
<br>
</span>I think no, may be there are some SAL_LOG if you are building with<br>
logging enabled but I’m not sure.<br>
<span><br>
> We are using new harfbuzz:<br>
><br>
> [   22s] [326/807] cumulate libharfbuzz0-1.4.6-1.1<br>
><br>
> And with the dejavu sans I get the feeling they are not present in the<br>
> buildroot at the moment, only liberation sans and google carlito fonts. I<br>
> will try to add them and see if it changes anything (results to be seen in<br>
> couple of hours).<br>
<br>
</span>The test (naïvely?) assumes DejaVu Sans is always present since it is<br>
one of the bundled fonts, if this is not true since the test needs to be<br>
conditionally enabled somehow.<br>
<span><br>
> Anyway I think the test should first font-config verify it got at least the<br>
> font it requiires.<br>
<br>
</span>VCL does not have a public API for that AFAIK.<br><br></blockquote><div>Yep the problem was caused by missing Dejavu Sans font.</div><div><br>Adding it to buildroot fixed the issue.</div><div><br></div><div>Now I have another failing test for debugging :)</div><div><br>Cheers</div><div><br>Tom </div></div><br></div></div>