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

publish to npm? #1

Open
mikeal opened this issue Nov 5, 2020 · 2 comments
Open

publish to npm? #1

mikeal opened this issue Nov 5, 2020 · 2 comments

Comments

@mikeal
Copy link

mikeal commented Nov 5, 2020

I’d like to use this, is there a reason it was never published to npm?

@RealDolos
Copy link
Owner

Well, because I never felt it was ready for primetime.

E.g. the ffi stuff is outright ugly (and I wouldn't be too sure it still works in newer node versions as is, regardless of the ugliness), forking to create a worker process to do sync fallocates is ugly too, the limitations are not great, etc.
It is probably better to re-implement this as a proper extension (be it something nan or neon based). Still a bit of a maintenance burden then, but it would avoid most of the ugliness and limitations.

On top of that, my projects that actually used or could use this have long been migrated to C++ or rust or go or C#, or abandoned.

@michaelwittig
Copy link

@mikeal I was just looking for ways to call fallocate in node and came across your issue here. Could you share how you implemented it in the end?

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

3 participants