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

handle grffile as a legacy package in template.tex #8

Merged
merged 5 commits into from
Nov 11, 2019

Conversation

cderv
Copy link
Contributor

@cderv cderv commented Nov 11, 2019

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

@jrennstich
Copy link

jrennstich commented Nov 11, 2019

Not working on my end unfortunately :-( Installed the new version, restarted RStudio. Do I need to reboot?

@cderv
Copy link
Contributor Author

cderv commented Nov 11, 2019

Can you try to

  • restart your R session
  • reinstall the package with remotes::install_github("rnuske/komaletter#8")
  • retry your rendering
    ?

I just got rid of grrfile for good here.

@jrennstich
Copy link

jrennstich commented Nov 11, 2019

Keeps insisting
tlmgr search --file --global '/grffile.sty'
Rebooted the Mac, reinstalled from github.

@cderv
Copy link
Contributor Author

cderv commented Nov 11, 2019

🤦‍♂ There were two times grffile mentioned in the template.tex

I tested with the draft from this package and it works on my computer.

Sorry for that...

@jrennstich
Copy link

jrennstich commented Nov 11, 2019

You're da bomb. Thanks so much1 Works perfectly now.

@cderv
Copy link
Contributor Author

cderv commented Nov 11, 2019

@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.
I was just trying to help @jrennstich with a quick fix.

Cheers.

@cderv
Copy link
Contributor Author

cderv commented Nov 11, 2019

It seems you can also wait for a stub in CTAN per this comment
rstudio/rmarkdown#1692 (comment)

@rnuske rnuske merged commit 6a2c97d into rnuske:master Nov 11, 2019
@rnuske
Copy link
Owner

rnuske commented Nov 11, 2019

Thanks a ton @cderv for researching, answering and providing a fix for the issue!

@cderv cderv deleted the fix-grffile branch November 12, 2019 07:03
\usepackage{graphicx,grffile}
\usepackage{graphicx}
% grffile has become a legacy package: https://ctan.org/pkg/grffile
\IfFileExists{grffile.sty}{%

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.

Copy link
Owner

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.

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)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Not able to knit documents with Rmarkdown anymore
4 participants