search

thk686 / odeintr

An R package for integrating differential equations in C++
18 stars 3 forks source link

Problems building from tarball #2

Closed goodsman closed 9 years ago

goodsman commented 9 years ago

Because of recent changes to the package, I have opted to build the package from its tarball in Rstudio running Windows 7. Two issues arise.

The first issue when checking the package is that roxygen2 is not available for R version 3.2.2.

The second issue arises when attempting to build the package and reload it, the check comes up with two problems, both of which are in the unused.hpp header file: -on line 57: "this" is not a potential constant expression -on line 67: both 'const' and 'constexpr' cannot be used here

Any insight into this?

goodsman commented 9 years ago

It seems that was the first issue above was my own mistake. However to build from tarball, the user must install roxygen2 and knitr themselves.

thk686 commented 9 years ago

I expect that is right. I will see if they need to be added to the deps.

THK

http://www.keittlab.org/

On Wed, Sep 23, 2015 at 12:28 PM, goodsman notifications@github.com wrote:

It seems that was the first issue above was my own mistake. However to build from tarball, the user must install roxygen2 and knitr themselves.

— Reply to this email directly or view it on GitHub https://github.com/thk686/odeintr/issues/2#issuecomment-142671873.

thk686 commented 9 years ago

http://www.keittlab.org/

On Wed, Sep 23, 2015 at 12:16 PM, goodsman notifications@github.com wrote:

Because of recent changes to the package, I have opted to build the package from its tarball in Rstudio running Windows 7. Two issues arise.

The first issue when checking the package is that roxygen2 is not available for R version 3.2.2.

You should not need roxygen2 as the git repo has the help files pre-built, ie, I run roxygen2 before pushing to the repo. If you are building from within RStudio, you need to tell it not to call roxygen2.

The second issue arises when attempting to build the package and reload it, the check comes up with two problems, both of which are in the unused.hpp header file: -on line 57: "this" is not a potential constant expression -on line 67: both 'const' and 'constexpr' cannot be used here

Any insight into this?

Which version of gcc? I think you need 4.8 to compile Boost Fusion, which is a dependency of Boost ODEINT.

— Reply to this email directly or view it on GitHub https://github.com/thk686/odeintr/issues/2.

thk686 commented 9 years ago

I can confirm that g++ 4.6.x does not work with the version of Boost libraries supplied by the BH R package. I had to hack my TravisCI setup to install g++ 4.8.x to get it to build. So I suspect this will not work with Windows until RTools is updated to g++ >= 4.8. The very latest RTools uses g++ 4.9.x, so there is some hope. I don't think it is released yet however.

thk686 commented 9 years ago

Could you try RTools 3.3 and let me know if it works?

goodsman commented 9 years ago

Even with the new version of RTools (3.3), I have been unable to build from the tarball.

On Fri, Sep 25, 2015 at 9:13 AM, Tim Keitt notifications@github.com wrote:

Could you try RTools 3.3 and let me know if it works?

— Reply to this email directly or view it on GitHub https://github.com/thk686/odeintr/issues/2#issuecomment-143248950.

Devin W. Goodsman Postdoctoral Researcher Spatial Ecology and Entomology University of Alberta

thk686 commented 9 years ago

I think it v3.3 has both 4.6 and 4.9 versions of g++. I don't know how to select the one that is used.

On Fri, Sep 25, 2015 at 4:23 PM, goodsman notifications@github.com wrote:

Even with the new version of RTools (3.3), I have been unable to build from the tarball.

On Fri, Sep 25, 2015 at 9:13 AM, Tim Keitt notifications@github.com wrote:

Could you try RTools 3.3 and let me know if it works?

— Reply to this email directly or view it on GitHub https://github.com/thk686/odeintr/issues/2#issuecomment-143248950.

Devin W. Goodsman Postdoctoral Researcher Spatial Ecology and Entomology University of Alberta

— Reply to this email directly or view it on GitHub https://github.com/thk686/odeintr/issues/2#issuecomment-143357849.

thk686 commented 9 years ago

I now include the necessary Boost libraries along with odeintr. It now compiles under g++ 4.6 and so should work with windows.