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

Improvment to auto install process when detecting error #7417

Merged
merged 8 commits into from
Nov 1, 2023

Conversation

cderv
Copy link
Collaborator

@cderv cderv commented Oct 30, 2023

This aims to fix #7252

This is another way to fix initial report of this problem at #7087 based on what we do in https://github.com/rstudio/tinytex R package

  • Loading 'expl3.sty' aborted! is considered as any other error, and expl3.sty package will be looked for
  • When triggerring install for a package, if already install we do run an update first
  • The update included a rebuild of format tree with fmtutils (https://tug.org/texlive/doc/fmtutil.html) - which could be outdated following some tlmgr update

Trigerring CI to see if this works ok. The issue from #7252 is hard to reproduce and test, but we haven't seen it in TinyTeX for while, after the same fix rstudio/tinytex@5946a2a (rstudio/tinytex#291)

@yihui pinging you in case you want to have a look

@cderv cderv force-pushed the tinytex/parse-aborted-error branch from 27b0370 to 29a2fed Compare October 30, 2023 14:19
@bhattmaulik
Copy link

I recently had the same issue. My tinytex installation is from the tinytex R package (tinytex::install_tinytex(bundle = "TinyTeX-2")). To solve it, I first updated the tlmgr (tinytex::tlmgr_update()). And finally, I also tried tinytex:::refresh_all. But none of them helped. Finally, I uninstalled and reinstalled tinytex, and then it got solved.

@cderv
Copy link
Collaborator Author

cderv commented Oct 30, 2023

Thanks for sharing @bhattmaulik.

I also tried tinytex:::refresh_all

I am surprised this did not work... Did the issue happened only with Quarto or also with R Markdown ? (as you are mention tinytex R package, I am assuming you are a Rmd user too)

(tinytex::install_tinytex(bundle = "TinyTeX-2"))

Also this should be the full bundle with schem-full isn't it ? I does not expect this one to have some issue as no missing package should be installed... 🤔

@bhattmaulik
Copy link

(1) I didn't try R Markdown. You are right that I used to be an R Markdown user earlier, but now I have completely moved to quarto. But you have given me a good idea to debug the problem, if it happens next time. (2) TinyTeX-2 is a full bundle. So, I also didn't expect to have this error. I had the issue with expl3.sty, as you have mentioned in the first comment. In fact, my first attempt was to do tinytex::parse_install, but it said it couldn't find anything for this command. Only after that I followed the other steps, which I mentioned in my previous comment.

@cderv
Copy link
Collaborator Author

cderv commented Oct 30, 2023

Ok thanks for the feedback. This may be indicated the issue is more complex that it seems.

Though tinytex::parse_install(file = "expl3.sty") should have done the right thing.

Anyhow, let see if this work ok.

@cderv cderv force-pushed the tinytex/parse-aborted-error branch from 318f82f to 4bc6ee1 Compare November 1, 2023 13:59
@cderv cderv merged commit 9eaa8d5 into main Nov 1, 2023
@cderv cderv deleted the tinytex/parse-aborted-error branch November 1, 2023 13:59
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.

detect tlmgr issue and refresh installation info
2 participants