<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">2017-05-22 10:53 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: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 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 <<a href="mailto:tomas.chvatal@gmail.com">tomas.chvatal@gmail.com</a>> wrote:<br>
> > I really have no clue what the test is doing or how to fix it. The commits<br>
> > introducing the tests are:<br>
> > <a href="https://cgit.freedesktop.org/libreoffice/core/commit/vcl/qa/cppunit/complextext.cxx?id=62592cf1ebcc8ef381fe53855684342d963e6138" rel="noreferrer" target="_blank">https://cgit.freedesktop.org/<wbr>libreoffice/core/commit/vcl/<wbr>qa/cppunit/complextext.cxx?id=<wbr>62592cf1ebcc8ef381fe5385568434<wbr>2d963e6138</a><br>
> > <a href="https://cgit.freedesktop.org/libreoffice/core/commit/vcl/qa/cppunit/complextext.cxx?id=9af45ef67824f3ec6e95c16e7deb005a73532d65" rel="noreferrer" target="_blank">https://cgit.freedesktop.org/<wbr>libreoffice/core/commit/vcl/<wbr>qa/cppunit/complextext.cxx?id=<wbr>9af45ef67824f3ec6e95c16e7deb00<wbr>5a73532d65</a><br>
><br>
> It can be a good idea to CC Khaled, then. :-)<br>
<br>
</span>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></blockquote><div><br></div><div>Oky Is there way to dump what dejavu sans it found?</div><div><br></div><div>We are using new harfbuzz:</div><div><pre style="color:rgb(0,0,0);word-wrap:break-word;white-space:pre-wrap">[   22s] [326/807] cumulate libharfbuzz0-1.4.6-1.1
</pre></div><div>And with the dejavu sans I get the feeling they are not present in the buildroot at the moment, only liberation sans and google carlito fonts. I will try to add them and see if it changes anything (results to be seen in couple of hours).</div><div><br></div><div>Anyway I think the test should first font-config verify it got at least the font it requiires.</div><div><br></div><div>Cheers</div><div><br></div><div>Tom</div><div> </div></div></div></div>