Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error if polyglossia is used with LuaLaTeX, a font with a name containing a space and a 'book-like' class #667

Closed
cfr42 opened this issue Nov 6, 2024 · 4 comments

Comments

@cfr42
Copy link

cfr42 commented Nov 6, 2024

The following example fails if compiled with lualatex. It seems any font with a space in the name will trigger the problem, but only if the class is 'book-like'. So e.g. article is fine.

\documentclass{book}
\usepackage{polyglossia}
\setmainlanguage[variant=british]{english}
\setmainfont{Minion Pro}
\begin{document}
test
\end{document}

The example works if:

  • fontspec is loaded, but not polyglossia,
  • article is substituted for book,
  • default fonts are used,
  • XeTeX is used rather than LuaTeX,
  • a font without a space in the name such as Crimson is substituted for Minion Pro.

ref.: https://tex.stackexchange.com/q/730204/ [but there's some sort of fontconfig weirdness there, which isn't relevant to LuaTeX anyhow and XeTeX has no problem].

The log (attached) seems to show fontspec sets the family up correctly, but LaTeX ends up looking for MinionPro rather than Minion Pro. So some interaction with polyglossia causes the space to be lost.

prawf-2.log

@cfr42
Copy link
Author

cfr42 commented Nov 6, 2024

Actually, probably a Harfbuzz issue, so polyglossia only indirectly responsible.

@Udi-Fogiel
Copy link
Collaborator

I think you already encountered that issue :)
#650

Is there anything new?

@Udi-Fogiel Udi-Fogiel closed this as not planned Won't fix, can't repro, duplicate, stale Nov 6, 2024
@cfr42
Copy link
Author

cfr42 commented Nov 6, 2024

Is there anything new?

No. I just didn't remember polyglossia changed the renderer until David Carlisle reminded me. I would have deleted this, but that doesn't seem to be possible. Apologies for the noise.

@Udi-Fogiel
Copy link
Collaborator

Udi-Fogiel commented Nov 7, 2024

@cfr42 All good. I wish luaotfload would release a fix already...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants