-
Notifications
You must be signed in to change notification settings - Fork 16
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
Using unallowed path? #40
Comments
You can use |
I too received an e-mail from CRAN out of the blue that getCRUCLdata violated policy in this way. I’ve been using rappdirs for several years with it and haven’t made any changes to that package on CRAN in at least a year I think. Has there been a policy change with CRAN somewhere? I just gave the go ahead to archive the package from CRAN and left it in the rOpenSci R Universe rather than try to sort the issue out. |
@hadley Should we have another function that works similarly to |
CRAN policy seems to be a moving target, and they don't seem at all interested in working with rappdirs, despite it being used by many packages, so yeah, seems like a backport might be a better option. |
Why doesn't rappdirs merely wrap (alternately rappdirs could try to always return tempdir path during testing/examples run on CRAN, but that's probably hard and fragile). |
You can use rappdirs to query directories for other apps as well, e.g. ❯ dir(rappdirs::user_config_dir("RStudio", NULL))
[1] "blob_storage" "Cache"
[3] "Code Cache" "config.json"
[5] "Cookies" "Cookies-journal"
[7] "DawnCache" "GPUCache"
[9] "Local Storage" "Network Persistent State"
[11] "Preferences" "QtWebEngine"
[13] "Session Storage" "SingletonCookie"
[15] "SingletonLock" "SingletonSocket"
[17] "TransportSecurity" should return the correct place on all platforms.
Yes, that does not really work if you use another package or tool during the tests/checks, that would be also redirected to tempdir, not just the tested package. |
@cboettig also what happens if there was important stuff stored in the directory that the older versions of R used? I think it's hard to totally automate this. |
Thanks @hadley @gaborcsardi , that all makes sense. I have migrated most of my packages to the Given that rappdirs is widely used (even the docs for Anyway thanks for your explanations and all you do to support the R community! 🚀 |
Yeah, I think at some point we'll just supersede this package and point folks to |
We could probably add some of these arguments to the README. |
Yeah, and I think we should probably steer new users away from the package. Maybe include a backported version of |
Just checking in on this. I recently had my package removed from CRAN for a policy violation due to the use of |
The plan is to deprecate rappdirs in the near future. |
@hadley Actually, there are at least three reasons for using rappdirs, still:
AFAICT there is nothing wrong with using rappdirs, as long as your tests and examples (and the tests and examples of other packages using your package!) do not create any files. This said, we can encourage people to use |
@gaborcsardi ah that makes sense. We can definitely keep it around, but most people will be best served by using the R functions now, I think. |
It is also nice that |
Happy New Year!
In my weathercan package I included an option for saving a data frame of stations data to a cache directory, found with rappdirs.
However, I recently heard from CRAN
I definitely don't blame you for my package getting booted, but I thought I might check in case I used the function incorrectly, or there was a change to CRAN policy. Any suggestions?
The text was updated successfully, but these errors were encountered: