-
Notifications
You must be signed in to change notification settings - Fork 41
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 downloadable book version? #27
Comments
Heya @apreshill 👋 Ah, the broken download link has been bothering me for a while! Thank you for this. Let's see how it goes. 😄 |
well this failed in a way I was not expecting: https://travis-ci.org/njtierney/rmd4sci/builds/570664147#L1252
Have you seen that error before? Also, I used to use tic, but once you showed me your recipe for getting bookdown to publish via netlify it worked a charm, and I removed tic. https://github.com/njtierney/rmd4sci/search?q=tic&type=Code |
Blurg! No I haven't seen it, but a google search suggests (to me) that Ghostscript may be associated with the PDF output. When I tried to build the PDF book locally, I fail with this error:
I would bet if you removed that build from your script it would work. |
Ah, thanks for tracing that issue down, @apreshill :) Let's see how this one goes. |
OK so removing the PDF build fixed it 🎉 Next steps will be getting the PDF build working and the MOBI version as well. |
I just ran into the same issue as @apreshill plus a few other fonts seemed to be missing on my machine when building but found that this managed to solve it for me (on ubuntu)
from https://gist.github.com/keeferrourke/d29bf364bd292c78cf774a5c37a791db#gistcomment-2716262 |
@njtierney Here is the fix to this issue: rstudio/bookdown-demo@ae54659 |
Thanks, @yihui - it works for EPUB now, but it looks like PDF causes some issues still: https://travis-ci.org/github/njtierney/rmd4sci/jobs/725481117#L4502 50% there though, thanks for your help, @yihui :) |
The problem that you mentioned seems to be for EPUB instead of PDF.
It was caused by rmd4sci/caption-reference-eqn.Rmd Lines 30 to 32 in ce9d405
I don't know why Pandoc doesn't support |
You didn't seem to build PDF at all: Line 24 in 1c734e2
|
Ack, sorry for that, @yihui, thanks for being a fresh pair of eyes. I'll more thoroughly read my travis build next time 🙂 |
To be able to build PDF, you have to either delete the font specifications (i.e. use default fonts): Lines 13 to 14 in 059dd5a
Line 5 in 059dd5a
or actually install the fonts, e.g. follow @murraycadzow's tip above. Cloning and installing all Google Fonts sounds like an overkill to me, since you only need two of them. If I were you, I won't bother doing this. The third option is to use fonts available on Ubuntu, and I believe there are many, but the downside is for people who do not use Ubuntu, they may not be able to build PDF by themselves. It depends on how much you really care about the typefaces in PDF. |
Hey @njtierney,
Not that you need to-you could disable that option all together. But you can add an epub or pdf version to your
.travis.yml
like this example:https://github.com/tidyverse/style/blob/e2d76307a7ec8feecb4b3da72501687b7f4ba770/.travis.yml#L17-L18
I would offer a PR but I don't know
tic
and am afraid of screwing up your build!Alison
The text was updated successfully, but these errors were encountered: