Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Need to look at the exit status as well #181

Closed
gaborcsardi opened this issue May 23, 2022 · 3 comments
Closed

Need to look at the exit status as well #181

gaborcsardi opened this issue May 23, 2022 · 3 comments
Labels
bug an unexpected problem or unintended behavior tidy-dev-day 🤓 Tidyverse Developer Day rstd.io/tidy-dev-day

Comments

@gaborcsardi
Copy link
Member

To see if the check process exited because of some error. E.g. if there is no internet, or something like r-lib/devtools#2376

@gaborcsardi gaborcsardi added the bug an unexpected problem or unintended behavior label May 23, 2022
@DavisVaughan
Copy link
Member

Here is the example from vetiver, which happened when CRAN was down:

── R CMD check ─────────────────────────────────────────────────────────────────
* using log directory '/home/runner/work/vetiver-r/vetiver-r/check/vetiver.Rcheck'
* using R version 4.2.0 (2022-04-22)
* using platform: x86_64-pc-linux-gnu (64-bit)
* using session charset: UTF-8
* using options '--no-manual --as-cran'
* checking for file 'vetiver/DESCRIPTION' ... OK
* this is package 'vetiver' version '0.1.4.9000'
* package encoding: UTF-8
* checking package namespace information ... OK
* checking package dependencies ...Warning in url(sprintf("%s/%s", cran, path), open = "rb") :
  URL 'https://cran.r-project.org/web/packages/packages.rds': status was 'Couldn't connect to server'
Error in url(sprintf("%s/%s", cran, path), open = "rb") : 
  cannot open the connection to 'https://cran.r-project.org/web/packages/packages.rds'
Execution halted

── R CMD check results ───────────────────────────────── vetiver 0.1.4.9000 ────
Duration: 18.8s

0 errors ✔ | 0 warnings ✔ | 0 notes ✔

@gaborcsardi gaborcsardi added the tidy-dev-day 🤓 Tidyverse Developer Day rstd.io/tidy-dev-day label Jul 17, 2024
@krlmlr
Copy link
Member

krlmlr commented Aug 15, 2024

What do we need to do here in addition to

rcmdcheck/R/package.R

Lines 242 to 249 in 17fa054

if (res$status != 0 && res$status != 1) {
stop(
call. = FALSE,
"R CMD check process failed with exit status ", res$status,
"\n\nStandard output and error:\n",
res$stdout
)
}

?

@gaborcsardi
Copy link
Member Author

Yeah, I don't really know what this was about. Thanks for taking a look!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug an unexpected problem or unintended behavior tidy-dev-day 🤓 Tidyverse Developer Day rstd.io/tidy-dev-day
Projects
None yet
Development

No branches or pull requests

3 participants