rocker-org / rocker-versioned

Run current & prior versions of R using docker
https://hub.docker.com/r/rocker/r-ver
GNU General Public License v2.0
297 stars 169 forks source link

rocker/verse does not come with curl #206

Closed 1beb closed 4 years ago

1beb commented 4 years ago

It looks like some batteries are missing from rocker/verse:latest. Not sure if this is intentional. There are other places where @cboettig has mentioned that curl is installed by default (refs #159) :

Steps to reproduce

user@host: ~# docker run -e PASSWORD=<PASS> --name rstudio -p 8787:8787 -d rocker/verse
user@host: ~# docker exec -it rstudio bin/bash
root@83e92e7deb10:/# curl -help
bash: curl: command not found
cboettig commented 4 years ago

apologies, libcurl is available, so you can do curl commands from R (e.g. via the curl package). but yeah, we should probably add the cli curl to verse as well. Should be fixed soon, and apologies for the slow reply.

1beb commented 4 years ago

No problem. I can submit a PR if you point me to the file(s) that needs to be updated. I think it would just be a matter of adding curl to https://github.com/rocker-org/rocker-versioned/blob/master/r-ver/latest.Dockerfile#L23 it looks like devel-san already has it.

I don't know what other files would need to have it. But if you list them, I can do the copy/paste.

cboettig commented 4 years ago

fixed