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

Patch \newcatcodetable before loading minim-alloc #154

Merged
merged 1 commit into from
Nov 2, 2023

Conversation

vlasakm
Copy link
Contributor

@vlasakm vlasakm commented Nov 2, 2023

In the latest release minim actually uses \newcatcodetable in minim-alloc.tex. This means that it still uses the standard OpTeX definition (which doesn't initialize the catcode table), but it expects the initialization.

We solve this by doing the patch to \newcatcodetable before loading minim-alloc.tex. It should have probably been done this way right from the beginning, it just didn't manifest.

In the latest release minim actually uses `\newcatcodetable` in
minim-alloc.tex. This means that it still uses the standard OpTeX definition
(which doesn't initialize the catcode table), but it expects the
initialization.

We solve this by doing the patch to `\newcatcodetable` _before_ loading
minim-alloc.tex. It should have probably been done this way right from
the beginning, it just didn't manifest.
@vlasakm
Copy link
Contributor Author

vlasakm commented Nov 2, 2023

The "problematic" minim version was released two weeks ago, so this hasn't been problem for that long.

I found it while preparing a new release of mmtex with my optex-minim test file: https://github.com/vlasakm/optex-minim/blob/master/examples/optex-minim-pgf-test.tex (in case you want to test yourself).

vlasakm added a commit to vlasakm/mmtex that referenced this pull request Nov 2, 2023
@olsak olsak merged commit 3e55213 into olsak:master Nov 2, 2023
@vlasakm vlasakm deleted the minim-newcatcodetable branch November 2, 2023 18:58
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

Successfully merging this pull request may close these issues.

2 participants