Hey, On Mon, May 22, 2017 at 4:22 PM, Rene Engelhard <r...@debian.org> wrote:
> On Mon, May 22, 2017 at 12:55:56PM +0200, Khaled Hosny wrote: > > On Mon, May 22, 2017 at 12:36:26PM +0200, Rene Engelhard wrote: > > > Hi, > > > > > > On Mon, May 22, 2017 at 11:38:43AM +0200, Khaled Hosny wrote: > > > > The test (naďvely?) assumes DejaVu Sans is always present since it is > > > > one of the bundled fonts, if this is not true since the test needs > to be > > > > conditionally enabled somehow. > > > > > > Maybe check it like the other fonts? If --without-fonts is specified > that is. > > > See https://cgit.freedesktop.org/libreoffice/core/tree/ > configure.ac#n11251 ff. > > > > That would do it I think. > > This patch should do it. (Just tested configure for now) > > Regards, > All tests in master should pass a --with-fonts=no build. See e.g. https://ci.libreoffice.org/job/lo_tb_random_config_linux/521/consoleFull I think I disabled all tests that depend on the additional fonts for the random config CI build. Regards, Markus
_______________________________________________ LibreOffice mailing list LibreOffice@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice
