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.
Good day,
I am not well acquainted with this project or rust or node api, but there has been this bug that has been bugging me for a while, when using napi-rs, I put msvc target on its own category, since it is different from clang or gcc, and added static linking for windows, I added a build.rs script for napi, so that you can
cargo test
on the repository. Additionally I added dynamical symbol loading for gnu targets as well, so you can either choose to static or dynamic load, since I do not have msvc, I can not test how this works for msvc, but since I did not change its path, it should work as it has worked before. I noticed some people have the same issue and someone even came with a patch!/Workaround!Fixes #2001
Fixes #1893
related #1480 (comment) (patch came from this thread)
Please review