Skip to content

Add warning about Bluetooth LE Device tracker not working on windows. #1177

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Oct 7, 2016
Merged

Add warning about Bluetooth LE Device tracker not working on windows. #1177

merged 1 commit into from
Oct 7, 2016

Conversation

joyrider3774
Copy link
Contributor

The Bluetooth LE Device tracker requires gattlib which does not install / work on windows so i added a simple warning about this to the documentation. I noticed this when trying it out on windows and during hass startup there was an error so i tried the below which returned OS Not supported error if you download the sources of gattlib you'll clearly see in setup.py it's linux only. I wouldnt have tried the component if i had known beforehand it would not work on windows... Hence adding the waring for other windows users.

C:\github\home-assistant\homeassistant\components\sensor>pip3 install gattlib
Collecting gattlib
  Using cached gattlib-0.20150805.tar.gz
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "C:\Users\Davy\AppData\Local\Temp\pip-build-1docr58y\gattlib\setup.py", line 48, in <module>
        raise OSError("Not supported OS")
    OSError: Not supported OS

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in C:\Users\Davy\AppData\Local\Temp\pip-build-1docr58y\gattlib\

@mention-bot
Copy link

@joyrider3774, thanks for your PR! By analyzing the history of the files in this pull request, we identified @open-homeautomation, @arsaboo and @fabaff to be potential reviewers.

@fabaff fabaff changed the base branch from next to current October 7, 2016 20:18
@fabaff fabaff changed the base branch from current to next October 7, 2016 20:19
@fabaff fabaff added the Hacktoberfest An PR on this issue (or the PR itself) is eligible towards Hacktoberfest! label Oct 7, 2016
fabaff
fabaff approved these changes Oct 7, 2016
View reviewed changes
Copy link
Member

@fabaff fabaff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks 🐦

@fabaff fabaff merged commit 3615f56 into home-assistant:next Oct 7, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fabaff fabaff fabaff approved these changes

Assignees
No one assigned
Labels
Hacktoberfest An PR on this issue (or the PR itself) is eligible towards Hacktoberfest!
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@joyrider3774 @mention-bot @fabaff