-
Notifications
You must be signed in to change notification settings - Fork 21
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
handle grffile as a legacy package in template.tex #8
Conversation
using the same fix as in Rmarkdown rstudio/rmarkdown@1f63943
Not working on my end unfortunately :-( Installed the new version, restarted RStudio. Do I need to reboot? |
like in pandoc
Can you try to
I just got rid of grrfile for good here. |
This reverts commit 199461f.
Keeps insisting |
🤦♂ There were two times I tested with the draft from this package and it works on my computer. Sorry for that... |
You're da bomb. Thanks so much1 Works perfectly now. |
@rnuske this seems to work. I let you dig into the issue following all the linked and see if the current fix is what you want. There should be room for improvement as I am not expert in Latex and did not know you package before. Cheers. |
It seems you can also wait for a stub in CTAN per this comment |
Thanks a ton @cderv for researching, answering and providing a fix for the issue! |
\usepackage{graphicx,grffile} | ||
\usepackage{graphicx} | ||
% grffile has become a legacy package: https://ctan.org/pkg/grffile | ||
\IfFileExists{grffile.sty}{% |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This test doesn't really do any harm but (other than for a few hours due to a bug in the texlive distribution over the weekend) it will always be true as we still distribute grffile, it just does nothing other than include graphicx, specifically so the first version would still work without requiring any change.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the assessment @davidcarlisle.
Do you suggest the problem has already passed? Or might the grffile still be missing for sometime in Tex Live distributions of various Linux distributions due to time lags? In that case the test might be helpful for a certain time. But if the probability of missing grffile is very low it might be more sensible to take the test out again now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
that problem (that it was not in the texlive server at all) will have passed, it was only missing for a few hours so unless sites were regularly updating they would never have seen the issue, and if they have updated since, the issue has gone... however as grffile doesn't do anything now texlive may move it out of the latex-recommended collection to latex-extra so people installing a more minimal texlive may not have it installed locally (this possibility hadn't occurred to me when I made the original comment)
This will close #7 and relates to rstudio/rmarkdown#1691
There was a change with
grffile
tex package that is now a legacy package and no more on Texlive.The template calls explicitly this 📦 and creates some errors.
This PR just uses the same fix as in Rmarkdown.
I should help fix it