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

rcmdcheck fails to build anytime from GitHub clone with inst/doc removed #225

Open
MichaelChirico opened this issue Aug 18, 2024 · 1 comment

Comments

@MichaelChirico
Copy link
Contributor

setwd(tempdir())
system("git clone --depth 1 https://github.com/cran/anytime")

system("R CMD build anytime")
# works with warning about vignettes having been rebuilt
# Warning: ‘inst/doc’ files
#     ‘anytime-introduction.Rnw’, ‘anytime-introduction.pdf’
#   ignored as vignettes have been rebuilt.
#   Run R CMD build with --no-build-vignettes to prevent rebuilding.

rcmdcheck::rcmdcheck("anytime") # prompts about presence of inst/doc
unlink("anytime/inst/doc", recursive=TRUE)
rcmdcheck::rcmdcheck("anytime")

The full log is quite the mess from BH, I didn't manage to parse through & see what's wrong.

I suppose the reason rcmdcheck() requires inst/doc removed beforehand is due to the warning R CMD build throws, but for my use case (see also #224), it shouldn't really matter. So maybe the path forward (and possibly also solving #224?) is to expose a way to build in the presence of inst/doc, ignoring the warning.

@gaborcsardi
Copy link
Member

cran/anytime contains a source R package, i.e. it was already built with R CMD build. You don't need to call R CMD build on it. If you want rcmdcheck to skip the build step, call it on a tarball.

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

2 participants