Skip to content

Update alarm_control_panel.egardia.markdown #3278

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 3 commits into from
Sep 9, 2017
Merged

Update alarm_control_panel.egardia.markdown #3278

merged 3 commits into from
Sep 9, 2017

Conversation

jeroenterheerdt
Copy link
Contributor

@jeroenterheerdt jeroenterheerdt commented Aug 30, 2017

Description:
Updated documentation to reflect changes in workings of Egardia component.

Pull request in home-assistant: home-assistant/core#9225

@@ -67,4 +67,20 @@ alarm_control_panel:
python3 [path/to/]egardiaserver.py &
```
**Do not forget the trailing & here; otherwise you will be having a hard time logging back into the machine.**
You can also use one of the methods mentioned here: https://www.dexterindustries.com/howto/run-a-program-on-your-raspberry-pi-at-startup/ or device your own. A sample shell script that can be used the init.d method and configures logging as well is shown below:
Copy link
Member

Choose a reason for hiding this comment

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

I think that it would help to provide a sample for systemd instead of the old init.d system.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

great idea. I will add it.

@jeroenterheerdt
Copy link
Contributor Author

Systemd example added.

@frenck frenck assigned frenck and fabaff and unassigned frenck Sep 6, 2017
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 changed the base branch from current to next September 7, 2017 13:55
@fabaff fabaff changed the base branch from next to current September 7, 2017 13:55
@fabaff fabaff added this to the 0.53 milestone Sep 7, 2017
@fabaff
Copy link
Member

fabaff commented Sep 7, 2017

Needs to go into current after the release of 0.53.

@balloob balloob merged commit 9c7562a into home-assistant:current Sep 9, 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.

4 participants