Skip to content

Specifiy batik version #81

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

Specifiy batik version #81

wants to merge 1 commit into from

Conversation

@physikerwelt
Copy link
Contributor

@physikerwelt physikerwelt commented Mar 30, 2015

The latest version batik 1.8 did not work for me.

@physikerwelt
Specifiy batik version
cd2bbd7 
The latest version batik 1.8 did not work for me.
@physikerwelt
Copy link
Contributor Author

physikerwelt commented Mar 30, 2015

I think this patch should land in 0.3.
Batik 1.8 can be tested in the future.
See the error I got for 1,8
(https://gist.github.com/physikerwelt/57b3884fb89b475d1994)

@pkra
Copy link
Contributor

pkra commented Mar 30, 2015

Please make PRs against develop not `master.

@pkra pkra closed this Mar 30, 2015
@pkra
Copy link
Contributor

pkra commented Mar 30, 2015

FWIW, it seems like the problem is that Batik changed their naming scheme (to -1.8.jar) and renaming the files breaks things. If you change mj-single/page to match the file names, things work. Yet another reason for #59...

@pkra pkra mentioned this pull request Mar 30, 2015
Closed
@dpvc
Copy link
Member

dpvc commented Mar 30, 2015

You can make a symbolic link named batik-rasterizer.jar to the batik-rasterizer-1.8.jar file, and MathJax-node will find and use that just fine. Not sure what the Windows version of that is, but this works on Mac OS X, and presumably, Linux

@physikerwelt physikerwelt deleted the patch-1 branch November 5, 2015 17:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@physikerwelt @pkra @dpvc