Re: [LyX/master] Revert "Fix bug #10263"
Am Sun, 13 Jun 2021 11:42:59 -0400 schrieb Scott Kostyshak : > On Sat, May 29, 2021 at 01:47:41PM -0400, Scott Kostyshak wrote: > > On Fri, May 28, 2021 at 10:48:51AM -0400, Scott Kostyshak wrote: > > > On Tue, Apr 13, 2021 at 10:13:16AM +0200, Jean-Marc Lasgouttes wrote: > > > > commit 441c6a93590698c3c57982c8b80179d6809fb106 > > > > Author: Jean-Marc Lasgouttes > > > > Date: Mon Apr 12 20:44:58 2021 +0200 > > > > > > > > Revert "Fix bug #10263" > > > > > > > > A series of commits, culminating at 812ff7de, pushed a few days > > > > later, > > > > fixes the bug at its root. This one is not needed anymore to fix > > > > > > > > This reverts commit 001f5a47861f04c985323677dfd17ef15b8c33a7 > > > > --- > > > > src/insets/InsetFoot.h |4 > > > > 1 files changed, 0 insertions(+), 4 deletions(-) > > > > > > > > diff --git a/src/insets/InsetFoot.h b/src/insets/InsetFoot.h > > > > index 1e4b0d2..0c3f65a 100644 > > > > --- a/src/insets/InsetFoot.h > > > > +++ b/src/insets/InsetFoot.h > > > > @@ -45,10 +45,6 @@ private: > > > > /// > > > > Inset * clone() const override { return new InsetFoot(*this); } > > > > /// > > > > - bool inheritFont() const override { return true; } > > > > - /// > > > > - bool allowMultiPar() const override { return true; } > > > > - /// > > > > docstring custom_label_; > > > > /// > > > > bool intitle_; > > > > -- > > > > > > The following ctest started failing with this commit: > > > > > > export/doc/ja/Tutorial_pdf5_systemF > > > > > > The diff between the "bad" (current master) and "good" (current master > > > with this commit reverted) is as follows: > > > > > > $ diff good.tex bad.tex > > > 61c61 > > > < \author{\LyX > > > プロジェクトチーム\thanks{なにかコメントや間違いの修正がある場合には,\protect\LyX > > > 文書化メーリングリスト(\protect\href{mailto:lyx-d...@lists.lyx.org}{lyx-d...@lists.lyx.org})までお知らせ下さい.この文書の翻訳は,当初人見光太郎氏が行った貢献に基づいています.}} > > > --- > > > > \author{\LyX プロジェクトチーム\thanks{{\normalsize > > > > なにかコメントや間違いの修正がある場合には,\protect\LyX > > > > 文書化メーリングリスト(\protect\href{mailto:lyx-d...@lists.lyx.org}{lyx-d...@lists.lyx.org})までお知らせ下さい.この文書の翻訳は,当初人見光太郎氏が行った貢献に基づいています.}}} > > > > > > > $ > > > > > > I have no idea why the {\normalsize ...} would cause the failure. Does > > > anyone know the problem? > > > > > > Note that we are already aware of several other Japanese documents that > > > do not compile with non-TeX fonts: > > > > > > export/doc/ja/(Additional|LaTeXConfig|Math|UserGuide).*_systemF > > > > > > so it would not be surprising if we need to add Tutorial.lyx to the list. > > > > > > By the way, if this report does not lead to a fix in LyX or a bug > > > report, we should log this wasted time in > > > development/autotests/ctests-costs-benefits.txt. > > > > I think the French Powerdot example might also have a footnote where > > this commit changes behavior. If I remove the footnote, compilation > > succeeds. I did not do a bisect on that document but it feels like a > > similar issue. > > Does anyone know what the core issue is? Here is the LaTeX code causing > the problem after this commit: > > \thanks{{\normalsize Traduction française Jean-Pierre > Chrétien, }{\normalsize\texttt{}}}{\normalsize}, > novembre > 2009.}}} > > Do we need to adapt lyx2lyx perhaps? Or is this a LyX LaTeX export > issue? > > Scott This compiles also if changing the font inside the footnote (Teletype->default) Kornel pgp6EgfWHUMRv.pgp Description: Digitale Signatur von OpenPGP -- lyx-devel mailing list lyx-devel@lists.lyx.org http://lists.lyx.org/mailman/listinfo/lyx-devel
Re: [LyX/master] Revert "Fix bug #10263"
On Sat, May 29, 2021 at 01:47:41PM -0400, Scott Kostyshak wrote: > On Fri, May 28, 2021 at 10:48:51AM -0400, Scott Kostyshak wrote: > > On Tue, Apr 13, 2021 at 10:13:16AM +0200, Jean-Marc Lasgouttes wrote: > > > commit 441c6a93590698c3c57982c8b80179d6809fb106 > > > Author: Jean-Marc Lasgouttes > > > Date: Mon Apr 12 20:44:58 2021 +0200 > > > > > > Revert "Fix bug #10263" > > > > > > A series of commits, culminating at 812ff7de, pushed a few days later, > > > fixes the bug at its root. This one is not needed anymore to fix > > > > > > This reverts commit 001f5a47861f04c985323677dfd17ef15b8c33a7 > > > --- > > > src/insets/InsetFoot.h |4 > > > 1 files changed, 0 insertions(+), 4 deletions(-) > > > > > > diff --git a/src/insets/InsetFoot.h b/src/insets/InsetFoot.h > > > index 1e4b0d2..0c3f65a 100644 > > > --- a/src/insets/InsetFoot.h > > > +++ b/src/insets/InsetFoot.h > > > @@ -45,10 +45,6 @@ private: > > > /// > > > Inset * clone() const override { return new InsetFoot(*this); } > > > /// > > > - bool inheritFont() const override { return true; } > > > - /// > > > - bool allowMultiPar() const override { return true; } > > > - /// > > > docstring custom_label_; > > > /// > > > bool intitle_; > > > -- > > > > The following ctest started failing with this commit: > > > > export/doc/ja/Tutorial_pdf5_systemF > > > > The diff between the "bad" (current master) and "good" (current master > > with this commit reverted) is as follows: > > > > $ diff good.tex bad.tex > > 61c61 > > < \author{\LyX プロジェクトチーム\thanks{なにかコメントや間違いの修正がある場合には,\protect\LyX > > 文書化メーリングリスト(\protect\href{mailto:lyx-d...@lists.lyx.org}{lyx-d...@lists.lyx.org})までお知らせ下さい.この文書の翻訳は,当初人見光太郎氏が行った貢献に基づいています.}} > > --- > > > \author{\LyX プロジェクトチーム\thanks{{\normalsize > > > なにかコメントや間違いの修正がある場合には,\protect\LyX > > > 文書化メーリングリスト(\protect\href{mailto:lyx-d...@lists.lyx.org}{lyx-d...@lists.lyx.org})までお知らせ下さい.この文書の翻訳は,当初人見光太郎氏が行った貢献に基づいています.}}} > > $ > > > > I have no idea why the {\normalsize ...} would cause the failure. Does > > anyone know the problem? > > > > Note that we are already aware of several other Japanese documents that > > do not compile with non-TeX fonts: > > > > export/doc/ja/(Additional|LaTeXConfig|Math|UserGuide).*_systemF > > > > so it would not be surprising if we need to add Tutorial.lyx to the list. > > > > By the way, if this report does not lead to a fix in LyX or a bug > > report, we should log this wasted time in > > development/autotests/ctests-costs-benefits.txt. > > I think the French Powerdot example might also have a footnote where > this commit changes behavior. If I remove the footnote, compilation > succeeds. I did not do a bisect on that document but it feels like a > similar issue. Does anyone know what the core issue is? Here is the LaTeX code causing the problem after this commit: \thanks{{\normalsize Traduction française Jean-Pierre Chrétien, }{\normalsize\texttt{}}}{\normalsize}, novembre 2009.}}} Do we need to adapt lyx2lyx perhaps? Or is this a LyX LaTeX export issue? Scott signature.asc Description: PGP signature -- lyx-devel mailing list lyx-devel@lists.lyx.org http://lists.lyx.org/mailman/listinfo/lyx-devel
Re: Anyone on TeX Live 2021 pretest ? (to confirm a regression for ja LilyPond)
On Mon, May 31, 2021 at 02:19:26PM -0400, Scott Kostyshak wrote: > On Mon, May 31, 2021 at 04:57:49PM +0200, Pavel Sanda wrote: > > On Sat, May 29, 2021 at 01:45:24PM -0400, Scott Kostyshak wrote: > > > I think there is a related issue: Some exports that use ps2pdf now seem > > > to need the flag "-dALLOWPSTRANSPARENCY" on TL21. Adding the following > > > line to my preferences file fixes some tests: > > > > > > \converter "ps" "pdf" "ps2pdf -dALLOWPSTRANSPARENCY $$i $$o" > > > "hyperref-driver=dvips > > > > Huh, this would be painful if true in general. > > Indeed it does seem annoying, but is only relevant for a few of our documents. > > > What exact version of ghostscript do you have? > > $ gs --version > 9.53.3 If interested, see also: https://www.lyx.org/trac/ticket/12303 Scott signature.asc Description: PGP signature -- lyx-devel mailing list lyx-devel@lists.lyx.org http://lists.lyx.org/mailman/listinfo/lyx-devel