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

Failing CRAN check on r-devel-linux-x86_64-fedora-clang #308

Closed
glin opened this issue Jan 6, 2023 · 2 comments
Closed

Failing CRAN check on r-devel-linux-x86_64-fedora-clang #308

glin opened this issue Jan 6, 2023 · 2 comments

Comments

@glin
Copy link
Owner

glin commented Jan 6, 2023

Tests currently fail on CRAN for one platform, r-devel-linux-x86_64-fedora-clang: https://cran.r-project.org/web/checks/check_results_reactable.html

checking tests ... [12s/58s] ERROR
Running ‘testthat.R’ [12s/57s]
Running the tests in ‘tests/testthat.R’ failed.
Complete output:

library(testthat)
library(reactable)

test_check("reactable")
free(): double free detected in tcache 2

The original failure was in version 0.4.0, where the Fedora tests failed due to a bug with the V8 system package in Fedora 36 and below, which CRAN uses. This was fixed in the 0.4.1 submission (029e6c3) - worked around by trying to detect that V8 bug and skip related tests. That fixed the fedora-gcc test failure, but unfortunately, fedora-clang started failing from a new issue that I just have no idea about.

It's hard to test because the R-hub fedora-clang image is on an older version of Fedora (33) than CRAN (36), where the V8 R package doesn't even install. I may end up skipping these tests on CRAN completely. Sorry if you've received this message from CRAN,

We have asked for an update fixing the check problems shown at
https://cran.r-project.org/web/checks/check_results_reactable.html
with no update from the maintainer thus far.

Thus, package reactable is now scheduled for archival on 2023-01-20,
and archiving this will necessitate also archiving its CRAN strong
reverse dependencies.

@glin glin pinned this issue Jan 6, 2023
@glin
Copy link
Owner Author

glin commented Jan 7, 2023

Now hopefully fixed in 9c067a5, and submitted to CRAN as v0.4.3

@glin
Copy link
Owner Author

glin commented Jan 7, 2023

Now fixed with the 0.4.3 release, and CRAN checks are now passing: https://cran.r-project.org/web/checks/check_results_reactable.html

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

1 participant