On 17/12/2019 16:20, Ashod Nakashian wrote:
Fair enough. I think we can still keep your patch, but restore the defines (i.e. keep them disabled) and I'll a comment/fixme (after you merge) that we should restore these tests to work with PDFium etc.
Fair enough. Pushed as <https://gerrit.libreoffice.org/plugins/gitiles/core/+/78761f9b0177b0ae8ccfc5323a530fe4404b9637%5E%21> "Make disabled PDF import tests compile again".
(By the way, I'm not interested in the tests themselves at all, just came across them in the context of <https://gerrit.libreoffice.org/plugins/gitiles/core/+/ced9702c9fcba0c702f0b910c3f1f5dae69a4f5a%5E!> "SdImportTest::testPDFImportShared fails for --disable-pdfium".)
_______________________________________________ LibreOffice mailing list LibreOffice@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/libreoffice