Skip to content

Use only bytes in build commands - #1

Merged
tfriedel merged 1 commit into
tfriedel:masterfrom
deisi:master
Mar 10, 2017
Merged

tfriedel merged 1 commit into
tfriedel:masterfrom
deisi:master

Conversation

@deisi

@deisi deisi commented Mar 10, 2017

Copy link
Copy Markdown

This commit makes sure, that only byte type objects are added with struct
obj. The old try and except was failing in newer python versions.

I've tested it with python3.5 and python3.6 and it seems to work. With python2.7 it doesn't, but I think that's not a problem.

This commit makes sure, that only byte type objects are added to struct
obj. The old try and except was failing in newer python versions.
@tfriedel

Copy link
Copy Markdown
Owner

thanks! I will test it soon with my home assistant installation and make a pull request there.

@tfriedel
tfriedel merged commit 7891331 into tfriedel:master Mar 10, 2017
@deisi

deisi commented Mar 10, 2017

Copy link
Copy Markdown
Author

Okay, I'm also working on adding the group features there. :)

@deisi

deisi commented Mar 10, 2017

Copy link
Copy Markdown
Author

w8 before you make the pr to HA, because I just realized, it will be super handy to add an on member function to the Group class, because, then I can add the lightify groups with only very little code change to HA.

@deisi

deisi commented Mar 10, 2017

Copy link
Copy Markdown
Author

hmmm now I realized, when I would add everything that I need to the Group class directly, it would end up, beeing almost identical to the Light class....

@deisi

deisi commented Mar 10, 2017

Copy link
Copy Markdown
Author

okay, I think I made up my mined. I checked the hue implementation and there it is handled within HA itself, so I believe its best to follow this as a reference. Thus no further changes to python_lightify needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants