Can't find tools::vignetteInfo() on R < 3.6

[krlmlr]

https://github.com/tidyverse/dbplyr/runs/727729099#step:9:30

tools::vignetteInfo() is available as of R 3.6.

[lorenzwalthert]

Given that {backports} re-implemented vignetteInfo() in r-lib/backports#36, is it worth restoring previous functionality? We could also just use a plain getFromNamespace("vignetteInfo", ns = "tools"), which is what's done in r-lib/backports#36.

[yihui]

@lorenzwalthert vignetteInfo() is used for a rather trivial feature in html_vignette() (i.e. warn against differences between vignette index entry and title), so I tend not to consider R < 3.6 for this specific feature. Thanks a lot for your suggestion!

[wch]

Is a new CRAN release planned any time soon? The reason I ask is that we have several repos which use GitHub Actions and report failing tests because of this issue, and I'm wondering whether we should (A) wait for this to be fixed in the CRAN version of rmarkdown, or (B) modify our workflows to install the dev version of rmarkdown. If there's going to be a release soon, we'll do (A). If not, we'll do (B).

[yihui]

Since the last released was made only about a week ago, I tend not to make a new release very soon. Normally I wouldn't consider new releases within a month of the last release, but if this problem is too much trouble, I can try to make the next release sooner (but still in one week or two: the main thing that holds me back is rmarkdown's thousands of rev deps to be checked).

So perhaps you'll have to do (B) for a couple of weeks. Sorry!

[hadley]

Please release as soon as possible; this is breaking a large number of workflows for us.

If you make the smallest patch to fix this problem, you do not need to re-run all the revdeps (especially since CRAN will run them too). If you really do need to run them, you should talk to @jimhester about taking advantage of our cloud revdep tools so that you can run them all in a couple of hours.

[hadley]

Could we please get an update on the release process?

[yihui]

@hadley Sorry I was mostly offline in the past few weeks, and my hands have been extremely tightly tied. I'll try my best to make the new release as soon as possible, but as I said above, it would still be in one week or two. Besides, I have an existing CRAN issue about another package sent from BDR that I haven't completely resolved, so I feel very nervous about making any new CRAN releases at this time.

It might actually be easier for me if you tell me which packages are currently broken, and I send pull requests to you add remotes::install_github('rstudio/rmarkdown') to your Travis or GHA configurations instead.

Apologies again for the trouble, and thanks for you patience and understanding!

[jimhester]

Every R package that uses rmarkdown for vignettes on R < 3.6 is broken.

[hadley]

If you need help with the release, we can do it.

[yihui]

@jimhester Yes, I fully acknowledge that every R package that has vignettes with the output format rmarkdown::html_vignette on R < 3.6 is broken. It was my bad that I failed to realize that tools::vignetteInfo() was not exported until R 3.6. It was my bad that I didn't test lower R versions on Travis earlier. I'm extremely sorry for breaking your workflows. Please give me just a few more days to prepare the release. Thank you very much for your patience!

[yihui]

On CRAN now: https://cran.r-project.org/package=rmarkdown

[wch]

Thanks!

[jimhester]

Thanks for doing a off schedule release for this @yihui!

[github-actions]

This old thread has been automatically locked. If you think you have found something related to this, please open a new issue by following the issue guide (https://yihui.org/issue/), and link to this old issue if necessary.