Fix/remove external buffers #2
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Merge custom branch into master
feat(*): add ci for beamcoder
fix(*): .github folder
fix(*): fix github push
Add script for install dependencies, fix publishing
LU-3176: remove segfault-handler
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Electron does not allow to use https://nodejs.org/api/n-api.html#napi_create_external_buffer
You can follow nodejs documentation or look at electron/electron#35801 directly
Because of this limitation I had to run old version of electron.
This PR fixes that problem.
I tried to avoid fix this issue itself, because I was not sure how to build beamcoder, how that c++ code should look, etc.
But it is just 3 lines of code, I tested it and it seems like it works.
I'm not sure about memory leaks.
My plan is just to release it and try to run electron on production