-
Notifications
You must be signed in to change notification settings - Fork 72
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
🤖 Add rpcview.vm #705
🤖 Add rpcview.vm #705
Conversation
faf4a1a
to
18fb229
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the fix @binjo! There is one comment that needs to be removed.
Could you also squash the commits (in order to maintain a clean commit history? Alternatively, we can merge this PR with squash and merge
Looks I don't have permission to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing dependency "Microsoft Visual C++ 2019 Redistributable" which is required for tool to work.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<description>RpcView is an open-source tool to explore and decompile all RPC functionalities present on a Microsoft system</description> | ||
<dependencies> | ||
<dependency id="common.vm" /> | ||
<dependency id="vcredist140" version="14.36.32532"/> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<dependency id="vcredist140" version="14.36.32532"/> | |
<dependency id="vcredist140" version="[14.36.32532]" /> |
We should ping the dependency to ensure we test updates or limit it to a lower range like version="[14.36.32532, 14.37)"
. This dependency is used also in nmap. I think we should create a metapackage to ensure both use the same version and avoid conflicts. I have added this to #673 (we can address it in a different PR).
d69944d
to
be88660
Compare
Automated PR from #691 🚀