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

Add Chocolatey LaTeXML test when MiKTeX is installed #8

Closed
ronaldtse opened this issue May 1, 2019 · 7 comments
Closed

Add Chocolatey LaTeXML test when MiKTeX is installed #8

ronaldtse opened this issue May 1, 2019 · 7 comments
Assignees
Labels
enhancement New feature or request

Comments

@ronaldtse
Copy link

According to metanorma/metanorma.org#83 , these packages in LaTeXML depend on external TeX distribution.

color.sty
colordvi.sty
ntheorem.sty
pgfkeys.code
pgfmath.code
siunitx.sty
tikz.sty
xcolor.sty

So the test for testing integration with MiKTeX could use any of these packages, such as ntheorem or color.

For xcolor, @dginev says:

Currently latexml won't stop you from using a package that may require a tex distribution, since in some cases you skirt away from the dependency. For instance xcolor only requires the tex distribution if you use the extended option set, but would be OK without one if you just load it with no options.

Originally posted by @ronaldtse in #7 (comment)

@ronaldtse ronaldtse added the enhancement New feature or request label May 1, 2019
@CAMOBAP
Copy link
Contributor

CAMOBAP commented May 9, 2019

@ronaldtse anything else need to be done in the scope of this ticket?

@ronaldtse
Copy link
Author

Yes, as described above:

So the test for testing integration with MiKTeX could use any of these packages, such as ntheorem or color.

You need to add a test that uses ntheorem and color that:

  • will fail if MiKTeX isn't installed;
  • will pass if MiKTeX is installed.

@CAMOBAP
Copy link
Contributor

CAMOBAP commented May 10, 2019

@ronaldtse can you give me a clue what a test there should be, should I just call latexmlc against some files which use ntheorem or color?

@ronaldtse
Copy link
Author

Yes, such as these examples (just 1 tex file for each package is enough):

https://www.dickimaw-books.com/latex/thesis/html/ntheorem.html

https://en.wikibooks.org/wiki/LaTeX/Colors

@CAMOBAP
Copy link
Contributor

CAMOBAP commented May 11, 2019

@ronaldtse tests added

@CAMOBAP
Copy link
Contributor

CAMOBAP commented May 22, 2019

I just have checked that this already done in the original latexml project.

They have a matrix with bare (no tex installation) and miktex

So I'm not sure that we need to do this at all.

@ronaldtse let me know what you think about this, should we still proceed?

@ronaldtse
Copy link
Author

Well, that’s great. Then we don’t need to do this. Thanks @dnigev !

@CAMOBAP CAMOBAP closed this as completed May 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants