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

Accept and ignore positional arguments shutdown #77

Merged
merged 1 commit into from
Aug 30, 2022
Merged

Conversation

mdegat01
Copy link
Owner

Fix this error on stop

Traceback (most recent call last):
Signal caught, exiting!
  File "/amr2mqtt/amr2mqtt.py", line 388, in main_loop
    amr_line = rtlamr.stdout.readline().strip()
TypeError: shutdown() takes 0 positional arguments but 2 were given
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
  File "/amr2mqtt/amr2mqtt.py", line 520, in <module>
    main_loop()
  File "/amr2mqtt/amr2mqtt.py", line 507, in main_loop
    time.sleep(2)
TypeError: shutdown() takes 0 positional arguments but 2 were given
Signal caught, exiting!

@mdegat01 mdegat01 added the bugfix Inconsistencies or issues which will cause a problem for users or implementors. label Aug 30, 2022
@mdegat01 mdegat01 merged commit 3f2a7fd into main Aug 30, 2022
@mdegat01 mdegat01 deleted the shutdown-args branch August 30, 2022 19:33
@github-actions github-actions bot locked and limited conversation to collaborators Sep 1, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bugfix Inconsistencies or issues which will cause a problem for users or implementors.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant