Re: [R-pkg-devel] Cryptic error on Windows but not Debian

2023-11-18 Thread Simon Urbanek
to be okay with the new reading of the policy. (At > least, the CRAN volunteers seem to accept packages which use this approach.) > > Best, > Chris > > > From: R-package-devel on behalf of > Simon Urbanek > Sent: Saturday, November 18, 2023 6:14 PM > To: Adam

Re: [R-pkg-devel] Cryptic error on Windows but not Debian

2023-11-18 Thread Adam
w reading of the policy. > (At least, the CRAN volunteers seem to accept packages which use this > approach.) > > Best, > Chris > > > From: R-package-devel on behalf > of Simon Urbanek > Sent: Saturday, November 18, 2023 6:14 PM > To: Adam > Cc: r-package-dev

Re: [R-pkg-devel] Cryptic error on Windows but not Debian

2023-11-18 Thread Kenny, Christopher
rom: R-package-devel on behalf of Simon Urbanek Sent: Saturday, November 18, 2023 6:14 PM To: Adam Cc: r-package-devel@r-project.org Subject: Re: [R-pkg-devel] Cryptic error on Windows but not Debian   Adam, no, it is your code in mm_authorize() that violates the CRAN policy, it is not abo

Re: [R-pkg-devel] Cryptic error on Windows but not Debian

2023-11-18 Thread Adam
Thank you so much for clarifying this. Best, Adam On Sat, Nov 18, 2023 at 6:14 PM Simon Urbanek wrote: > Adam, > > no, it is your code in mm_authorize() that violates the CRAN policy, it is > not about the test. You may not touch user's .Renviron and there is no > reason to resort to such dras

Re: [R-pkg-devel] Cryptic error on Windows but not Debian

2023-11-18 Thread Simon Urbanek
Adam, no, it is your code in mm_authorize() that violates the CRAN policy, it is not about the test. You may not touch user's .Renviron and there is no reason to resort to such drastic measures. If you want to cache user's credentials, you have to do it in a file located via tools::R_user_dir()

Re: [R-pkg-devel] Cryptic error on Windows but not Debian

2023-11-18 Thread Adam
Thank you dearly, Simon, for pointing out the policy. May a test do the following? 1. Save the user's original value for env var X. 2. Write a new value for env var X during a test. 3. Write back the original value for env var X at the end of the test. An example: test_that("mm_authorize() sets

Re: [R-pkg-devel] Cryptic error on Windows but not Debian

2023-11-18 Thread Simon Urbanek
Adam, > On Nov 19, 2023, at 9:39 AM, Adam wrote: > > Dear Ivan, > > Thank you for explaining in such depth. I had not submitted to CRAN before. > I will look into tools::R_user_dir(). > > - May you point me toward the policy that the package should not edit > .Renviron? It is the policy yo

Re: [R-pkg-devel] Cryptic error on Windows but not Debian

2023-11-18 Thread Adam
Dear Ivan, Thank you for explaining in such depth. I had not submitted to CRAN before. I will look into tools::R_user_dir(). - May you point me toward the policy that the package should not edit .Renviron? I thought I was following best practices, but maybe things have changed. For instance, this

Re: [R-pkg-devel] Cryptic error on Windows but not Debian

2023-11-18 Thread Ivan Krylov
В Fri, 17 Nov 2023 17:08:13 -0500 Adam пишет: > 1. The example should not be running in the first place, as it has > @examplesIf megamation:::mm_has_creds(), which should be FALSE for > CRAN (it depends on env. variables) The tarball I have fished from CRAN says version 0.2.0, but your GitHub re