Hey,
On Mon, May 22, 2017 at 4:22 PM, Rene Engelhard <rene@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
Context
Privacy Policy |
Impressum (Legal Info) |
Copyright information: Unless otherwise specified, all text and images
on this website are licensed under the
Creative Commons Attribution-Share Alike 3.0 License.
This does not include the source code of LibreOffice, which is
licensed under the Mozilla Public License (
MPLv2).
"LibreOffice" and "The Document Foundation" are
registered trademarks of their corresponding registered owners or are
in actual use as trademarks in one or more countries. Their respective
logos and icons are also subject to international copyright laws. Use
thereof is explained in our
trademark policy.