-
Notifications
You must be signed in to change notification settings - Fork 61
Prebuilt version broken #9
Comments
I'm seeing the same issue. Also:
|
take a look at my pull request, it fixes it amongst other things |
@kobigurk could you kindly provide a compiled version of your fork? According to brew page ( Homebrew/legacy-homebrew#29988 ) emscripten formula still broken on yosemite and i can't get it to work by manually installing it. Thanks in advance |
Hi Chris, Take a look at my fork, it has both the compiled version and a minified one. |
I already checked your dist folder but it looks like they are still the same as the original version. Or I'm wrong? |
You're correct, I mistakenly didn't commit the new compiled files. They're pushed now, thanks, take a look. |
Awesome! Thanks, pulled and now it works properly with recording longer than 1 minute! Really thanks Kobi I owe you a beer!! |
No problem, have fun :-) |
Yes, the prebuilt minified version is bad. I am no emscripten expert, but I got it working by "exporting" all wrapped c functions like this: Module["_get_lame_version"] = _get_lame_version; etc and then ran it through closure myself. Got the code size down to 512kb which isn't too shabby. |
@malstrom72 Can you share the minified version? |
When trying to load the prebuilt version I'm getting this.. any help?
The text was updated successfully, but these errors were encountered: