volatility3 icon indicating copy to clipboard operation
volatility3 copied to clipboard
verified publisher icon volatilityfoundation

Add Snappy-C Library Instructions to README

Open emabrey opened this issue 3 years ago • 2 comments

ADD the installation instructions for the snappy-c library, to address snappy-c.h related installation error. Issue: #801

emabrey avatar Aug 03 '22 12:08 emabrey

Thanks, but I'm not keen on providing fixes for every dependency's failure to install correctly. Pip should explain which package has failed and why, and it's up to the user to determine why that package failed and install it correctly. I would be willing to point to a bug report on the python-snappy package indicating how to fix the issue, but I think providing our own explanation for a different package's failure to work in certain conditions, sets the wrong tone, and could easily explode into lots and lots of fixes for lots of packages.

ikelos avatar Aug 03 '22 22:08 ikelos

Currently pip3 install -r requirements.txt never works unless you install the snappy-c C libraries first, because the snappy-python package doesn't even attempt to install it under any conditions. This isn't a one-off, the snappy-python library requires those c libraries to be installed before you install it, or the installation fails with the header error I originally mentioned.

The library tells you to install the C devel library as a pre-requisite dependency: image

emabrey avatar Aug 04 '22 01:08 emabrey