Skip to content

Update recorder.markdown #1921

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.

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

Merged
merged 1 commit into from
Feb 13, 2017
Merged

Update recorder.markdown #1921

merged 1 commit into from
Feb 13, 2017

Conversation

nodomain
Copy link

Updated to include the "native" mysqlclient package as documented in https://community.home-assistant.io/t/error-in-setting-up-recorder-with-mysql/7705/4 - verified that it works as well ;-)

Description:

Pull request in home-assistant (if applicable): home-assistant/home-assistant#

Updated to include the "native" mysqlclient package as documented in https://community.home-assistant.io/t/error-in-setting-up-recorder-with-mysql/7705/4 - verified that it works as well ;-)
@mention-bot
Copy link

@nodomain, thanks for your PR! By analyzing the history of the files in this pull request, we identified @fabaff, @Maxr1998 and @pvizeli to be potential reviewers.

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.

Looks like this change revert #1821.

@nodomain
Copy link
Author

You are right - obviously the hint regarding virtual env was missing which made it a pain for me ;)

@DavidLP
Copy link
Contributor

DavidLP commented Jan 30, 2017

We could think about adding this as an additional option to the one before. Since installing the mysqlclient under windows can be troublesome.

@Hoobie7
Copy link
Contributor

Hoobie7 commented Feb 1, 2017

For what it's worth I could never get the pymysql working reliably but mysqlclient seems to be working well for the 24 hours it's been up. I am using a RPi3 for HASS with Windows MySQL box.

@fabaff fabaff merged commit b6d33e3 into home-assistant:current Feb 13, 2017
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.

5 participants