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 LECmd by Eric Zimmerman #243

Closed
wants to merge 1 commit into from

Conversation

thejoelpatrol
Copy link
Contributor

This tool parses Windows shortcut files (LNK), which can do all kinds of stuff besides simply point to local disk files.

This version requires .NET version 4, which is apparently already installed in FLARE VM. (there's also a version for .NET 6 which did not work for me in the VM). I'm not sure if there's a way to declare that dependency, or if we can just assume it's there.

@Ana06 Ana06 added the 🆕 package New package request/idea/PR label Mar 23, 2023
@Ana06
Copy link
Member

Ana06 commented Oct 31, 2023

@thejoelpatrol sorry that we haven't reviewed this PR before.

This version requires .NET version 4, which is apparently already installed in FLARE VM. (there's also a version for .NET 6 which did not work for me in the VM). I'm not sure if there's a way to declare that dependency, or if we can just assume it's there.

You should require the dependency explicitly. Likely the dependency is either installed by default in your windows version or required by another package installed by flare-vm. The tool should be able to install correctly in other environments. See other .net packages, for example:

<dependency id="dotnet5-desktop-runtime" version="5.0.6" />

@Ana06 Ana06 added the 🌀 FLARE-VM A package or feature to be used by FLARE-VM label Oct 31, 2023
@naacbin naacbin mentioned this pull request Dec 8, 2023
@Ana06
Copy link
Member

Ana06 commented Feb 20, 2024

@thejoelpatrol is it ok with you if we close this PR in favour of #778?

@thejoelpatrol
Copy link
Contributor Author

No problem, go ahead

@Ana06 Ana06 closed this Feb 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🌀 FLARE-VM A package or feature to be used by FLARE-VM 🆕 package New package request/idea/PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants