mdz / python-smarttub

Python API for SmartTub enabled hot tubs
MIT License
6 stars 9 forks source link

Update setup.py, add exclude for tests #9

Closed onkelbeh closed 3 years ago

onkelbeh commented 3 years ago

otherwise build will fail, it tries to install a packages "tests" at top level:

>>> Jobs: 1 of 2 complete, 1 failed                 Load avg: 1.04, 0.69, 0.65
 * Package:    dev-python/python-smarttub-0.0.12
 * Repository: HomeAssistantRepository
 * Maintainer: b@edevau.net
 * Upstream:   mdz@alcor.net
 * USE:        abi_x86_64 amd64 elibc_glibc kernel_linux python_targets_python3_8 test userland_GNU
 * FEATURES:   network-sandbox preserve-libs sandbox userpriv usersandbox
>>> Unpacking source...
>>> Unpacking python-smarttub-0.0.12.tar.gz to /var/tmp/portage/dev-python/python-smarttub-0.0.12/work
>>> Source unpacked in /var/tmp/portage/dev-python/python-smarttub-0.0.12/work
>>> Preparing source in /var/tmp/portage/dev-python/python-smarttub-0.0.12/work/python-smarttub-0.0.12 ...
>>> Source prepared.
>>> Configuring source in /var/tmp/portage/dev-python/python-smarttub-0.0.12/work/python-smarttub-0.0.12 ...
>>> Source configured.
>>> Compiling source in /var/tmp/portage/dev-python/python-smarttub-0.0.12/work/python-smarttub-0.0.12 ...
 * python3_8: running distutils-r1_run_phase distutils-r1_python_compile
running install_egg_info
running egg_info
writing python_smarttub.egg-info/PKG-INFO
writing dependency_links to python_smarttub.egg-info/dependency_links.txt
writing requirements to python_smarttub.egg-info/requires.txt
writing top-level names to python_smarttub.egg-info/top_level.txt
reading manifest file 'python_smarttub.egg-info/SOURCES.txt'
writing manifest file 'python_smarttub.egg-info/SOURCES.txt'
Copying python_smarttub.egg-info to /var/tmp/portage/dev-python/python-smarttub-0.0.12/image/_python3.8/usr/lib/python3.8/site-packages/python_smarttub-0.0.12-py3.8.egg-info
running install_scripts
 * ERROR: dev-python/python-smarttub-0.0.12::HomeAssistantRepository failed (install phase):
 *   Package installs 'tests' package which is forbidden and likely a bug in the build system.
mdz commented 3 years ago

Thanks for the report and patch. Could you try with the current version python-smarttub==0.0.22 and confirm that it fixes the issue? Or show me how to reproduce?

I can't reproduce your failure in my environment, but I confirmed that it no longer installs tests at top level

onkelbeh commented 3 years ago

Hi, does not occur anymore with python-smarttub-0.0.22.