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

R 3.6.0 dependency? #4

Closed
tedmoorman opened this issue Mar 1, 2022 · 2 comments
Closed

R 3.6.0 dependency? #4

tedmoorman opened this issue Mar 1, 2022 · 2 comments

Comments

@tedmoorman
Copy link

tedmoorman commented Mar 1, 2022

Hi, is there any reason the package is dependent on R 3.6.0?

I tried to build the package for 3.5.3 and received the following message:

* checking for file ‘reclin2-master/DESCRIPTION’ ... OK
* preparing ‘reclin2’:
* checking DESCRIPTION meta-information ... OK
* cleaning src
* installing the package to process help pages
* saving partial Rd database
Error in loadVignetteBuilder(pkgdir, TRUE) : 
  vignette builder 'simplermarkdown' not found
Execution halted

I was able to install the package without a build, but just curious in case I run into any trouble.

@djvanderlaan
Copy link
Owner

I don't think the package uses features specific to R3.6 or later. Development started with R 3.6 and later, so I haven't tested with older versions than that. You try running the tests in the package to b sure.

The package does depend on simplermarkdown during building for building the vignettes. simplermarkdown was only recently released. I believe that when releasing a new package it is made available by CRAN for the then current version, the previous version and newer versions. So, I suppose simplermarkdown is not available for R3.5. It is only needed for the vignettes, so should not be needed for an install or building binary packages. Also, simplermarkdown should work on earlier versions.

@tedmoorman
Copy link
Author

I ran the R code for all the vignettes, and everything looked good. When I ran check() in devtools I received an error related to simplermarkdown. It looks like that's the only compatibility issue, so I'll close this out. Thanks for the package!

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