torproject / stem

Python controller library for Tor
https://stem.torproject.org/
GNU Lesser General Public License v3.0
257 stars 75 forks source link

`zstd` vs. `zstandard` #143

Open mtelka opened 9 months ago

mtelka commented 9 months ago

The piece of code in prereq.py claims there is namespace clash between zstd and zstandard, but after installing latest zstandard (version 0.21.0) I see no such clash:

$ pip install zstandard
...
$ cd ~/.local/
$ find . -type f
./lib/python3.9/site-packages/zstandard-0.21.0.dist-info/WHEEL
./lib/python3.9/site-packages/zstandard-0.21.0.dist-info/RECORD
./lib/python3.9/site-packages/zstandard-0.21.0.dist-info/INSTALLER
./lib/python3.9/site-packages/zstandard-0.21.0.dist-info/METADATA
./lib/python3.9/site-packages/zstandard-0.21.0.dist-info/LICENSE
./lib/python3.9/site-packages/zstandard-0.21.0.dist-info/top_level.txt
./lib/python3.9/site-packages/zstandard-0.21.0.dist-info/REQUESTED
./lib/python3.9/site-packages/zstandard/py.typed
./lib/python3.9/site-packages/zstandard/__init__.py
./lib/python3.9/site-packages/zstandard/backend_cffi.py
./lib/python3.9/site-packages/zstandard/_cffi.cpython-39.so
./lib/python3.9/site-packages/zstandard/__init__.pyi
./lib/python3.9/site-packages/zstandard/backend_c.cpython-39.so
./lib/python3.9/site-packages/zstandard/__pycache__/__init__.cpython-39.pyc
./lib/python3.9/site-packages/zstandard/__pycache__/backend_cffi.cpython-39.pyc
$
mtelka commented 9 months ago

Related: #142

mtelka commented 9 months ago

The zstandard project changed their primary module from zstd to zstandard long time ago (in version 0.9.0): https://github.com/indygreg/python-zstandard/commit/eda279a391e9d912ef33e55a50929770534f5df1