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

Problem accessing bioconductor during revdep_check() #376

Open
willgearty opened this issue Mar 22, 2024 · 11 comments
Open

Problem accessing bioconductor during revdep_check() #376

willgearty opened this issue Mar 22, 2024 · 11 comments

Comments

@willgearty
Copy link

willgearty commented Mar 22, 2024

I'm trying to run revdep_check() in a github action, but I'm running into this error:

Error: Error: (converted from warning) unable to access index for repository https://bioconductor.org/packages/3.17/data/annotation/bin/macosx/big-sur-x86_64/contrib/4.3:
cannot open URL 'https://bioconductor.org/packages/3.17/data/annotation/bin/macosx/big-sur-x86_64/contrib/4.3/PACKAGES'

I get the same error when I try with a windows runner:

Error: Error: (converted from warning) unable to access index for repository https://bioconductor.org/packages/3.17/data/annotation/bin/windows/contrib/4.3:
cannot open URL 'https://bioconductor.org/packages/3.17/data/annotation/bin/windows/contrib/4.3/PACKAGES'

I'd appreciate any advice you might have, thanks.

@willgearty
Copy link
Author

The weird thing is, I ran this exact same github action for a different R package two weeks ago without any problems, and that other R package has bioconductor dependencies as well.

@willgearty
Copy link
Author

I also had no problems running revdep_check() locally.

@dieghernan
Copy link

dieghernan commented Mar 23, 2024

Same issue here on Windows and Mac (local and gh actions).

When running revdep_check() on Linux (local and gh-action) doesn’t happen, so my guess is that something changed in the Bioconductor server setup for binaries (as the error urls suggests).

@etiennebacher
Copy link

Same issue here. Note that this is also the case when I use bioc = FALSE

@etiennebacher
Copy link

In the meantime, I have a fork where I disabled most of the things that were related to Bioconductor:

pak::pak("etiennebacher/revdepcheck")

It seems to work fine, but of course this won't check for revdep failures on Bioconductor.

@willgearty
Copy link
Author

@gaborcsardi any thoughts?

@gaborcsardi
Copy link
Member

Try setting the R_BIOC_VERSION env var to 3.18.

@willgearty
Copy link
Author

willgearty commented Apr 2, 2024

That did it, thanks! Shouldn't revdepcheck always be using the most recent version of bioconductor?

@gaborcsardi
Copy link
Member

You always need to use the one that is appropriate for the current R version. Which used to be 3.17, but now it is 3.18, and Bioconductor deleted their 3.17 repository, I don't really understand why.

@dieghernan
Copy link

Try setting the R_BIOC_VERSION env var to 3.18.

This solves my issue in GH Actions:

https://github.com/dieghernan/nominatimlite/actions/runs/8537095790

Full action code: https://github.com/dieghernan/nominatimlite/blob/main/.github/workflows/revdepcheck.yaml

@friendly
Copy link

friendly commented May 6, 2024

If not already done, please handle this internally in the revdepcheck package so that users are not flummoxed by this bioconductor problem. At the minimum, you could use Sys.getenv ("R_BIOC_VERSION") to generate a warning or error that would be comprehensible instead of making maintainers google for this error.

Checking reverse dependencies is now a big part of CRAN submission / verification, that I am also surprised that revdepcheck is not on CRAN

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

5 participants