From 7b262bb9cc13bf04ed1272eca4fd6daab74ba332 Mon Sep 17 00:00:00 2001 From: Jenny Bryan Date: Sun, 28 Jul 2024 15:05:15 -0700 Subject: [PATCH] revdepcheck::cloud_check() The main run has one laggard: ``` > revdepcheck::cloud_status() [0/1/217/0 - 218] 01:28:52 ``` I don't know an easy way to tell what this 1 package is. In the meantime, I'm starting a new job to recheck scaper on R 4.4.0, because that seems to be the problem there. --- cran-comments.md | 37 +++++++++++-- revdep/README.md | 16 ++++-- revdep/cran.md | 17 ++++-- revdep/failures.md | 61 ++++++++++++++++++++- revdep/problems.md | 134 ++++++++++++++++++++++++++++++++++++++------- 5 files changed, 231 insertions(+), 34 deletions(-) diff --git a/cran-comments.md b/cran-comments.md index 44b7bf32e..de5560446 100644 --- a/cran-comments.md +++ b/cran-comments.md @@ -1,8 +1,35 @@ -This is a patch release in response to a 2024-01-22 email from Kurt Hornik about -check NOTEs related to Rd \usage sections. I have eliminated the NOTEs. - -I did NOT rerun reverse dependency checks for this patch release. - ## R CMD check results 0 errors | 0 warnings | 0 notes + +## revdepcheck results + +We checked 217 reverse dependencies, comparing R CMD check results across CRAN and dev versions of this package. + + * We saw 3 new problems + * We failed to check 1 packages + +Issues with CRAN packages are summarised below. + +### New problems + +* circle + This package makes reference to a function that is now removed from usethis. + This function has been hard-deprecated (i.e. throws an error with advice on + how to update) since usethis v2.2.0, released in December 2020. + I have opened an issue in circle's GitHub repository. + +* exampletestr + This package has a brittle test that checks for exact wording of an error + message originating in usethis. + That test fails because usethis's error message has changed slightly. + I have opened an issue in exampletestr's GitHub repository. + +* pharmaverseadam + I saw the NOTE about the installed package size being large. + I believe this is spurious/random as I don't see how this could be related to + usethis. + +### Failed to check + +* scaper (NA) diff --git a/revdep/README.md b/revdep/README.md index 1ecf9100a..23b2f2eba 100644 --- a/revdep/README.md +++ b/revdep/README.md @@ -1,8 +1,16 @@ # Revdeps -## New problems (1) +## Failed to check (1) -|package |version |error |warning |note | -|:-------|:-------|:------|:-------|:----| -|[cffr](problems.md#cffr)|0.5.0 |__+1__ | | | +|package |version |error |warning |note | +|:-------|:-------|:-----|:-------|:----| +|scaper |0.1.0 |1 | | | + +## New problems (3) + +|package |version |error |warning |note | +|:---------------|:-------|:--------|:-------|:------| +|[circle](problems.md#circle)|0.7.2 |1 |__+1__ | | +|[exampletestr](problems.md#exampletestr)|1.7.1 |1 __+1__ | | | +|[pharmaverseadam](problems.md#pharmaverseadam)|1.0.0 | | |__+1__ | diff --git a/revdep/cran.md b/revdep/cran.md index 633030f13..3b19c4fbf 100644 --- a/revdep/cran.md +++ b/revdep/cran.md @@ -1,15 +1,24 @@ ## revdepcheck results -We checked 181 reverse dependencies, comparing R CMD check results across CRAN and dev versions of this package. +We checked 217 reverse dependencies, comparing R CMD check results across CRAN and dev versions of this package. - * We saw 1 new problems - * We failed to check 0 packages + * We saw 3 new problems + * We failed to check 1 packages Issues with CRAN packages are summarised below. ### New problems (This reports the first line of each new failure) -* cffr +* circle + checking dependencies in R code ... WARNING + +* exampletestr checking tests ... ERROR +* pharmaverseadam + checking installed package size ... NOTE + +### Failed to check + +* scaper (NA) diff --git a/revdep/failures.md b/revdep/failures.md index 9a2073633..64892937f 100644 --- a/revdep/failures.md +++ b/revdep/failures.md @@ -1 +1,60 @@ -*Wow, no problems at all. :)* \ No newline at end of file +# scaper + +
+ +* Version: 0.1.0 +* GitHub: NA +* Source code: https://github.com/cran/scaper +* Date/Publication: 2023-10-19 07:20:02 UTC +* Number of recursive dependencies: 161 + +Run `revdepcheck::cloud_details(, "scaper")` for more info + +
+ +## In both + +* checking whether package ‘scaper’ can be installed ... ERROR + ``` + Installation failed. + See ‘/tmp/workdir/scaper/new/scaper.Rcheck/00install.out’ for details. + ``` + +## Installation + +### Devel + +``` +* installing *source* package ‘scaper’ ... +** package ‘scaper’ successfully unpacked and MD5 sums checked +** using staged installation +** R +** inst +** byte-compile and prepare package for lazy loading +Error in loadNamespace(j <- i[[1L]], c(lib.loc, .libPaths()), versionCheck = vI[[j]]) : + namespace ‘Matrix’ 1.5-4.1 is already loaded, but >= 1.6.4 is required +Calls: ... namespaceImportFrom -> asNamespace -> loadNamespace +Execution halted +ERROR: lazy loading failed for package ‘scaper’ +* removing ‘/tmp/workdir/scaper/new/scaper.Rcheck/scaper’ + + +``` +### CRAN + +``` +* installing *source* package ‘scaper’ ... +** package ‘scaper’ successfully unpacked and MD5 sums checked +** using staged installation +** R +** inst +** byte-compile and prepare package for lazy loading +Error in loadNamespace(j <- i[[1L]], c(lib.loc, .libPaths()), versionCheck = vI[[j]]) : + namespace ‘Matrix’ 1.5-4.1 is already loaded, but >= 1.6.4 is required +Calls: ... namespaceImportFrom -> asNamespace -> loadNamespace +Execution halted +ERROR: lazy loading failed for package ‘scaper’ +* removing ‘/tmp/workdir/scaper/old/scaper.Rcheck/scaper’ + + +``` diff --git a/revdep/problems.md b/revdep/problems.md index d9ef6a546..c61bc2482 100644 --- a/revdep/problems.md +++ b/revdep/problems.md @@ -1,14 +1,55 @@ -# cffr +# circle
-* Version: 0.5.0 -* GitHub: https://github.com/ropensci/cffr -* Source code: https://github.com/cran/cffr -* Date/Publication: 2023-05-05 12:00:02 UTC -* Number of recursive dependencies: 71 +* Version: 0.7.2 +* GitHub: https://github.com/ropensci/circle +* Source code: https://github.com/cran/circle +* Date/Publication: 2022-08-24 07:50:02 UTC +* Number of recursive dependencies: 76 -Run `revdepcheck::cloud_details(, "cffr")` for more info +Run `revdepcheck::cloud_details(, "circle")` for more info + +
+ +## Newly broken + +* checking dependencies in R code ... WARNING + ``` + Missing or unexported object: ‘usethis::github_token’ + ``` + +## In both + +* checking running R code from vignettes ... ERROR + ``` + Errors in running code in vignettes: + when running code in ‘circle.Rmd’ + ... + + > knitr::opts_chunk$set(collapse = TRUE, comment = "#>") + + > knitr::include_graphics("../man/figures/user-key.png") + + When sourcing ‘circle.R’: + Error: Cannot find the file(s): "../man/figures/user-key.png" + Execution halted + + ‘circle.Rmd’ using ‘UTF-8’... failed + ‘tic.Rmd’ using ‘UTF-8’... OK + ``` + +# exampletestr + +
+ +* Version: 1.7.1 +* GitHub: https://github.com/rorynolan/exampletestr +* Source code: https://github.com/cran/exampletestr +* Date/Publication: 2023-06-11 03:10:02 UTC +* Number of recursive dependencies: 97 + +Run `revdepcheck::cloud_details(, "exampletestr")` for more info
@@ -16,23 +57,76 @@ Run `revdepcheck::cloud_details(, "cffr")` for more info * checking tests ... ERROR ``` + Running ‘spelling.R’ Running ‘testthat.R’ Running the tests in ‘tests/testthat.R’ failed. - Last 13 lines of output: - 'test-write_citation.R:46:3', 'test-write_citation.R:111:3' - - ══ Failed tests ════════════════════════════════════════════════════════════════ - ── Failure ('test-cff_to_bibtex.R:416:3'): Errors ────────────────────────────── - `b <- cff_to_bibtex("testthat")` produced warnings. + Complete output: + > library(testthat) + > library(exampletestr) + > + > get_os <- function() { + + sysinf <- Sys.info() + + if (!is.null(sysinf)) { + ... + 6. └─exampletestr:::extract_examples("detect", tempdir(check = TRUE)) + 7. └─usethis::local_project(path = pkg_dir, quiet = TRUE) + 8. └─usethis::proj_set(path = path, force = force) + 9. └─usethis:::ui_abort(...) + 10. └─cli::cli_abort(...) + 11. └─rlang::abort(...) - [ FAIL 1 | WARN 10 | SKIP 110 | PASS 311 ] - Deleting unused snapshots: - • write_bib/append.bib - • write_bib/ascii.bib - • write_bib/noext.bib - • write_citation/append - • write_citation/noext + [ FAIL 1 | WARN 0 | SKIP 0 | PASS 27 ] Error: Test failures Execution halted ``` +## In both + +* checking running R code from vignettes ... ERROR + ``` + Errors in running code in vignettes: + when running code in ‘one-file-at-a-time.Rmd’ + ... + > knitr::opts_knit$set(root.dir = paste0(tempdir(), + + "/", "tempkg")) + + > usethis::proj_set(".") + + When sourcing ‘one-file-at-a-time.R’: + Error: ✖ Path '/tmp/RtmpGIYvQt/file132af710168/vignettes/' does not appear to + ... + + When sourcing ‘whole-package.R’: + Error: ✖ Path '/tmp/RtmpauCurR/file133f1dd70a93/vignettes/' does not appear to + be inside a project or package. + ℹ Read more in the help for `usethis::proj_get()`. + Execution halted + + ‘one-file-at-a-time.Rmd’ using ‘UTF-8’... failed + ‘one-function-at-a-time.Rmd’ using ‘UTF-8’... OK + ‘whole-package.Rmd’ using ‘UTF-8’... failed + ``` + +# pharmaverseadam + +
+ +* Version: 1.0.0 +* GitHub: https://github.com/pharmaverse/pharmaverseadam +* Source code: https://github.com/cran/pharmaverseadam +* Date/Publication: 2024-07-01 09:30:02 UTC +* Number of recursive dependencies: 120 + +Run `revdepcheck::cloud_details(, "pharmaverseadam")` for more info + +
+ +## Newly broken + +* checking installed package size ... NOTE + ``` + installed size is 7.9Mb + sub-directories of 1Mb or more: + data 7.5Mb + ``` +