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

Fatal error #878

Closed
nthiery opened this issue Oct 18, 2017 · 1 comment
Closed

Fatal error #878

nthiery opened this issue Oct 18, 2017 · 1 comment
Labels
bug
Milestone

Comments

@nthiery
Copy link

@nthiery nthiery commented Oct 18, 2017

Dear LaTeXML devs,

Running

latexmlc Cours.tex --dest latexml/cours.html

on the following Cours.tex file:

\documentclass{article}
\makeatletter
\AtBeginPart{\advance\beamer@tempcount by 1}
\makeatother
\begin{document}
  COUCOU
\end{document}

Triggers the following error:

	at /usr/local/share/perl/5.24.1/LaTeXML/Package/TeX.pool.ltxml line 1220, <$IN> line 3.
	In Core::Definition::Primitive[\advance ... from TeX.pool.ltxml line 1220
	 <= Core::Stomach[@0x5579eb02b750] <= Core::Definition::Primitive[Begin] <= Core::Stomach[@0x5579eb02b750]
3 errors; 1 fatal error; 1 undefined macro[\AtBeginPart]

Conversion complete: 3 errors; 1 fatal error; 1 undefined macro[\AtBeginPart].
Status:conversion:3
3 errors; 1 fatal error; 1 undefined macro[\AtBeginPart]
Error! Did not write file latexml/cours.html

I am running the latest latexml installed by cloning this repo.

That's just for info. I can workaround this by redefining AtBeginPart to do nothing when running under latexmjl.

@dginev dginev added the bug label Oct 18, 2017
@dginev
Copy link
Collaborator

@dginev dginev commented Oct 18, 2017

Thanks for the report! It looks to me that there is still sufficient reason to produce a regular error message (or more), but definitely no reason for a Fatal failure. Since this is a very nice minimal example, could be a nice and easy robustness patch.

May give it a stab this weekend.

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

Successfully merging a pull request may close this issue.

None yet
2 participants