Skip to content

Always enable the dotnet module.#222

Merged
plusvic merged 1 commit intoVirusTotal:masterfrom
wxsBSD:enable_dotnet
Jan 31, 2023
Merged

Always enable the dotnet module.#222
plusvic merged 1 commit intoVirusTotal:masterfrom
wxsBSD:enable_dotnet

Conversation

@wxsBSD
Copy link
Contributor

@wxsBSD wxsBSD commented Jan 31, 2023

When building prior to this commit the dotnet module would be disabled unless you explicitly asked for it. This is the opposite of what the default is in libyara. Fix it by always building the dotnet module.

Tested with:

wxs@mbp yara-python % PYTHONPATH=build/lib.macosx-10.9-universal2-3.9 python3 -c 'import yara; print(yara.modules)'
['tests', 'pe', 'elf', 'math', 'time', 'console', 'string', 'dotnet', 'hash']
wxs@mbp yara-python %

Without this change the dotnet module would not be in the list.

When building prior to this commit the dotnet module would be disabled unless
you explicitly asked for it. This is the opposite of what the default is in
libyara. Fix it by always building the dotnet module.

Tested with:

```
wxs@mbp yara-python % PYTHONPATH=build/lib.macosx-10.9-universal2-3.9 python3 -c 'import yara; print(yara.modules)'
['tests', 'pe', 'elf', 'math', 'time', 'console', 'string', 'dotnet', 'hash']
wxs@mbp yara-python %
```

Without this change the dotnet module would not be in the list.
@plusvic plusvic merged commit d61262b into VirusTotal:master Jan 31, 2023
@wxsBSD wxsBSD deleted the enable_dotnet branch January 31, 2023 15:49
plusvic added a commit that referenced this pull request Feb 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants