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

Relax R >= 3.4 Dependency to R >= 3.2 #24

Closed
gmonaie opened this issue Mar 2, 2018 · 3 comments
Closed

Relax R >= 3.4 Dependency to R >= 3.2 #24

gmonaie opened this issue Mar 2, 2018 · 3 comments

Comments

@gmonaie
Copy link

@gmonaie gmonaie commented Mar 2, 2018

I'm able to build from source and run with R 3.2 if i modify the DESCRIPTION file

> session_info()
Session info ---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
 setting  value                       
 version  R version 3.2.4 (2016-03-10)
 system   x86_64, darwin13.4.0        
 ui       RStudio (1.1.423)           
 language (EN)                        
 collate  en_US.UTF-8                 
 tz       America/New_York            
 date     2018-03-02                  

Packages -------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
 package              * version    date       source        
 bit                    1.1-12     2014-04-09 CRAN (R 3.2.4)
 bit64                  0.9-7      2017-05-08 CRAN (R 3.2.5)
 CVXR                 * 0.95       2018-03-02 local         
 ECOSolveR              0.4        2018-02-18 CRAN (R 3.2.4)
 gmp                    0.5-13.1   2017-03-10 CRAN (R 3.2.5)
 Matrix                 1.2-4      2016-03-02 CRAN (R 3.2.4)
 R.methodsS3            1.7.1      2016-02-16 CRAN (R 3.2.3)
 R.oo                   1.21.0     2016-11-01 CRAN (R 3.2.5)
 R.utils                2.6.0      2017-11-05 CRAN (R 3.2.5)
 R6                     2.0.1      2014-10-29 CRAN (R 3.2.4)
 Rcpp                   0.12.15    2018-01-20 CRAN (R 3.2.4)
 RcppParallel           4.3.20     2016-08-16 CRAN (R 3.2.4)
 Rmpfr                  0.6-1      2016-11-15 CRAN (R 3.2.5)
 withr                  2.1.1      2017-12-19 CRAN (R 3.2.4)
 XML                    3.98-1.4   2016-03-01 CRAN (R 3.2.4)
 xts                  * 0.10-0     2017-07-07 CRAN (R 3.2.4)
 yaml                   2.1.13     2014-06-12 CRAN (R 3.2.0)
 zoo                  * 1.7-12     2015-03-16 CRAN (R 3.2.4)
@bnaras
Copy link
Collaborator

@bnaras bnaras commented Mar 3, 2018

Actually, we would not be surprised if it compiles even with R 3.0.x. Glad that you got it going.

As for relaxing the requirement, I'm not sure we want to do that for several reasons.

  1. We don't want to run our full suite of tests (including the manual ones) on too many older versions (even dockerized or travis'd)
  2. Knowledgeable users can build from source if needed.

Perhaps, a note would satisfy? That users have been able to build from source and perform all tests?

@gmonaie
Copy link
Author

@gmonaie gmonaie commented Mar 3, 2018

@bnaras
Copy link
Collaborator

@bnaras bnaras commented May 14, 2018

Added a comment to README

@bnaras bnaras closed this May 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.