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

python dependency #1

Closed
eviltik opened this issue Oct 30, 2017 · 7 comments
Closed

python dependency #1

eviltik opened this issue Oct 30, 2017 · 7 comments

Comments

@eviltik
Copy link

eviltik commented Oct 30, 2017

Hi @silverwind !

Thank you for this module.

Suggestion 1: mention that this module need python to be installed on the system.
Suggestion 2: rewrite the python code into a js one 💃

@silverwind
Copy link
Owner

silverwind commented Nov 15, 2017

  1. It's mentioned at the top of the readme, maybe not prominent enough.
  2. Very hard as there is nothing comparable to Python's netaddr in JS land, so I'd first have to create a general-purpose IP library that's up to the task.

@eviltik
Copy link
Author

eviltik commented Nov 15, 2017

I was looking for a libs like your's, but finaly i've made something just to fit my needs here https://github.com/eviltik/cidr-clean. Features are not the same as yours.

https://github.com/rs/node-netmask under the wood, it can be interesting for your general-purpose IP lib perhaps

Cheers !

@eviltik eviltik closed this as completed Nov 15, 2017
@silverwind
Copy link
Owner

Unfortunately, it has no IPv6 support, which I consider a must-have.

@eviltik
Copy link
Author

eviltik commented Nov 15, 2017

absolutely :(

rs/node-netmask#8

@eviltik
Copy link
Author

eviltik commented Nov 26, 2017

@silverwind
Copy link
Owner

I've started a rewrite in JS, it's not quite bug-free yet and a bit heavy on the dependencies, but should get there eventually.

@silverwind
Copy link
Owner

2.0.0 is out which is now pure Javascript.

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