Re: test failures with new fonts-crosextra-carlito

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

Am 26.06.23 um 09:06 schrieb Mike Kaganski:
On 26.06.2023 8:15, Rene Engelhard wrote:
Hi,

Am 25.06.23 um 13:29 schrieb Rene Engelhard:
my beta1 (didn't try 7.5 yet) builds fail with a new verson of the fonts-crosextra-carlito fonts (used for Calibri). LibreOffice itself ships

My 7.5.4 builds needs the following in addition to  the already posted (where applicable):

rene@frodo:~/Debian/Pakete/LibreOffice/libreoffice/libreoffice-7.5.4.2$ diff -u sw/qa/extras/ooxmlexport/ooxmlexport14.cxx-old sw/qa/extras/ooxmlexport/ooxmlexport14.cxx --- sw/qa/extras/ooxmlexport/ooxmlexport14.cxx-old    2023-06-26 06:30:09.119195022 +0200 +++ sw/qa/extras/ooxmlexport/ooxmlexport14.cxx    2023-06-26 06:30:18.891263759 +0200
@@ -1346,8 +1346,7 @@
  #if !defined(MACOSX)
  DECLARE_OOXMLEXPORT_TEST(testTdf146346, "tdf146346.docx")
  {
-    // This was 2 (by bad docDefault vertical margins around tables in footnotes)
-    CPPUNIT_ASSERT_EQUAL(1, getPages());
+    CPPUNIT_ASSERT_EQUAL(2, getPages());
  }
  #endif

I take that one back, it seems to be flaky. I get 2, the apply the patch, then somehow get 1, revert the patch, get 2 again etc. I'll disable it for now.[...]

> 2. A problem in unit test, claiming one page in master, when actually it's two.

And the test seems to be flaky...


Regards,


Rene




[Index of Archives]     [LARTC]     [Bugtraq]     [Yosemite Forum]     [Photo]

  Powered by Linux