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

Command On Button Press Does Not Work In Combination With Docker #128

Closed
7 of 9 tasks
ploth opened this issue Feb 22, 2019 · 4 comments
Closed
7 of 9 tasks

Command On Button Press Does Not Work In Combination With Docker #128

ploth opened this issue Feb 22, 2019 · 4 comments
Assignees
Labels
Milestone

Comments

@ploth
Copy link

ploth commented Feb 22, 2019

What is the purpose of your issue?

  • Bug report (encountered problems with amazon-dash)
  • Feature request (request for a new functionality)
  • Question
  • Other

Guideline for bug reports

  • amazon-dash version: master
  • Python version: 3.7 (Docker)
  • Pip & Setuptools version: not relevant
  • Operating System: Docker (alpine)
  • The pip install or setup install command has been completed without errors
  • The python -m amazon_dash.install command has been completed without errors
  • The amazon-dash discovery command works without errors
  • I have created/edited the configuration file
  • Amazon-dash service or amazon-dash --debug run works

Description

Dockerfile uses Alpine Linux which has no support for bash.

Error executing the device Button: Error running command /usr/bin/env bash -c echo test: The error code 127 has returned. Stderr: env: can't execute 'bash': No such file or directory

Error stack:

https://github.com/Nekmo/amazon-dash/blob/develop/amazon_dash/execute.py#L36
results in /usr/bin/env bash

because bash is the default shell here
https://github.com/Nekmo/amazon-dash/blob/develop/amazon_dash/execute.py#L39

and here is no shell name provided if you want to execute a command with your dash button
https://github.com/Nekmo/amazon-dash/blob/develop/amazon_dash/execute.py#L169

@Nekmo Nekmo self-assigned this Feb 23, 2019
@Nekmo Nekmo added the bug label Feb 23, 2019
@Nekmo
Copy link
Owner

Nekmo commented Feb 23, 2019

Bash will be added to the dockerfile, thanks!

https://www.cyberciti.biz/faq/alpine-linux-install-bash-using-apk-command/

@ploth
Copy link
Author

ploth commented Feb 23, 2019

Would be the easiest fix, yes :)

@ploth
Copy link
Author

ploth commented Feb 24, 2019

It would be nice if you can push a new PyPi package too. Thanks :)

@Nekmo
Copy link
Owner

Nekmo commented Mar 11, 2019

Yes, it will be included in the next release, thanks 👍

@Nekmo Nekmo added this to the v1.3.3 milestone Mar 11, 2019
@Nekmo Nekmo closed this as completed Mar 14, 2019
@Nekmo Nekmo modified the milestones: v1.3.3, v1.4.0 Mar 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants