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

Unavailable packages causing build failure #57

Closed
cdeterman opened this issue Mar 22, 2016 · 22 comments
Closed

Unavailable packages causing build failure #57

cdeterman opened this issue Mar 22, 2016 · 22 comments

Comments

@cdeterman
Copy link

I have had several builds successfully complete previously but suddenly my build is failing complaining about packages not available.

The build page is here

with the relevant lines:

Packages required but not available: 'Rcpp' 'assertive' 'RcppEigen' 'BH'
Package suggested but not available: 'knitr'

Clearly, these packages should be available as they are all used commonly.

@krlmlr
Copy link
Owner

krlmlr commented Mar 22, 2016

I'm aware of this, I believe this is because https://cran.r-project.org/bin/windows/contrib/3.4/ is unavailable. Not much I can do about it, but you might ask CRAN.

Try switching to R-stable or R-patched for the time being.

@krlmlr
Copy link
Owner

krlmlr commented Mar 23, 2016

Actually, it's not CRAN but R-devel who should be consulted for this (R-devel is R 3.3.0, it's dubious why it's accessing the 3.4 path).

@jimhester
Copy link
Contributor

@krlmlr R-devel has been bumped to R 3.4 as of r70342

@krlmlr
Copy link
Owner

krlmlr commented Mar 24, 2016

Oh, I see. Then it's dubious why R-devel is 3.4.0... What's 3.3.x then?

@cdeterman
Copy link
Author

This seems to get part of the way there but now the devtools installation appears to be failing as the dependencies httr, jsonlite, mime, curl, openssl fail to install. You can see example here.

@krlmlr
Copy link
Owner

krlmlr commented Mar 24, 2016

@jeroenooms: Is the 3.3.0/3.4.0 confusion due to the new toolchain? I guess it would be safest to default to R-patched for now...

@jaredhuling
Copy link

This issue is still a problem and the suggested fixes by krlmlr do not work

@krlmlr
Copy link
Owner

krlmlr commented May 22, 2016

@jaredhuling: Could you point me to a failing build?

@jaredhuling
Copy link

I should have done that initially. Here it is:
https://ci.appveyor.com/project/jaredhuling/oem/build/job/j87ik7jmwh532cw4

@krlmlr
Copy link
Owner

krlmlr commented May 22, 2016

Does it work if you add RcppEigen to the "Imports" of your package?

@krlmlr
Copy link
Owner

krlmlr commented May 22, 2016

Oh, it looks like RcppEigen only has 64-bit libraries. Could you the following setting in appveyor.yml:

environment:
  global:
    WARNINGS_ARE_ERRORS: 1

  matrix:
  - R_VERSION: stable
    R_ARCH: x64

See also

- R_VERSION: stable
and the surrounding lines.

Could RcppCore/RcppEigen#34 be related?

@eddelbuettel
Copy link
Contributor

eddelbuettel commented May 22, 2016

We do not yet know what RcppCore/RcppEigen#34 actually is as it, not unlike Nessy, has only been spotted in one habitat, the Dortmund CRAN builder. Which has nothing to do with appveyor. So I'd suggest that it may be best not to refer to it at this point.

@krlmlr
Copy link
Owner

krlmlr commented May 22, 2016

@eddelbuettel: Thanks for chiming in. I only observe that RcppEigen_0.3.2.8.1.zip downloaded from CRAN doesn't have a "libs/i386" directory.

@jaredhuling
Copy link

@krlmlr setting to x64 fixed the issue. Thanks

@wlandau
Copy link

wlandau commented Oct 12, 2018

I am currently having a similar issue where the unavailability of graph (through Bioconductor) causes the failure. A couple strange things about that particular failure:

  1. WARNINGS_ARE_ERRORS was not set, yet the error message begins with "Error (converted from warning)".
  2. graph was only included in the installation because it is a dependency of CodeDepends, which is only a "Suggests" package. drake and its tests are ready for CodeDepends to be absent, and I strongly prefer that AppVeyor just roll with it if graph cannot be installed.

@gaborcsardi
Copy link

  1. is because of an update to the remotes package, which errors on warnings now, unless an env var is set. (see the remotes README).

So if you set the env var, your build will continue.

@krlmlr maybe you want to set this by default?

cc @jimhester

@wlandau
Copy link

wlandau commented Oct 12, 2018

Thanks, @gaborcsardi! A newer build is continuing as desired. For anyone looking on, I glanced here and set R_REMOTES_NO_ERRORS_FROM_WARNINGS to true here in appveyor.yml.

@krlmlr
Copy link
Owner

krlmlr commented Oct 12, 2018

Thanks. I'd rather keep the defaults lean, but let's watch how this evolves.

Maybe just add support for multiple CRAN-like repos, so that graph and others can be installed automatically? Alternatively, you could use r_install_bioc to install manually:

"install_bioc"|"bioc_install")

@jeroen
Copy link
Contributor

jeroen commented Oct 12, 2018

You can also setRepositories(ind = 1:4) before building so that bioc dependencies are automatically available.

@wlandau
Copy link

wlandau commented Oct 12, 2018

Thanks @krlmlr and @jeroen. I may come back to your suggestions depending on how ropensci/drake#41 plays out.

Come to think of it, drake's AppVeyor builds requiring graph were actually working until late last night. I see that graph version 1.58.2 was released recently, so I guess it may have taken some time for the Windows builds to show up on Bioconductor. I like the env var solution because drake has 26 "Suggests" packages, and except for 2 or 3 of those, I just skip the affected tests when they are not installed.

@LiNk-NY
Copy link
Contributor

LiNk-NY commented Oct 12, 2018

Hi all,
Please look into using BiocManager::repositories() and also #125
Thanks.

@krlmlr
Copy link
Owner

krlmlr commented Oct 13, 2018

Thanks. Can someone please open a new issue for discussing integration of BioConductor?

richfitz added a commit to vimc/orderly that referenced this issue Nov 28, 2018
Caught in the crossfire with remotes changing behaviour

krlmlr/r-appveyor#57
krlmlr/r-appveyor#127
r-lib/remotes#202
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

9 participants