On 2016-05-22, Scott Kostyshak wrote: > On Sun, May 22, 2016 at 07:22:12PM +0000, Guenter Milde wrote: >> On 2016-05-22, Scott Kostyshak wrote: ... >> >> > >>>>> On 2016-05-20, Scott Kostyshak wrote: >> >> > >> >> > >>>>>> There have been some changes in the luatex engine that will be >> >> > >>>>>> shipped with TeX Live 2016 that cause many of our tests to fail >> >> > >>>>>> (500 something instead of 100 something on TL 2015). The root >> >> > >>>>>> issue is discussed on the LuaTeX mailing list at [1].
>> >> > >>>>>> There is a transitional package available, luatex85, that should >> >> > >>>>>> make our current LuaTeX code work. A reasonable approach for now >> >> > >>>>>> would be to use the package if it is available and to not use it >> >> > >>>>>> if not, keeping the rest of our LuaTeX export code the same. >> >> > ... >> >> > >>>>> We could just add >> >> > ... >> >> > >>>> \IfFileExists{luatex85.sty}{\usepackage{luatex85}}{} >> >> > ... >> >> > >> >> > >> Should we do this only when we use code that we think needs it? Or >> >> > >> should we just add it to all of our LuaTeX exports? >> >> > >> >> > I propose to do this with every LuaTeX export >> After reading the luatex85 documentation I changed my mind. >> Unless LyX uses the offending commands directly, this is a LaTeX issue. LyX uses the offending commands directly (see below), so we have to do something. >> Can you specify which tests fail? > I think the root issue for the failing tests is the following: > 1. Start a new document. > 2. Go to Document > Settings > Page Layout and select A4 format. Save > settings. > 3. Compile with LuaTeX > An undefined control sequence error is given. > Would you prefer the list of tests that fail or is the above MWE good > enough? It is fine. Even without the new luatex, I see that the offending part is the use of: %%%%%%%%%%%%%%%%%%%%%%%%%%%%%% LyX specific LaTeX commands. \pdfpageheight\paperheight \pdfpagewidth\paperwidth in case a non-standard paper size is set. IMO, LyX should write %%%%%%%%%%%%%%%%%%%%%%%%%%%%%% LyX specific LaTeX commands. \IfFileExists{luatex85.sty}{\usepackage{luatex85}}{} \pdfpageheight\paperheight \pdfpagewidth\paperwidth if exporting to LuaTeX and \pdfpage* commands are required. (With a comment in the code to change this to the new syntax next year or so...) Günter