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 7zip.plugin.asar.vm #1141

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Add 7zip.plugin.asar.vm #1141

wants to merge 1 commit into from

Conversation

d35ha
Copy link
Member

@d35ha d35ha commented Sep 30, 2024

Closes #1112.

@d35ha d35ha force-pushed the 7z-Asar-Plugin branch 4 times, most recently from 74295e2 to 1a63e9b Compare September 30, 2024 11:49
$pluginsDir = Join-Path ${Env:ProgramFiles} "7-Zip\Formats"
New-Item -ItemType Directory -Force -Path $pluginsDir | Out-Null

$zipUrl = "https://www.tc4shell.com/binary/Asar.zip"
Copy link
Member

Choose a reason for hiding this comment

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

this URL does not contain a version and will break with updates.

Copy link
Member Author

Choose a reason for hiding this comment

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

No versioned URL is available 🥲

Copy link
Member

Choose a reason for hiding this comment

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

what about install asar using VM-Install-Node-Tool instead of the 7zip plugin? Without a versioned URL, this package will eventually break with a tool update. You should be able to create the package using automation (as I did in #1048) by reviewing #1112, add the send PR label to it, and test the created PR label. Alternatively, you can create the PR manually as you did in #1140, but using the automation would be a good opportunity to get familiar with it 😉

packages/7zip.plugin.asar.vm/tools/chocolateyinstall.ps1 Outdated Show resolved Hide resolved
@Ana06 Ana06 added 🆕 package New package request/idea/PR 🌀 FLARE-VM A package or feature to be used by FLARE-VM labels Sep 30, 2024
@Ana06
Copy link
Member

Ana06 commented Sep 30, 2024

Remember to add labels when creatings PRs 😉

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.

npm package proposal: @electron/asar.vm
2 participants