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

Mail says PREPERROR but build log says SUCCESS #445

Closed
baktoft opened this issue Jan 26, 2021 · 11 comments
Closed

Mail says PREPERROR but build log says SUCCESS #445

baktoft opened this issue Jan 26, 2021 · 11 comments

Comments

@baktoft
Copy link

baktoft commented Jan 26, 2021

First off, thanks for this awesome tool!
However, I am getting mixed signals on some of my runs. As subject indicates, I get different results for the same build in mail vs build log. See images below.
Full build log available here: https://builder.r-hub.io/status/yaps_1.2.2.9003.tar.gz-e8f306bd41a24d90a1921d46ff0c8d48

Any advice will be highly appreciated.
Thanks,
\hb

image

image

@gaborcsardi
Copy link
Collaborator

Yes, that is misleading indeed. PREPERROR means that R-hub could not run the tests, probably because some dependency failed to install. Look at the detailed logs to see what exactly happened.

@baktoft
Copy link
Author

baktoft commented Jan 26, 2021

Indeed it seems to be missing 'devtools' and 'xml2' for installing 'vdiffr'.
image

I have 'vdiffr' in DESCRIPTION Suggests and use if(requireNamespace("vdiffr", quietly = TRUE)){ ...tests...} for these particular tests. Furthermore, I am using rhub::check_for_cran(env_vars = c('_R_CHECK_FORCE_SUGGESTS_' = "false")) when checking on r-hub.

Is there anything I can do to fix this?

@gaborcsardi
Copy link
Collaborator

Check in the log why they failed to install.

@gaborcsardi
Copy link
Collaborator

Yeah, xml2 does not compile, I am not sure why.

@baktoft
Copy link
Author

baktoft commented Jan 26, 2021

It seems to boil down to xml2 failing to install.
image

Additionally, devtools report missing dependencies roxygen2 and rversions - both of these in turn depends on xml2.

@gaborcsardi
Copy link
Collaborator

In any case, this container is somewhat old, so let me try to update it.

@baktoft
Copy link
Author

baktoft commented Jan 26, 2021

Thanks! While you are at it, rhub/ubuntu-gcc-release seems to need an update too. On this, xml2 and openssl fails to install.

@TNonet
Copy link

TNonet commented Mar 24, 2021

Hi,

I seem to be facing a very similar issue.

I submitted a "rhub::check_with_sanitizers()" test to rhub this morning.
After a few minutes, I received this in an email, but the logs (https://builder.r-hub.io/status/L0Learn_2.0.2.tar.gz-d7ff3723487940dfa2a77d0bbdb1d6b7) say "success".

image

However, when I search for "error" in this log file (to see if there is a dependency issue), the only errors seem to be at the end when rhub/docker is attempting to remove files.

If anyone has any advice or insights into this issue, it would be greatly appreciated.

A wonderful tool!

@gaborcsardi
Copy link
Collaborator

@TNonet your issue is the same as #448 I think, and it is indeed a bug.

@SimonDedman
Copy link

Throwing a +1 on the pile:
image

Build success and only 1 match to "ERROR" during find-in-page,

L1060: echo R_REMOTES_NO_ERRORS_FROM_WARNINGS=true

@gaborcsardi
Copy link
Collaborator

This issue is about the previous R-hub system, it does not apply to the new system, so I am closing it now. Please see https://r-hub.github.io/rhub/ for the new system, R-hub v2.

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

4 participants