fix cabot_app.py termination

CMU-cabot / cabot-ble-server

MIT License

1 stars 1 forks source link

fix cabot_app.py termination #29

Closed daisukes closed 5 months ago

daisukes commented 5 months ago

Fix the issue that ./cabot_app.py cannot be terminated by ctrl+c

docker compose run --rm ble bash
./cabot_app.py

# ctrl+c

t-hatakeyam commented 5 months ago

https://github.com/CMU-cabot/TODO-Consortium/issues/295#issuecomment-2132738041 にてreport-submitterが必要となっていますが。こちらのプルリクに含めたほうがよろしいでしょうか

miyazakifuki commented 5 months ago

プルリクに含めるとは具体的にどういうことでしょうか？ report-submitterは別の管理なのでble-serverの方で何かする必要は無いと思うのですが。

t-hatakeyam commented 5 months ago

変更した内容で、test50_log_requestエラーが発生したため、report-submitterの.install.shを実行する必要があるため、cabot-ble-server内に配置する必要があるのではないかと思ったためです。今回のプルリクに関係ないのであれば、無視してください。

miyazakifuki commented 5 months ago

スマホからlog_requestが来ると、cabot-ble-serverはreport-submitterのスクリプトを呼び出すのでreport-submitterがないとエラーになります。(正確には.install.shにて/opt/report-submitterのシンボリックリンクを作り、シンボリックリンクを通して呼び出します。) そのためそのような操作が必要なのですが、report-submitterはcabot-ble-serverとは独立した動きもするものなのでcabot-ble-server内に含める想定はありません。少なくとも今回の変更とは無関係です。