On Sun, 23 Mar 2025 at 12:32, Scott Kostyshak <skost...@lyx.org> wrote:
> On Sat, Mar 22, 2025 at 11:29:45PM +0100, Thibaut Cuvelier wrote: > > On Mon, 10 Feb 2025 at 15:28, Scott Kostyshak <skost...@lyx.org> wrote: > > > > > On Mon, Feb 10, 2025 at 03:11:45PM +0100, Thibaut Cuvelier wrote: > > > > On Mon, 10 Feb 2025 at 11:17, Scott Kostyshak <skost...@lyx.org> > wrote: > > > > > > > > > On Sun, Feb 09, 2025 at 12:37:18AM +0100, Scott Kostyshak wrote: > > > > > > The following ctests are failing for me: > > > > > > > > > > > > > > > > > ... > > > > > > > > > > Thibaut, do you think these can be related to your recent work? > Can you > > > > > reproduce the test failures? > > > > > > > > > > > > I can reproduce locally at 7606da5e, bbb79522, and f70409b3. I > believe > > > it's > > > > one commit near f70409b3 that created that problem, > > > > > > Thanks a lot for confirming you can reproduce and taking a look! > > > > > > I could confirm that f70409b3 is indeed the culprit. A symptom of having > > \text converted to garbled <mtext> is not due to a change in > InsetMathFont, > > but rather in MathStream (there is no related change near InsetMathFont). > > That leaves off 7.42% of this commit 😅. More precisely, it's in lines > > 679-705 of MathStream.cpp (as of f70409b3) or code called from there. > > > > In the end, the bug (or, at least, a bug) is when calling the docstring > > constructor to have a string of one character: I inverted the two > > arguments. Quite silly! > > > > I have pushed two patches that solve this problem > > (eab73de5f134930c84bb75d78c9aa8b91558b418 and > > 2e12882ec6fbf37c0480e98099f01fa355f45bc0). > > Thanks for the fixes! I've unignored the xhtml tests at 271b9c2d. Now > you have to do a fresh build (or at least rerun your 'cmake' command > from the build directory). Your commit fixed a lot of tests. Now only > the following ones are failing for me: > > 1989 - export/doc/Math_xhtml (Failed) > 2051 - export/doc/UserGuide_xhtml (Failed) > 2100 - export/doc/ar/UserGuide_xhtml (Failed) > 2319 - export/doc/de/Math_xhtml (Failed) > 2376 - export/doc/de/UserGuide_xhtml (Failed) > 2547 - export/doc/es/Math_xhtml (Failed) > 2604 - export/doc/es/UserGuide_xhtml (Failed) > 2794 - export/doc/fr/Math_xhtml (Failed) > 2851 - export/doc/fr/UserGuide_xhtml (Failed) > 3199 - export/doc/ja/Math_xhtml (Failed) > 3244 - export/doc/ja/UserGuide_xhtml (Failed) > 3506 - export/doc/ru/Math_xhtml (Failed) > 3563 - export/doc/ru/UserGuide_xhtml (Failed) > 4187 - export/examples/Graphics_and_Insets/XY-Pic_xhtml (Failed) > 5637 - export/examples/fr/Graphics_and_Insets/XY-Pic_xhtml (Failed) > 6515 - export/examples/ru/Graphics_and_Insets/XY-Pic_xhtml (Failed) I have fixed a few more bugs, now, Math should pass (and possibly others). In particular, I fixed a few issues that were present in LyX 2.4. I think that 922c8107 should make it into 2.4.
-- lyx-devel mailing list lyx-devel@lists.lyx.org https://lists.lyx.org/mailman/listinfo/lyx-devel