search

linux-test-project / lcov

LCOV
GNU General Public License v2.0
866 stars 234 forks source link

Update README: installation instructions #281

Closed bytebunny closed 3 months ago

bytebunny commented 3 months ago

The readme installation instructions contain a command that does not work for some of the packages lcov depends on. For instance, trying to install Scalar::Util as per current suggested command (perl -MCPAN -e "install Scalar::Util") will result in: Can't locate object method "install" via package "Scalar::Util" at -e line 1.

Additionally, the maintainers might want to offer a one-liner to install all the perl packages at once:

for package in Capture::Tiny DateTime Devel::Cover Digest::MD5 File::Spec \
    JSON Memory::Process Module::Load::Conditional Scalar::Util Time::HiRes; \
    do yes "yes" | perl -MCPAN -e "install(${package})"; done
henry2cox commented 3 months ago

Thanks for the fix... Happy to add the additional command suggestion as well (say, in a new PR) - but the issue there is that it is very likely to go out-of-sync with the code (in the same way that the list in the README tends to get out-of-sync). I guess one could imagine putting update command into a Makefile to make that eventuality somewhat less likely.