doudz / homeassistant-zigate

zigate component for Home Assistant
MIT License
48 stars 15 forks source link

Unable to install package zigate==0.29.1 using hassio #41

Closed Jsl-1 closed 5 years ago

Jsl-1 commented 5 years ago

Hi doudz,

The last version does not work with hassio. gcc is not available on hassio docker image, wiringpi could not be build :/

2019-03-15 14:44:35 ERROR (SyncWorker_5) [homeassistant.util.package] Unable to install package zigate==0.29.1: Failed building wheel for wiringpi Command "/usr/local/bin/python3 -u -c "import setuptools, tokenize;file='/tmp/pip-install-2mh7fmbw/wiringpi/setup.py';f=getattr(tokenize, 'open', open)(file);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, file, 'exec'))" install --record /tmp/pip-record-k1bxssnv/install-record.txt --single-version-externally-managed --prefix --compile --user --prefix=" failed with error code 1 in /tmp/pip-install-2mh7fmbw/wiringpi/

Is there a way to ignore this dependency ?

doudz commented 5 years ago

Oh, I see. Is it the same with 0.29.0 ?

doudz commented 5 years ago

It depends on Rpi.GPIO

Jsl-1 commented 5 years ago

It's ok with 0.29.0

doudz commented 5 years ago

So I'll revert to Rpi.GPIO

doudz commented 5 years ago

This dependancy is only needed on Rpi, but I don't know if I can add a conditionnal requirement in setup.py

Jsl-1 commented 5 years ago

Maybe it can work the same way you add dependencies for mqtt, and add option in home assistant to use it or not ?

doudz commented 5 years ago

no because you can't choose which package you want in HA

doudz commented 5 years ago

Should be ok now with 0.29.3

Jsl-1 commented 5 years ago

Works perfectly 👍 Sorry that you had to revert :/