search

ucd-ipo / agroft

Agricultural Field Trial Statistics Package
http://ucd-ipo.github.io/agroft/
BSD 3-Clause "New" or "Revised" License
4 stars 5 forks source link

Impossible to install devtools on Ubuntu 14.04 LTS with apt r-base version #74

Closed moorepants closed 9 years ago

moorepants commented 9 years ago

See: https://github.com/hadley/devtools/issues/847

I somehow have devtools installed on my laptop (14.04) but when I tried on another machine (also 14.04) devtools fails to install rversions because of a missing xml2 dependency.

One solution is to install a newer version of R from another repository, but that is such a headache for users to do. In my opinion, devtools should work with 14.04 (it is the current LTS release!).

moorepants commented 9 years ago

This means that devtools probably shouldn't be a dependency. We need to ship shinyAce 0.2.0 with the app somehow so users can install from a tarball.

rkingdc commented 9 years ago

This provides another dependency problem. Windows users will need Rtools installed to build from source. That is a separate executable file to download from CRAN and another step beyond installing base R.

On Thu, Aug 20, 2015 at 2:20 PM, Jason K. Moore notifications@github.com wrote:

This means that devtools probably shouldn't be a dependency. We need to ship shinyAce 0.2.0 with the app somehow so users can install from a tarball.

— Reply to this email directly or view it on GitHub https://github.com/ucd-ipo/aip-analysis/issues/74#issuecomment-133133860 .

rkingdc commented 9 years ago

Oh wait, we can distribute a binary (hopefully). Disregard.

On Thu, Aug 20, 2015 at 2:26 PM, Ian Kyle iankyle1@gmail.com wrote:

This provides another dependency problem. Windows users will need Rtools installed to build from source. That is a separate executable file to download from CRAN and another step beyond installing base R.

On Thu, Aug 20, 2015 at 2:20 PM, Jason K. Moore notifications@github.com wrote:

This means that devtools probably shouldn't be a dependency. We need to ship shinyAce 0.2.0 with the app somehow so users can install from a tarball.

— Reply to this email directly or view it on GitHub https://github.com/ucd-ipo/aip-analysis/issues/74#issuecomment-133133860 .

moorepants commented 9 years ago

I noticed that devtools requires different steps on Windows too.

Is there any reason I can't use the version of shinyAce that is on CRAN?

rkingdc commented 9 years ago

I think it was either 0.5 or 0.6 that the app depends on. Check the DESCRIPTION file. I think CRAN version is at 0.6.1 or something. So it should be fine.

On Thu, Aug 20, 2015 at 2:33 PM, Jason K. Moore notifications@github.com wrote:

I noticed that devtools requires different steps on Windows too.

Is there any reason I can't use the version of shinyAce that is on CRAN?

— Reply to this email directly or view it on GitHub https://github.com/ucd-ipo/aip-analysis/issues/74#issuecomment-133143814 .

moorepants commented 9 years ago

I think you are talking about shinyBS. shinyAce is the hold up. It has version 0.1.0 on CRAN but 0.2.0 on github.

rkingdc commented 9 years ago

ah whoops. Yeah I was confused. Sorry. I know it won't work with the earlier version because of a missing function in the old version, and I think it is one of the functions that makes the code that is displayed dynamic.

On Thu, Aug 20, 2015 at 4:07 PM, Jason K. Moore notifications@github.com wrote:

I think you are talking about shinyBS. shinyAce is the hold up. It has version 0.1.0 on CRAN but 0.2.0 on github.

— Reply to this email directly or view it on GitHub https://github.com/ucd-ipo/aip-analysis/issues/74#issuecomment-133174121 .

moorepants commented 9 years ago

I switched to shinyAce 0.1.0 and only had to remove the wordwrap argument. Seems to work fine.

rkingdc commented 9 years ago

ohhhh right. that was it. Just make sure long formulas and stuff don't run off the page.

On Thu, Aug 20, 2015 at 4:25 PM, Jason K. Moore notifications@github.com wrote:

I switched to shinyAce 0.1.0 and only ha to remove the wordwrap argument. Seems to work fine.

— Reply to this email directly or view it on GitHub https://github.com/ucd-ipo/aip-analysis/issues/74#issuecomment-133181749 .

moorepants commented 9 years ago

Closing this, it's just a problem endemic to R packaging, it is not the culture to support "old" versions of R, nor is install.packages() useful for installing anything but the latest versions of packages. Typically, in app development you want to pin you app to specific versions of the dependencies, otherwise it is bound to break when downstream packages have backwards incompatibilities.