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

CRAN re-submission #1044

Closed
lorenzwalthert opened this issue Oct 29, 2022 · 19 comments · Fixed by #1053
Closed

CRAN re-submission #1044

lorenzwalthert opened this issue Oct 29, 2022 · 19 comments · Fixed by #1053

Comments

@lorenzwalthert
Copy link
Collaborator

lorenzwalthert commented Oct 29, 2022

Dear maintainer,

Please see the problems shown on
https://cran.r-project.org/web/checks/check_results_styler.html.

Please correct before 2022-11-09 to safely retain your package on CRAN.

The CRAN Team

Seems like only occurring on R devel with fedora flavor. Also, it's not clear if the problem was already present for version 1.7.0, as the below screenshot has no data for 1.7.0 anymore. @IndrajeetPatil did you check the CRAN results page before 1.8.0 wsa released?

Screenshot 2022-10-29 at 15 40 53

@lorenzwalthert
Copy link
Collaborator Author

lorenzwalthert commented Oct 29, 2022

This seems like a needle in the heystack thing to find -.-
Potential sources:

@lorenzwalthert
Copy link
Collaborator Author

lorenzwalthert commented Oct 29, 2022

I am trying to reproduce the problem with {rhub}:

@lorenzwalthert
Copy link
Collaborator Author

@krlmlr I guess here we might need your help.

@krlmlr
Copy link
Member

krlmlr commented Oct 29, 2022

Do I understand correctly that none of these checks have shown the error?

Is there a way to find out when this error started to occur? A hint in the most recent R-devel changes? Unfortunately, the Wayback Machine is of no help here, they don't archive CRAN check results.

@IndrajeetPatil
Copy link
Collaborator

@IndrajeetPatil did you check the CRAN results page before 1.8.0 wsa released

IIRC, yes, I did. But there was only the NOTE about is_shebang.Rd file; nothing about detritus in the temp directory.

Is the problem that we are leaving the cache directory behind, or something similar?

Btw, we error on a NOTE in R CMD check run on GHA, and the fact that it's still not showing errors makes me think that this is a change that was very recently introduced in R-devel, and we may start seeing this failure on GHA soon.

error-on: '"note"'

@lorenzwalthert
Copy link
Collaborator Author

Do I understand correctly that none of these checks have shown the error?

yes.

Wayback Machine is of no help here, they don't archive CRAN check results.

yes, tried that also.

@lorenzwalthert
Copy link
Collaborator Author

lorenzwalthert commented Oct 29, 2022

Is the problem that we are leaving the cache directory behind, or something similar?

I don’t think so, these problems looked differently: #836

@krlmlr
Copy link
Member

krlmlr commented Oct 31, 2022

These machines are fairly special:

Maybe we can even replicate the problems locally?

@lorenzwalthert
Copy link
Collaborator Author

lorenzwalthert commented Oct 31, 2022

Mirracles happen. r-devel-linux-x86_64-debian-gcc no longer shows a note. 😮. Maybe it was just a bug in R devel. Let's see if the other note also disappears.

@lorenzwalthert
Copy link
Collaborator Author

@krlmlr do you think we should to write on r devel? I have never done that before. 7 days till removal.

@krlmlr
Copy link
Member

krlmlr commented Nov 2, 2022

Let's wait for five more days, and then push an update to CRAN if the failures persist. This will buy us a bit more time.

@lorenzwalthert
Copy link
Collaborator Author

Ok, and what update should we push if we don't even know what the root cause is?

@krlmlr
Copy link
Member

krlmlr commented Nov 3, 2022 via email

@krlmlr
Copy link
Member

krlmlr commented Nov 3, 2022

Also, the last checks with this R version ran with

using R Under development (unstable) (2022-10-21 r83155)

Let's give it some more time.

@lorenzwalthert
Copy link
Collaborator Author

For your last point: Agree, all other checks have newer R versions.

@lorenzwalthert
Copy link
Collaborator Author

@krlmlr days passed and the NOTE still appears on one machine, which uses the same old 20222-10-21 R devel. Should we re-submit with all tests skipped? How would I do that? Just comment out the code in tests/test that.R?

@krlmlr
Copy link
Member

krlmlr commented Nov 7, 2022

Commenting out sounds good. The error is on CRAN's end, but we don't want styler to be pulled, which is likely to happen automatically if we don't act. And acting means resubmitting, e-mails are likely to be ignored.

@IndrajeetPatil
Copy link
Collaborator

Update is on CRAN! 🎉

@lorenzwalthert
Copy link
Collaborator Author

lorenzwalthert commented Nov 8, 2022

Screenshot 2022-11-08 at 08 34 53

and all checks green. Let's hope it was not the tests that caused a problem but just a transient flavour problem.

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

Successfully merging a pull request may close this issue.

3 participants