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

Reply via email to