-
Notifications
You must be signed in to change notification settings - Fork 2
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
Automatic install scripts for #11 #23
Conversation
@@ -0,0 +1,98 @@ | |||
# Management script for dwyl smart home |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@th0mas totally get that this is "WIP", but it's looking good so far and thanks for your conscientiousness! 🙌
@nelsonic can you test this out on your system? Should be able to run |
Just confirmed that the
Could be worth adding a link to where people can download/install the latest Python. 💭 |
https://docs.python-guide.org/starting/install3/osx/
then install python:
⏳ |
After installing with
Had to follow: https://dev.to/malwarebo/how-to-set-python3-as-a-default-python-version-on-mac-4jjf
Now have:
Running the script now. |
Pretty sure |
@nelsonic you shouldn't have to worry about this if you run |
Yeah think were working on the assumption that users have Postgres and Elixir installed? If not that's ~5 lines to prompt user and install |
Side note: |
|
Thats odd, how long did you leave it for? And try running |
Will run with |
|
@@ -114,6 +114,80 @@ def gen_token(): | |||
print(proc.stdout.decode(sys.stdout.encoding)) | |||
print("\nSet this as the authorization header in your favourite API development tool.") | |||
|
|||
# Install brew, otherwise exit | |||
def must_install_brew(): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is so much more maintainable than a bash script. 👍
@nelsonic Want to try this again? |
@th0mas when you are next available, we will do a Zoom remote-pairing session to re-test this. 📆 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@th0mas the updates you made last week all look good. 👍
Install scripts to allow for a "One Click" setup of smart home.
Can be used by running:
This will download and install the smart-home-security-system repositories and do any required configuration changes.
Will fix #11