fix for #25 breaks memoize on LuaTeX

[cfr42]

Refs: https://tex.stackexchange.com/q/731942/
Suspect commit: 40f67e2
Related: #25

This is Jasper's example:

\documentclass{article}
\usepackage{memoize}
\usepackage{tikz}
\begin{document}
\tikz \draw[red] (-5,-5) rectangle (5,5);

Hello World!

\tikz \draw[green] (-5,-5) rectangle (5,5);
\end{document}

jasper-test.log

If I amend the example as follows

\documentclass{article}
\usepackage{memoize}
\usepackage{tikz}
\ExplSyntaxOn
\makeatletter
\sys_if_engine_luatex:T
{
  \def\mmz@shipout@unrotate{}%
}
\makeatother
\ExplSyntaxOff
\begin{document}
\tikz \draw[red] (-5,-5) rectangle (5,5);

Hello World!

\tikz \draw[green] (-5,-5) rectangle (5,5);
\end{document}

it compiles fine. [Sorry - I know you don't like expl3.]

Jasper reports that adding \makeatletter\def\mmz@shipout@unrotate{}\makeatother to the preamble results in compilation successful compilation with updated TeX Live, whereas I tested with TeX Live pre the November LaTeX release. (I used a clone of your repo to test as the older memoize doesn't produce the error.)

[sasozivanovic]

Thanks for reporting this, and for the pull request — even if I didn't use it, as I changed the offending \pdfpageattr to \pdfvariable pageattr instead (c862f94). It completely slipped my mind I have to do this for LuaTeX.

Once I have more time on my hands, I need to investigate (as you have suggested on TeX.SE) whether the "unrotate" fix is even needed in LuaTeX.

[cfr42]

Thanks for reporting this, and for the pull request — even if I didn't use it, as I changed the offending \pdfpageattr to \pdfvariable pageattr instead (c862f94). It completely slipped my mind I have to do this for LuaTeX.

It was only in case you didn't have time to do anything else on the grounds it would then be less broken on LuaTeX. I didn't check what happened with rotation - I just copied the XeTeX code on the grounds something which worked in non-rotation cases would be better than something which didn't work at all ....

[sasozivanovic]

I just copied the XeTeX code on the grounds something which worked in non-rotation cases would be better than something which didn't work at all ....

A perfectly sound approach!

It was only in case you didn't have time to do anything else on the grounds it would then be less broken on LuaTeX.

And I indeed don't have the time to explore the LuaTeX situation further right now, or to address the other issue (#33) you have raised recently. Soon(ish) ... and thanks again for all the reports and TeX.SE activity!