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

Do you want this published on npm? #8

Open
iameli opened this issue Jul 14, 2016 · 3 comments
Open

Do you want this published on npm? #8

iameli opened this issue Jul 14, 2016 · 3 comments

Comments

@iameli
Copy link
Contributor

iameli commented Jul 14, 2016

I forked and added a package.json here so that I could install with npm and experiment with using it to sync audio feeds in Stream Kitchen.

If you like I could throw in a couple scripts to automatically pull down the latest version of quiet and push it to npm.

(Thanks a ton for the project, by the way. FWIW I'd like to eventually write node bindings to the C code, but for now I'm just using the emscripten module server-side and it works fine.)

@brian-armstrong
Copy link
Member

I had been holding off on publishing because quiet.js doesn't adhere to the javascript exports pattern (notably, quiet.js creates global variables and doesn't handle dependencies nicely). I will be the first to admit that I don't know the node etiquette very well but it seemed like everyone does that, and I figured the change would end up breaking existing usages a little. I got blocked because it turned out to be somewhat nonobvious to get working with the emscripten blobs, especially libfec, which also creates some global state.

If you have any opinions on that, I'd definitely like to hear them. I will decide within a few days how I feel about this issue.

Also, glad to hear it might be useful to you. Quiet is in a weird sort of limbo state where I think it's mostly working and useful but there aren't any production uses of it, aside from the demos I've made. I'll definitely be curious to hear your feedback after you have integrated it.

@dcntrlz
Copy link

dcntrlz commented Jul 22, 2017

@brian-armstrong
Copy link
Member

I need to just do this already. The modules are mine - they're just placeholders. This task is blocked on me getting quiet.js's API into a cleaner state that doesn't pollute global namespace. It's almost there, I just need to get motivated on it to finish

@millette millette mentioned this issue Aug 2, 2017
@eliliam
Copy link

eliliam commented Oct 13, 2017

@iameli I am trying to use this library on a node server, how exactly did you impliment it and do you have any code I can work with? I really like the idea of porting a version of this to NPM.

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

4 participants