Skip to content

Test corner case that broke the wmf fork#132

Closed
physikerwelt wants to merge 1 commit into
mathjax:masterfrom
physikerwelt:patch-1
Closed

Test corner case that broke the wmf fork#132
physikerwelt wants to merge 1 commit into
mathjax:masterfrom
physikerwelt:patch-1

Conversation

@physikerwelt
Copy link
Copy Markdown
Contributor

This example breaks the MathJax-node wmf fork
See the output on
https://travis-ci.org/physikerwelt/MathJax-node/builds/95994288
Math Processing Error: Maximum call stack size exceeded
Error: Maximum call stack size exceeded
Thereafter, MathJax-node does not do anything

This example breaks the MathJax-node wmf fork
@physikerwelt
Copy link
Copy Markdown
Contributor Author

@dpvc can you have a look at his issue?

@pkra pkra mentioned this pull request Dec 10, 2015
@pkra
Copy link
Copy Markdown
Contributor

pkra commented Dec 10, 2015

The test isn't actually valid (there's no outcome) but I can reproduce the problem once it's fixed.

It seems to be an upstream problem, since it doesn't happen with MathJax v2.6-beta1.

I'm guessing it's mathjax/MathJax#1151. So it comes down to the fact that we didn't update mathjax-node to MathJax v2.5.2. Sorry about that!

I'm closing this as it's an upstream bug. I've opened #133 instead.

@pkra pkra closed this Dec 10, 2015
@pkra pkra added the Upstream label Dec 10, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants