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

fix: fix bug in filesystem.insertLink #300

Closed
wants to merge 1 commit into from

Conversation

jrainlau
Copy link
Contributor

In MacOS, if I run asar.createPackage() to pack a folder into .asar, which contains "symlink", it would get wrong after I run asar.extractAll() from the generated .asar.

Let's look at this Beyond Compare result:

企业微信截图_0ef755c7-3d85-4037-8d72-906e533b1fe1

The folder in left is the origin one, and the folder in right was generated by using asar.extractAll() from the origin one's .asar ( folderLeft --> folderLeft.asar --> folderRight).

Inside the left folder, file Resources were pointed to Versions/Current/Resources, but the same file inside the right folder were pointed to Versions/A/Resources, it's wrong.

@jrainlau jrainlau requested a review from a team as a code owner January 31, 2024 12:07
.gitignore Outdated
@@ -8,3 +8,4 @@
.node-version
npm-debug.log
.idea
.vscode
Copy link
Member

Choose a reason for hiding this comment

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

This change seems to be unrelated to the fix

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry for my mistake. I just removed it in 07a14a8

chore: bump electronjs/node in .circleci/config.yml to 2.2.0

Co-authored-by: electron-roller[bot] <84116207+electron-roller[bot]@users.noreply.github.com>

fix: fix bug in filesystem.insertLink
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants