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

Build PDF through tinytex::latexmk() #1222

Merged
merged 10 commits into from
Dec 15, 2017
Merged

Build PDF through tinytex::latexmk() #1222

merged 10 commits into from
Dec 15, 2017

Conversation

yihui
Copy link
Member

@yihui yihui commented Dec 15, 2017

Basically the function latexmk() in this package has been moved to the tinytex package (and improved there).

The other major change is that with keep_tex = TRUE, we only need one pass of Pandoc now (.Rmd -> .tex, and .tex is compiled to PDF through latexmk() instead of Pandoc). Previously two passes were needed (one for .tex and one for .pdf), which was inefficient.

Also note that tinytex does not require the LaTeX distribution TinyTeX or TeX Live (e.g., MikTeX is fine).

yihui added 5 commits December 3, 2017 14:33
… PDF directly

this will make it faster to build PDF with keep_tex, because Pandoc does not need to convert Markdown to LaTeX twice (once for .tex, and once for .pdf)
@yihui yihui added this to the v1.9 milestone Dec 15, 2017
@@ -74,6 +74,7 @@ Imports:
methods,
stringr (>= 1.2.0)
Suggests:
tinytex,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What about importing it and making it the default behavior?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah I was about to ask you. My only concern is that tinytex is a new package on CRAN, and I'm not sure if users with R versions lower than 3.3 are able to install it since CRAN won't provide binaries. Well, they surely can, but they just have to install.packages('tinytex', type = 'source') (it is a pure R package and does not require compilation).

I was wondering if we need to wait a little longer before fully routing to tinytex.

@jjallaire
Copy link
Member

Yeah, that's a good point. Maybe wait a while. Note that when we do installs of rmarkdown in RStudio we can work around this by explicitly doing a source install of tinytex (in fact, we can do this right away even if tinytex is a Suggests).

@jmcphers

@yihui yihui merged commit cf05cb2 into master Dec 15, 2017
@yihui yihui deleted the feature/tinytex branch December 15, 2017 23:00
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 3, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants