USEPA / cflinuxfs3-CRAN

Precompiled CRAN repo intended to aid in cloud.gov builds
MIT License
0 stars 1 forks source link

Problem with package magrittr #13

Closed elinork closed 4 years ago

elinork commented 4 years ago

I'm getting this message in the logs about magrittr: package magrittr was installed before R 4.0.0: please re-install it

thomascjohnson commented 4 years ago

I know – I'm trying to fix it. Sorry about reopening and closing the issue, but please don't close a package request issue – I need them open to be able to run the pipeline. No need to do anything, just letting you know from here on

elinork commented 4 years ago

OK, sorry, still figuring this stuff out! Thanks for all the help!

thomascjohnson commented 4 years ago

Try now – it just added it: https://github.com/USEPA/cflinuxfs3-CRAN/runs/1254638311?check_suite_focus=true

elinork commented 4 years ago

It's still giving me the same error message and crashing

thomascjohnson commented 4 years ago

That's... odd, because the R version is definitely 4.0.2: https://github.com/USEPA/cflinuxfs3-CRAN/blob/master/.github/workflows/main.yml

Hmm, I'll look into it

elinork commented 4 years ago

I tried removing that dependency just to see, and now it's complaining about shinyjs, which does have an updated version. Should I try entering a package request for that one and see whether that helps?

thomascjohnson commented 4 years ago

Can you show me your r.yml file?

thomascjohnson commented 4 years ago

Along with the full output of your cf push?

elinork commented 4 years ago

The code is on GitHub here: https://github.com/kblocksom/eForms_NARS (I think it's public, but let me know if not.) In terms of the logs, I don't know if there's anything concerning in those, so I'll email them to you.

elinork commented 4 years ago

I may have gained a little insight into this today... I think I was wrong to ask for magrittr to be added in the first place, and I should take it out of the r.yml. The R buildpack comes with shiny, forecast, and Rserve; and forecast has a dependency on magrittr. So I think the message I was getting was some sort of conflict between the version in the buildpack and me trying to install it from here. I found a few commands that gave me the full dependency tree for what's presumably in the buildpack:

library('miniCRAN')
pkgs <- c("shiny", "forecast", "Rserve")
sort(pkgDep(pkgs, suggests = FALSE, enhances = FALSE))

Which produces the list: [1] "assertthat" "backports" "base64enc" "BH"
[5] "callr" "cli" "colorspace" "commonmark"
[9] "crayon" "curl" "desc" "digest"
[13] "ellipsis" "evaluate" "fansi" "farver"
[17] "fastmap" "forecast" "fracdiff" "ggplot2"
[21] "glue" "gtable" "htmltools" "httpuv"
[25] "isoband" "jsonlite" "labeling" "later"
[29] "lattice" "lifecycle" "lmtest" "magrittr"
[33] "MASS" "Matrix" "mgcv" "mime"
[37] "munsell" "nlme" "nnet" "pillar"
[41] "pkgbuild" "pkgconfig" "pkgload" "praise"
[45] "prettyunits" "processx" "promises" "ps"
[49] "quadprog" "quantmod" "R6" "RColorBrewer" [53] "Rcpp" "RcppArmadillo" "rlang" "rprojroot"
[57] "Rserve" "rstudioapi" "scales" "shiny"
[61] "sourcetools" "testthat" "tibble" "timeDate"
[65] "tseries" "TTR" "urca" "utf8"
[69] "vctrs" "viridisLite" "withr" "xtable"
[73] "xts" "zoo"

...Maybe those should be excluded from this repo?