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

#3 Make emscripten module build and import browserify-friendly #8

Closed
wants to merge 3 commits into from

Conversation

klesun
Copy link

@klesun klesun commented Dec 26, 2020

Solves the "unrar.RarArchive is not a constructor"

#3

@klesun
Copy link
Author

klesun commented Dec 26, 2020

See https://stackoverflow.com/a/47528954/2750743

Emscripten has some issues with browserify which are solved by compiling with MODULARIZE flag.

I suppose  was some older versions syntax...
It was reporting original code
@YuJianrong
Copy link
Owner

@klesun Thanks for you PR.
I'm currently working on a big refactory to make it works more smoothly on webpack. Hope it can solve your issue.
It will be released soon.

@klesun
Copy link
Author

klesun commented Dec 30, 2020

Oh, cool, good luck!

@YuJianrong
Copy link
Owner

This issue solved in the new release 1.0.0

@YuJianrong YuJianrong closed this Dec 30, 2020
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