r-lib / vdiffr

Visual regression testing and graphical diffing with testthat
https://vdiffr.r-lib.org
Other
180 stars 31 forks source link

Add libpng to `SystemRequirements`? #111

Closed glin closed 3 years ago

glin commented 3 years ago

Hi, vdiffr appears to require libpng, but it isn't listed in the SystemRequirements field. Would you be open to adding libpng (and any other system requirements I may have missed) to SystemRequirements? This would help Linux users install and run the package, and also RSPM with building Linux binary packages for vdiffr.

For example, something like this would work:

SystemRequirements: C++11, libpng
lionel- commented 3 years ago

yup sounds good. Would you like to send a PR? cc @thomasp85