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 when loading RTL languages after LaTeX update #647

Closed
Udi-Fogiel opened this issue Jun 4, 2024 · 3 comments
Closed

Error when loading RTL languages after LaTeX update #647

Udi-Fogiel opened this issue Jun 4, 2024 · 3 comments

Comments

@Udi-Fogiel
Copy link
Collaborator

\documentclass{article}
\usepackage{polyglossia}
\setotherlanguage{kurdish}

\stop

now fails on master, after latex3/latex2e@0256b60.
This is related to the location we use \RequireBidi. It is used in the key direction in polyglossia setup, and in kurdish the different variants set the direction using \SetLanguageKeys, and so \RequireBidi is issued again. It happens inside \__xpg_register_language:nn in a group where the options are set again to retrieve the correct babel name etc.,

Note that in the branch I'm currently working on, this problem does not exist, as there I don't spam RequireBidi, but I also don't warn when an RTL language is used, but bidi was not loaded (I'm still figuring out what best method would be to do that).

In any case, I think for now the best thing to do is to make \RequireBidi a no-op after one use.

@Udi-Fogiel
Copy link
Collaborator Author

BTW, we should probably test ourselves if a gloss is being loaded in a group, just to be safe, but I'll leave that for after the release.

@jspitz
Copy link
Collaborator

jspitz commented Jul 12, 2024

So this is resolved, right? If so, mark fixedindev, please

@Udi-Fogiel
Copy link
Collaborator Author

Yes, bug is only present in the dev, not in 2.1, so I'll simply close it....

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