crossbario / crossbar

Crossbar.io - WAMP application router
https://crossbar.io/
Other
2.05k stars 275 forks source link

Is netaddr a unused dependency? #1880

Closed suliuzh closed 2 years ago

suliuzh commented 3 years ago

Hi I noticed that netaddr was removed from source code in pull #1315. At the same time, ipaddress was used. But in configuration files such as requirements-min.txt, netaddr is still listed as a direct dependency. So is netaddr a unused dependency? Should ipaddress be listed as a direct dependency? image

oberstet commented 2 years ago

above code is indeed now using a Python stdlib package https://docs.python.org/3.9/library/ipaddress.html

however, other dependencies of crossbar still use netaddr - indirectly:

(cpy39_10) (base) oberstet@intel-nuci7:~/scm/crossbario/crossbar$ pipdeptree --reverse --packages netaddr
Warning!!! Possibly conflicting dependencies found:
* crossbar==22.3.2.dev4
 - werkzeug [required: >=0.14.1,<2, installed: 2.0.3]
 - MarkupSafe [required: >=1.1.1,<2, installed: 2.1.1]
------------------------------------------------------------------------
netaddr==0.8.0
  - crossbar==22.3.2.dev4 [requires: netaddr>=0.8.0]
  - multiaddr==0.0.9 [requires: netaddr]
    - ipfshttpclient==0.8.0a2 [requires: multiaddr>=0.0.7]
      - web3==5.28.0 [requires: ipfshttpclient==0.8.0a2]
        - cfxdb==22.3.1 [requires: web3>=5.28.0]
          - crossbar==22.3.2.dev4 [requires: cfxdb>=22.3.1]
        - crossbar==22.3.2.dev4 [requires: web3>=5.13.1]

IOW: it is correct as it is