home-assistant / ada

Hey Ada!
Apache License 2.0
103 stars 18 forks source link

setup requires wheel #12

Closed rveachkc closed 4 years ago

rveachkc commented 4 years ago

When installing ada into a virtual environment, observed the following error:

Building wheels for collected packages: ada
  Running setup.py bdist_wheel for ada ... error
  Complete output from command /home/pi/ada_venv/bin/python3 -u -c "import setuptools, tokenize;__file__='/tmp/pip-req-build-2qc_p13f/setup.py';f=getattr(tokenize, 'open', open)(__file__);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, __file__, 'exec'))" bdist_wheel -d /tmp/pip-wheel-zff943an --python-tag cp37:
  usage: -c [global_opts] cmd1 [cmd1_opts] [cmd2 [cmd2_opts] ...]
     or: -c --help [cmd1 cmd2 ...]
     or: -c --help-commands
     or: -c cmd --help

  error: invalid command 'bdist_wheel'

It installs correctly from my fork with this line in the setup.py file.

homeassistant commented 4 years ago

Hi @rveachkc,

It seems you haven't yet signed a CLA. Please do so here.

Once you do that we will be able to review and accept this pull request.

Thanks!

frenck commented 4 years ago

Your Python installation is missing the Python Wheels. Some Linux distributions (like Ubuntu) have put this in a separate package. So make sure you have is installed.

For Ubuntu: apt install python3-wheels

rveachkc commented 4 years ago

@frenck, I understand the problem. I was able to get past it, but it's behaving like an undocumented requirement. I was hoping to fix that for others who may find the same.

This was observed on a fresh install of raspbian buster in a virtual environment. (python 3.7.3, installed via apt).

frenck commented 4 years ago

It is a specific requirement from your platform. Wheels is normally bundled with Python.