search

JusticeRage / Gepetto

IDA plugin which queries uses language models to speed up reverse-engineering
GNU General Public License v3.0
2.87k stars 263 forks source link

Update README.md #24

Closed xCENTx closed 1 year ago

xCENTx commented 1 year ago

the original instruction about using the interpreter were unclear and confusing to say the least. Why not just open a CMD window and install the requirements while transferring the files in the first place. Installing requirements via IDA will only work 50% of the time and is really a method for advanced users anyways. 90% of the time the IDA python script handler will catch on a new line and fail to resolve the script.

Installing the requirements.txt immediately after placing the file where it belongs is the proper way of doing things. You''ll then open IDA and have a properly working Gepetto plugin , the other method requires restarting IDA.

JusticeRage commented 1 year ago

Hi! Thanks for opening this PR. I think that unfortunately the instructions you suggest are less clear and not as accurate as the original ones. In the current version of the README, the reader is encouraged to figure out which Python installation is used by IDA, and install the pip packages with this one specifically. In your version, you propose to open a command interpreter and invoke whichever Python shows up first in the path. But IDA may very well be using a different one, in fact I think this has a high chance of being the case. If so, your proposed instructions wouldn't work.