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

[installer] Fix creation of Admin Command Prompt #933

Merged
merged 1 commit into from
Mar 4, 2024
Merged

Conversation

Ana06
Copy link
Member

@Ana06 Ana06 commented Mar 4, 2024

Fix argument in the creation of the Admin Command Prompt shortcut (added to the FLARE-VM taskbar) that fails the installation of installer.vm. The bug was introduced in #818, but it was not used until the version was fixed 5 days ago in #925.

I tried to fix this issue in #931 🙈 I should have tested that properly. Sorry for that. I have tested this change locally (by using our test_install.ps1 script removing the installer.vm exception).

Fixes mandiant/flare-vm#573

Fix argument in the creation of the Admin Command Prompt shortcut (added
to the FLARE-VM taskbar) that fails the installation of `installer.vm`.
@Ana06 Ana06 added the 🐛 bug Something isn't working label Mar 4, 2024
@Ana06 Ana06 self-assigned this Mar 4, 2024
@emtuls emtuls self-requested a review March 4, 2024 18:12
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.

Tested locally and it works on my end. Good to merge!

@Ana06 Ana06 merged commit 11f59e9 into mandiant:main Mar 4, 2024
4 checks passed
@@ -2,7 +2,7 @@
<package xmlns="http://schemas.microsoft.com/packaging/2015/06/nuspec.xsd">
<metadata>
<id>installer.vm</id>
<version>0.0.0.20240304</version>
<version>0.0.0.20240305</version>
Copy link
Member Author

Choose a reason for hiding this comment

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

we are in the future 🚀

Copy link

Choose a reason for hiding this comment

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

so can i just use the installer.ps1 script again on the read me page? or do i have to do something special now?

Copy link
Member Author

Choose a reason for hiding this comment

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

you can just use the installer.ps1 in the flare-vm repo. It will take the latest package from myget

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Invalid path: C:\Tools\Admin Command Prompt.lnk
3 participants