r-devel / r-dev-env

A containerised development environment for editing and compiling the R source code.
https://contributor.r-project.org/r-dev-env/
MIT License
29 stars 6 forks source link

dockerfile: updated dockerfile with valgrind installation #154

Closed StarTrooper08 closed 1 month ago

StarTrooper08 commented 2 months ago

updated dockerfile with valgrind installation

PR for issue number : #101

hturner commented 1 month ago

It seems that valgrind is already installed (this is on a codespace on the main branch):

@hturner ➜ /workspaces/r-dev-env $ dpkg -l valgrind
Desired=Unknown/Install/Remove/Purge/Hold
| Status=Not/Inst/Conf-files/Unpacked/halF-conf/Half-inst/trig-aWait/Trig-pend
|/ Err?=(none)/Reinst-required (Status,Err: uppercase=bad)
||/ Name           Version           Architecture Description
+++-==============-=================-============-=============================================================
ii  valgrind       1:3.18.1-1ubuntu2 amd64        instrumentation framework for building dynamic analysis tools

I thought we may also need valgrind-devel, but it seems this does not exist on Ubuntu and the valgrind package already has the header (.h) files (https://packages.ubuntu.com/focal/amd64/valgrind/filelist). So we can close this PR and I will update the issue to make it a documentation one.