-
Notifications
You must be signed in to change notification settings - Fork 15k
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 env var to export a hint file to optimize ASAR ordering #3902
Conversation
@@ -25,6 +25,12 @@ Starts the process as a normal Node.js process. | |||
|
|||
Prints Chrome's internal logging to console. | |||
|
|||
## `ELECTRON_LOG_ASAR_READS` | |||
|
|||
When Electron reads from an ASAR file, log the the read offset and file path |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typo: the the
Some asar resources are loaded from the c++ side (example). Will this code log them as well? |
Hey folks thanks for the feedback—sorry I dropped the ball on this one. I'll address those typos! @etiktin, this isn't currently logging resources loaded from the C++ side. I'm not sure if it's possible to log those to the same file, and ensure that they're written to the logged in the correct order... |
@bengotow Do you mind rebasing this on master branch? Recently we have converted all our CoffeeScript files to JavaScript. |
This is a work in progress—need to make this behavior optional via a new flag.
Glad to see everything is going to ES6. I've converted my changes to JS and updated the pull request. Also double-checked to make sure it still works correctly. |
👍 |
Add env var to export a hint file to optimize ASAR ordering
Overview here: atom/atom#10163