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

🤖 Add rpcview.vm #705

Merged
merged 2 commits into from
Oct 27, 2023
Merged

🤖 Add rpcview.vm #705

merged 2 commits into from
Oct 27, 2023

Conversation

vm-packages
Copy link
Contributor

Automated PR from #691 🚀

@vm-packages vm-packages added the 🆕 package New package request/idea/PR label Oct 18, 2023
Copy link
Member

@Ana06 Ana06 left a 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

packages/rpcview.vm/tools/chocolateyinstall.ps1 Outdated Show resolved Hide resolved
@binjo
Copy link
Contributor

binjo commented Oct 23, 2023

Looks I don't have permission to squash and merge online.

@binjo binjo self-assigned this Oct 23, 2023
emtuls
emtuls previously requested changes Oct 23, 2023
Copy link
Member

@emtuls emtuls left a 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.

packages/rpcview.vm/rpcview.vm.nuspec Show resolved Hide resolved
Copy link
Member

@Ana06 Ana06 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for all the work @binjo (and @emtuls for the review)

<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"/>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<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).

@Ana06 Ana06 dismissed emtuls’s stale review October 27, 2023 08:23

Requested changed addressed

@binjo binjo merged commit ad8fea6 into main Oct 27, 2023
6 checks passed
@Ana06 Ana06 added this to the FLARE-VM 2023 Q4 milestone Oct 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🆕 package New package request/idea/PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants