search

silverwind / cidr-tools

Tools to work with IPv4 and IPv6 CIDR
98 stars 10 forks source link

python dependency #1

Closed eviltik closed 6 years ago

eviltik commented 7 years ago

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 commented 6 years ago
  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 commented 6 years ago

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 !

silverwind commented 6 years ago

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

eviltik commented 6 years ago

absolutely :(

https://github.com/rs/node-netmask/issues/8

eviltik commented 6 years ago

btw, this can help

silverwind commented 6 years ago

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 commented 6 years ago

2.0.0 is out which is now pure Javascript.