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

Add makeindex CTAN package in default TinyTeX-1 bundle package ? #345

Closed
cderv opened this issue Dec 1, 2021 · 3 comments
Closed

Add makeindex CTAN package in default TinyTeX-1 bundle package ? #345

cderv opened this issue Dec 1, 2021 · 3 comments

Comments

@cderv
Copy link
Contributor

cderv commented Dec 1, 2021

When running CI to check R packages, TinyTeX is often installed using setup-tinytex action which will by default install TinyTeX-1
https://github.com/yihui/tinytex/blob/3041a6cec0d2e0d6515e6b11b7624d513a78cca9/tools/install-bin-unix.sh#L20

However, it happens than R needs to build manual using this TinyTeX install but it seems makeindex is not available by default
https://github.com/r-lib/pkgdown/runs/4371903989?check_suite_focus=true#step:7:239

Warning: Warning in sys2(makeindex, shQuote(idxfile)) : '"makeindex"' not found
Error in texi2dvi(file = file, pdf = TRUE, clean = clean, quiet = quiet,  : 
  unable to run 'makeindex' on 'Rd2.idx'
Error in running tools::texi2pdf()

Maybe shipping makeindex CTAN package in the TinyTeX release bundle install would help that ?

@yihui you know more about me on this topic. I may be completely wrong.

@yihui
Copy link
Member

yihui commented Dec 1, 2021

$ tlmgr info makeindex
package:     makeindex
category:    Package
shortdesc:   Makeindex development sources
longdesc:    The package contains the development sources of makeindex,
             as derived from the texlive subversion repository.
installed:   Yes
revision:    52851
sizes:       run: 37k, bin: 353k
relocatable: No
cat-license: other-free
cat-topics:  index
collection:  collection-basic

I'm okay with adding it to https://github.com/yihui/tinytex/blob/main/tools/pkgs-yihui.txt so that it will be available in the TinyTeX bundle, but I'm not sure if it should be included in TinyTeX-1. Please feel free to commit to the main branch if you want to do the former. For the latter case, I feel it is relatively rare to require makeindex when checking R packages (i.e., it's rare to have to build PDF manuals), and it's simple enough for the package author to run tlmgr install makeindex by themselves if it is really needed.

@cderv cderv closed this as completed in 5dec9cb Dec 1, 2021
@cderv
Copy link
Contributor Author

cderv commented Dec 1, 2021

so that it will be available in the TinyTeX bundle,

That will be enough.

it's simple enough for the package author to run tlmgr install makeindex by themselves if it is really needed

I agreed.

@rabutler-usbr
Copy link

tinytex::tlmgr_install('makeindex') helped me solve this on Windows (and not knowing where tlmgr is on my system).

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

No branches or pull requests

3 participants