Add the lighthouse IDA Pro plugin #911
Conversation
d35ha
force-pushed
the
ida-plugins
branch
2 times, most recently
from
9308663 to
39bcf91
Compare
d35ha
added
🆕 package
There was a problem hiding this comment.
@d35ha thanks for the PR!
it requires DynamoRIO or Intel PIN.
Then they should be installed by this package (likely as a dependency)
As mentioned in #913 (review), we can only merge PRs in which the linter and test succeed. We need to exclude packages without categories (no shortcut is created in the Tools directory) in the linter. Now we have ida.plugin.capa.vm in the EXCLUSIONS array. I suggest we change it by ida.plugin, which will exclude all IDA plugins.
| # Remove the temp installation folder | ||
| Remove-Item $toolDir -Recurse | ||
|
|
||
| # Install the code coverage binaries |
There was a problem hiding this comment.
please add to the comment why they are needed
| $packageArgs = | ||
| @{ | ||
| packageName = ${Env:ChocolateyPackageName} | ||
| unzipLocation = $toolDir |
There was a problem hiding this comment.
Use $pluginsDir directly as unzip location instead of downloading to %TEMP% and copying items over
| $zipUrl = "https://github.com/gaasedelen/lighthouse/releases/download/v$toolVersion/CodeCoverage-v$toolVersion-98830.zip" | ||
| $zipSha256 = "5b3e9659e934f8d7f4751cc0e20aa76de28822bc4f28b4936aa9be9c9e108595" | ||
|
|
||
| $toolDir = Join-Path ${Env:RAW_TOOLS_DIR} $toolName |
There was a problem hiding this comment.
why do we need this in %RAW_TOOLS_DIR%?
Required by #719.
Only works with IDA Pro and it requires DynamoRIO or Intel PIN.