rubik / radon

Various code metrics for Python code
http://radon.readthedocs.org/
MIT License
1.75k stars 118 forks source link

Packaging issue with the 'tests/' directory #58

Closed saimn closed 10 years ago

saimn commented 10 years ago

Hi, as your tests directory is included in the distributed tarball as a top-level package, it is installed in python's site-packages (which can cause conflicts with other packages doing the same thing):

~ $ ls ~/lib/virtualenvs/radon/lib/python3.4/site-packages/tests
__init__.py  pathfinder_tests/  __pycache__/  run.py  test_cli.py  test_complexity_utils.py  test_complexity_visitor.py  test_halstead.py  test_other_metrics.py  test_raw.py

One option would be to exclude it from the distributed tarball, and the other option is to move it to radon.tests.

rubik commented 10 years ago

Oh, thank you for reporting this, I wasn't really aware of it. Actually, if you look into the project history, you'll see that once the tests were inside radon/, but I took them out because there were some problems with tox. I think I can solve it now and put them inside radon/ again. I cannot remove them from the tarball because then tox won't find them and won't be able to execute them.

rubik commented 10 years ago

I've released a new version on PyPI (0.5.3)

saimn commented 10 years ago

Thanks !