search

cea-sec / Sibyl

A Miasm2 based function divination.
Other
531 stars 55 forks source link

PYCParser requirement issue #74

Closed guedou closed 6 years ago

guedou commented 6 years ago

Calling sibyl find produces an error if pycparser is not installed. Do you think that it could be useful to add a requirements,txt that will install it?

commial commented 6 years ago

Hey, thanks for reporting it. It should not be a mandatory dependency. Actually, the code should already manage this case (sibyl/commons.py). I'll try to reproduce and investigate

guedou commented 6 years ago

Here are the steps to reproduce:

$ git clone https://github.com/cea-sec/Sibyl
$ cd Sibyl
$ virtualenv -p python2 ve_sibyl
$ source ve_sibyl/bin/activate
$ pip install git+https://github.com/cea-sec/miasm/
$ python setup.py install
$ sibyl find
[..]
ImportError: No module named pycparser
commial commented 6 years ago

Bug reproduced, I'm updating the issue title