Skip to content
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

parameterize install.sh #2

Merged
merged 3 commits into from
Dec 8, 2021
Merged

parameterize install.sh #2

merged 3 commits into from
Dec 8, 2021

Conversation

ykhemani
Copy link

  • Parameterize install.sh
  • Add usage, logging, example output
  • Allow override of DNS server, image name, friendly name, model name
  • Move docker-compose.yml to template.
  • Reflect usage changes in README.md, also minor cleanup in README.md.

* Parameterize `install.sh`
* Add usage, logging, example output
* Allow override of DNS server, image name, friendly name, model name
* Move docker-compose.yml to template.
@shawaj
Copy link

shawaj commented Oct 21, 2021

@TonyTromp this is totally unrelated to this PR, but issues are off so this is the only place I could put this comment.

Just wanted to say - awesome work! Glad someone had more time to commit to it than me.

Was trying to get it working in balena-sound a while ago but got a bit busy

@old-square-eyes
Copy link

old-square-eyes commented Oct 23, 2021

Agree. Sorry for hijacking. Awesome work. Have it working containerised on Rasperian alongside Pihole, and Home Assistant on a Pi 3b. I think you should turn on issues. Even if you don't have time to respond to everything. Might do a PR to flesh out the steps for non HiFi berry folks.

@TonyTromp
Copy link
Owner

TonyTromp commented Dec 8, 2021 via email

Copy link
Owner

@TonyTromp TonyTromp left a comment

Choose a reason for hiding this comment

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

Looks great. Thanks

@TonyTromp TonyTromp merged commit f366466 into TonyTromp:master Dec 8, 2021
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.

None yet

4 participants