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

acro broken with latest release of l3kernel #268

Open
dbitouze opened this issue Feb 15, 2024 · 5 comments
Open

acro broken with latest release of l3kernel #268

dbitouze opened this issue Feb 15, 2024 · 5 comments

Comments

@dbitouze
Copy link
Contributor

dbitouze commented Feb 15, 2024

After last l3kernel update, the following MCE:

\documentclass{article}
\usepackage{acro}
\begin{document}
Foo.
\end{document}

fails to compile:

! Undefined control sequence.
\l__acro_tmpa_prop

Other packages are affected as well and the reason is explained here. Hence, adding:

\prop_new:N \l__acro_tmpa_prop

in acro.sty is enough to fix this issue.

@bdtc
Copy link

bdtc commented Feb 17, 2024

Similar for chemmacros, chemformula, chemnum.

josephwright added a commit to josephwright/acro that referenced this issue Feb 19, 2024
@josephwright
Copy link
Contributor

@bdtc I could see the issue in chemnum and have logged a PR (cgnieder/chemnum#23), and with chemformula can see an unrelated issue with loading l3keys2e (cgnieder/chemformula#21). But I don't see the prop issue with either chemformula or chemmacros: do you have examples available? I'll sort PRs if I know what needs fixing.

@josephwright
Copy link
Contributor

I've put in a PR for this issue and mailed @cgnieder to check on status.

@bdtc
Copy link

bdtc commented Feb 19, 2024

For chemmacros reactant:

\documentclass{article}
\usepackage{chemmacros}
\begin{document}
\DeclareChemReactant{dichloropentane}{name={\iupac{2,4-di|chloro|pentane}}}
\reactant{dichloropentane}
\end{document}

For chemformula:

\documentclass{article}
\usepackage{chemformula}
\begin{document}
test
\end{document}

gives:

! Undefined control sequence.
l.481 \ProcessKeysPackageOptions

which is almost certainly what you also found.

Sorry I was not more specific, it was late and I was processing tests in bulk.

@josephwright
Copy link
Contributor

For chemmacros reactant:

\documentclass{article}
\usepackage{chemmacros}
\begin{document}
\DeclareChemReactant{dichloropentane}{name={\iupac{2,4-di|chloro|pentane}}}
\reactant{dichloropentane}
\end{document}

The's the issue in chemnum.

For chemformula:

\documentclass{article}
\usepackage{chemformula}
\begin{document}
test
\end{document}

gives:

! Undefined control sequence.
l.481 \ProcessKeysPackageOptions

which is almost certainly what you also found.

Yes, it's an unrelated issue I've also reported.

Sorry I was not more specific, it was late and I was processing tests in bulk.

No problem!

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

3 participants