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

trombone.play() results in "Illegal instruction: 4" #1

Open
tandrewnichols opened this issue Apr 27, 2017 · 9 comments
Open

trombone.play() results in "Illegal instruction: 4" #1

tandrewnichols opened this issue Apr 27, 2017 · 9 comments

Comments

@tandrewnichols
Copy link

It does play correctly, but when the sound ends, this error occurs. Tried from both the REPL and a node script using node 7.7.2.

@tandrewnichols
Copy link
Author

Even a try/catch around trombone.play() does not prevent this error (which I assume means it's something in the compiled code).

@tandrewnichols
Copy link
Author

Ah, it seems this is a Mac-related issue? The OS doesn't understand some instruction. Maybe you can't fix that? (but it would be cool if you could)

@bchociej
Copy link
Owner

bchociej commented May 1, 2017

I strongly suspect there's a problem with the lame package on Mac. But I'd like to switch to a pure JS version of this whole thing if at all possible, so perhaps I'll just try and make that happen when I have some spare cycles

@tandrewnichols
Copy link
Author

Is there a way I can test that to confirm?

@bchociej
Copy link
Owner

bchociej commented May 1, 2017

Give the latest master a try, also published to npm as 1.1.0

@tandrewnichols
Copy link
Author

Nope . . . still the same. Would version of node make a difference?

@tandrewnichols
Copy link
Author

This seems to be an issue in the speaker dependency: TooTallNate/node-speaker#95

@tandrewnichols
Copy link
Author

The instructions there (npm install speaker --mpg123-backend=openal) did fix the issue. Maybe you want to list that as a peer dependency and add instructions in the README to install with that package with that particular command? Unless you know of a way to add it as a package dependency to be install with that particular flag.

@tandrewnichols
Copy link
Author

Ah, adding this to my package.json works:

"scripts": {
  "install": "npm install speaker --mpg123-backend=openal"
}

So you could remove the dependency on speaker and install it manually after install.

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

No branches or pull requests

2 participants