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

False positive warning during BiocCheck - Bioconductor software dependencies #207

Closed
boyiguo1 opened this issue Jan 10, 2024 · 6 comments
Closed

Comments

@boyiguo1
Copy link

Hi,

Thanks for creating and maintaining the package!

I recently experienced false positive warnings that relates to the check on Bioconductor sofware dependencies during Bioconductor Single Package Building.

In short, I tried "Imports" or "Depends" on Bioconductor packages, where the check kept prompt warning messages.

  • Checking for Bioconductor software dependencies...
    • WARNING: No Bioconductor dependencies detected; consider a CRAN
      submission.

See more details via Bioconductor/Contributions#3264 (comment)

I don't see other issues mentioned this problem, and hence report here.

Thanks!

Best,
Boyi

@LiNk-NY
Copy link
Contributor

LiNk-NY commented Jan 10, 2024

Hi @boyiguo1
I can't seem to reproduce this locally:

> BiocCheck:::checkDESCRIPTIONFile("~/bioc/tpSVG/")
* Checking License: for restrictive use...
* Checking for recommended DESCRIPTION fields...
* Checking for Bioconductor software dependencies...
    Bioconductor dependencies found in Imports & Depends (62%).
* Checking for pinned package versions in DESCRIPTION...

This suggests that there may be something different in the SPB... I would have to investigate more.

@boyiguo1
Copy link
Author

Thank you very much!

@LiNk-NY
Copy link
Contributor

LiNk-NY commented Jan 11, 2024

@boyiguo1
No problem! Can you bump your package version and push to git.bioconductor.org for another build?

@boyiguo1
Copy link
Author

I just did. The warning message is still there with BiocCheckVersion: 1.39.16. Here is the build report (https://bioconductor.org/spb_reports/tpSVG_buildreport_20240112075527.html).

In addition, I saw there's a new error message

  • Checking package installation calls in R code...
    Error in .getDirFiles(rowdf["filename"]) :
    'fpaths' input must be a character vector
    Calls: BiocCheck ... findSymbolsInParsedCode -> apply -> FUN -> sprintf -> .getDirFiles
    Execution halted

I'm not 100% sure if it is caused by updates in v1.39.16. I didn't had the error last build run where there's no significant change except I put a package from "Depends" to "Imports" (boyiguo1/tpSVG@89ee5a3).

Thank you so much!

@LiNk-NY
Copy link
Contributor

LiNk-NY commented Jan 12, 2024

Thanks Boyi @boyiguo1
Please disregard the false positive warning on the SPB.
I have updated the package to resolve the error. Thanks!

@LiNk-NY
Copy link
Contributor

LiNk-NY commented Jan 12, 2024

I found the issue. It looks like there is a difference when the check is run on the tarball.
It should be patched in version 1.39.18.

@LiNk-NY LiNk-NY closed this as completed Jan 12, 2024
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

2 participants