Bug #11288: start_watchdog in test/lib/test/unit.rb is meaningless

• Status: Closed
• Assignee: -
• Target version: -
• ruby -v: -
• Backport: 2.0.0: UNKNOWN, 2.1: UNKNOWN, 2.2: UNKNOWN

[ngoto (Naohisa Goto)]

test/lib/test/unit.rb の 317行目付近の start_watchdog は、
_run_parallel メソッド内の408行目から呼ばれますが、
子プロセスを起動する前に呼んでいるため、
start_watchdog 内で Process.wait2 するスレッドを起動しているものの、
すぐに Errno::ECHILD 例外が発生して当該スレッドは終了してしまいます。
つまり、名前に反して watchdog を行うことはありません。

このように、start_watchdog は、現在、まったく何の働きもしていないはずのため、削除しても問題なさそうです。
または、ちゃんと watchdog するように修正するか、どちらかだと思います。

[ngoto (Naohisa Goto)]

• Related to Bug #11265: deadlock on Solaris 10 since r50900 added

[kosaki (Motohiro KOSAKI)]

なくて今誰も困ってないのですから削除に一票。
欲しくなったら戻すのはいつでも出来るでしょ。そのためにバージョン管理してるんだから。

[ngoto (Naohisa Goto)]

• Status changed from Open to Closed

r36385 (2012/7/14) にて、launch_worker の行が watchdog = start_watchdog の後ろに移動していました。

大丈夫そうなので、r50994 にて削除しました。