Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NLTK is required by IDA Magic String but not by Diaphora #278

Open
joxeankoret opened this issue Oct 3, 2023 · 1 comment
Open

NLTK is required by IDA Magic String but not by Diaphora #278

joxeankoret opened this issue Oct 3, 2023 · 1 comment

Comments

@joxeankoret
Copy link
Owner

joxeankoret commented Oct 3, 2023

As reported privately (and not by the first time), Diaphora complains when Python's NLTK toolkit is not installed when, in truth, it isn't required by Diaphora, as the only features from IDA Magic Strings that Diaphora uses is the ability to find compilation units.

So, either I extract from IDA Magic Strings the specific code I want and put it somewhere in a specific place for Diaphora, or I make the 'fork' of this plugin in Diaphora not complaining.

@joxeankoret joxeankoret added this to the Diaphora 3.2 milestone Oct 3, 2023
@mazegen
Copy link

mazegen commented Oct 25, 2023

I think it's all right if Diaphora requires the NLTK toolkit. My issue was that the requirement of NLTK toolkit is undocumented (AFAIK) - I had to wait one hour for the SQL database to be generated only to learn that "averaged_perceptron_tagger" is not installed.

@joxeankoret joxeankoret removed this from the Diaphora 3.2 milestone Oct 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants