Skip to content
This repository has been archived by the owner on Oct 10, 2019. It is now read-only.

Prebuilt version broken #9

Open
thepixelmonk opened this issue Sep 15, 2014 · 10 comments
Open

Prebuilt version broken #9

thepixelmonk opened this issue Sep 15, 2014 · 10 comments

Comments

@thepixelmonk
Copy link

When trying to load the prebuilt version I'm getting this.. any help?

uncaught exception: abort() at qd@http://localhost:3000/assets/libmp3lame.min.js?body=1:223
O@http://localhost:3000/assets/libmp3lame.min.js?body=1:1
Qi@http://localhost:3000/assets/libmp3lame.min.js?body=1:1
Lame</j.cwrap@http://localhost:3000/assets/libmp3lame.min.js?body=1:236
Lame<@http://localhost:3000/assets/libmp3lame.min.js?body=1:580
@http://localhost:3000/assets/libmp3lame.min.js?body=1:1

"Assertion failed: Cannot call unknown function get_lame_version (perhaps LLVM optimizations or closure removed it?)"
@tokudu
Copy link

tokudu commented Sep 24, 2014

I'm seeing the same issue. Also:

Uncaught abort() at Error
    at Error (native)
    at qd (http://localhost:5000/js/libmp3lame.min.js:223:143)

@kobigurk
Copy link

take a look at my pull request, it fixes it amongst other things

@xkraty
Copy link

xkraty commented Nov 17, 2014

@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
Chris

@kobigurk
Copy link

Hi Chris,

Take a look at my fork, it has both the compiled version and a minified one.
https://github.com/kobigurk/libmp3lame-js

@xkraty
Copy link

xkraty commented Nov 17, 2014

I already checked your dist folder but it looks like they are still the same as the original version. Or I'm wrong?

@kobigurk
Copy link

You're correct, I mistakenly didn't commit the new compiled files. They're pushed now, thanks, take a look.

@xkraty
Copy link

xkraty commented Nov 17, 2014

Awesome! Thanks, pulled and now it works properly with recording longer than 1 minute! Really thanks Kobi I owe you a beer!!

@kobigurk
Copy link

No problem, have fun :-)

@malstrom72
Copy link

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.

@stalane
Copy link

stalane commented Jun 30, 2015

@malstrom72 Can you share the minified version?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants