nyagos が Ctrl-C で終了してしまう件

SIGINT をキャッチしたら子プロセスを kill する

あ、厳密にいうと、

• SIGINT をキャッチしたら、context.WithCancel を使って現在やっている処理にキャンセル処理を通知する（ shell/loop.go の (*Shell)Loop 関数内 ）
• 外部プロセスの終了待ちをしている goroutine がキャンセルを検知すると、外部プロセスに kill する（ shell/interpret.go の (*Shell)startAndWaitProcess 関数内 )

という動作をしています

一つの可能性としては、signal.Notify （禁止）と signal.Stop （解除）が頻繁すぎて、そこにスキが出来ているというのもあるかもしれない

• シェルの基本ループ (shell/Loop.go の (*Shell)Loop関数
  • 一行入力 in "go-readline-ny"
    • １文字キー入力( "mattn/go-tty" )
      • (tty.*TTY)Raw()実行時にENABLE_PROCESSED_INPUT がリセットされ、Raw()解除時に元に戻される
      • この間、Ctrl-C で割り込み(SIGINT)自体が発生しない
  • コマンド実行部 (shell/Interpreter.go)
    • １文ごとに signal.Notify → コマンド実行 → signal.Stop を繰り返す
    • siginal.Notify～Stopの間、SIGINT は context のキャンセルに解釈されなおす

一行入力とコマンド実行部との遷移の間が割り込みを受け付ける隙になっている可能性が高い。signal 操作と go-tty は衝突しないので、ループの外側に signal.Notify～Stop の範囲を広げてみるか

• (#342) Add signal.Ignore(os.Interrpt) on the top level experimentally · zetamatta/nyagos@5af8404

トップレベルで、signal.Ignore(os.Interrpt) を入れてみました。多分、下位の signal.Notify/Stop や ENABLE_PROCESSED_INPUTのリセットと干渉しないので、これで直るはず…

この修正は "frame" というサブパッケージでやってますが、これは何かというと main から数えて呼び出し階層の２番目くらいにあるサブパッケージです。

• main - 形式だけのトップレベル（frameとmainsを呼ぶだけ）
  • frame - Lua に依存しない部分の初期化処理と終了処理を担う
    • mains - Lua に依存する部分の初期化処理と終了処理とメインルーチン
      • shell - シェルのメインルーチン(loop.go)とコマンドインタープリタ(interpreter.go)
      • "go-readline-ny" - 一行入力のパッケージ（元々はサブパッケージだったが独立させた）

複雑な階層構造になってますが、これは Lua に依存する箇所を mains だけに限定したかったためです。これをしなければ、lua53.dll → GopherLua へのスムーズな切り替えが出来なかったと思います。

直っていない。

start nyagos.exe
more
[Ctrl-C]

で、ターミナルがクローズしてしまう。別のウインドウなので、親の nyagos が kill しているわけではない

more を実行する時に一時的に ENABLE_PROCESSED_INPUT をリセットして、Ctrl-C を押されても割り込みが発生しないようにした。

• (#342) Disable Ctrl-C while more is running · zetamatta/nyagos@51131a4

が、そうすると、キーボード入力の Enter が LF のない CR だけになってしまって、ずっと同じ行で入力し続ける形になっておかしい！
（が、落ちるよりマシなので、とりあえず commit してる。しかし、コレ、どうすりゃいいんだ？）

scoop/shims 以下にある実行ファイルは、scoop/apps 以下にある本物の EXE ファイルを呼ぶためのブースター。下記がソースだが、ぱっと見、子プロセスを kill するようなコードは見受けられない。プロセスツリーのトップが殺されたら連鎖的に子供も殺されるような仕組みでもあるのかなぁ

https://github.com/lukesampson/scoop/blob/master/supporting/shimexe/shim.cs

zenn.dev の Book 「scoop / nyagos で始めるコマンドライン生活」が ~\scoop/apps/nyagos.exe を使うような記載になっていたのを、~\scoop\apps\nyagos\current\nyagos.exe を指定するよう修正した。

• 「nyagos」のインストール
• Windows Terminal で nyagos を使おう

この記事が迷える人を増やす要因になっていた。もっと早く修正すべきだった。判断が遅い！

あー。うちの記事でも当時のサンプルコードをそのまま載せてました。直さなきゃ

https://text.baldanders.info/remark/2020/10/windows-terminal-and-nyagos-and-scoop/

お、これがベストソリューションだったかも！？

これが決定版かもしれぬ。more の中で Ctrl-C を押しても、more だけが止まって、その後に引きずらないようになった。

• (#342) more: stop with Ctrl-C with not SetConsoleMode but signal · zetamatta/nyagos@af54fda

ただし、signal.Notify～signal.Stop の範囲がまだ狭いので

• Ctrl-C kills nyagos.exe itself when more runs without redirect · Issue #342 · zetamatta/nyagos

で追加報告いただいている事象がこれでは治らない可能性が高い。
signal.Notify～signal.Stopの範囲の見直しする必要がありそうだ。

単純に channel をクローズしてはどうか？

package main

import (
	"time"
	"os/signal"
	"os"
)

func main(){
	c1 := make(chan os.Signal,1)
	c2 := make(chan os.Signal,1)

	go func(){
		for _ = range c1{
			println("first signal")
		}
	}()

	go func(){
		for _ = range c2{
			println("second signal")
		}
	}()

	signal.Notify(c1,os.Interrupt)
	signal.Notify(c2,os.Interrupt)

	println("Type Ctrl-C in 10 seconds")
	time.Sleep(time.Second*10)

	close(c1)

	println("Type Ctrl-C in 10 seconds")
	time.Sleep(time.Second*10)

	close(c2)
}

$ signalnotify.exe
Type Ctrl-C in 10 seconds
first signal
second signal
Type Ctrl-C in 10 seconds
panic: send on closed channel

goroutine 8 [running]:
os/signal.process(0x64ae00, 0x697710)
        C:/go/src/os/signal/signal.go:244 +0x17f
os/signal.loop()
        C:/go/src/os/signal/signal_unix.go:23 +0x45
created by os/signal.Notify.func1.1
        C:/go/src/os/signal/signal.go:150 +0x4b

ダメに決まってるだろ！

close ではなく、普通に signal.Stop を使ってみた。

	signal.Notify(c1,os.Interrupt)
	signal.Notify(c2,os.Interrupt)

	println("Type Ctrl-C in 10 seconds")
	time.Sleep(time.Second*10)

	signal.Stop(c2)

	println("Type Ctrl-C in 10 seconds")
	time.Sleep(time.Second*10)
	signal.Stop(c1)
	println("All done")

$ signalnotify.exe
Type Ctrl-C in 10 seconds
second signal
first signal
Type Ctrl-C in 10 seconds
first signal
All done

普通に行けた。signal.Notify は多重に機能する。

さて、全部の signal.Notify を解除した時の挙動は、何もしなかった時の状況に戻るわけだが、その時の挙動はどうか？

	signal.Notify(c1,os.Interrupt)
	signal.Notify(c2,os.Interrupt)

	println("Type Ctrl-C in 5 seconds")
	time.Sleep(time.Second*5)

	signal.Stop(c2)

	println("Type Ctrl-C in 5 seconds")
	time.Sleep(time.Second*5)
	signal.Stop(c1)

	println("Type Ctrl-C in 5 seconds")
	time.Sleep(time.Second*5)

	println("All done")

$ signalnotify.exe
Type Ctrl-C in 5 seconds
second signal
first signal
Type Ctrl-C in 5 seconds
first signal
Type Ctrl-C in 5 seconds

3回 Ctrl-C を押下した時、Notify をすべて解除した時はプロセスが殺されている（ All done が表示されない )。

では、最初に signal.Ignore を設定してはどうか？

	signal.Ignore(os.Interrupt)
	signal.Notify(c1, os.Interrupt)
	signal.Notify(c2, os.Interrupt)

	println("Type Ctrl-C in 5 seconds")
	time.Sleep(time.Second * 5)

	signal.Stop(c2)

	println("Type Ctrl-C in 5 seconds")
	time.Sleep(time.Second * 5)
	signal.Stop(c1)

	println("Type Ctrl-C in 5 seconds")
	time.Sleep(time.Second * 5)

	println("All done")

$ signalnotify.exe
Type Ctrl-C in 5 seconds
second signal
first signal
Type Ctrl-C in 5 seconds
first signal
Type Ctrl-C in 5 seconds

うん、ダメっぽい！

signal.Notify が一つでも残っていれば Ctrl-C で強制停止されないわけだから、まぁ、大丈夫と見てよいかな…

（一つも signal.Notify が設定されていない状態というのは、シェルのメイン処理ループ外にいるということなので。そういうタイミングはほとんどない）

処理しない signal.Notify を用意してもいいけど、そちらの channel が Ctrl-C で満タンになると panic になるので、消し込むためだけの goroutine が必要になって、もったいない。

いけてそうな感じではあるけれども、疑似テスト（自分で、~/scoop/app/nyagos/4.4.8_0/nyagos を差し替えた）結果なので、まぁ、直ったとは断言できぬ状況

こちらについては [Go]os.StdinのEOFの後に改行が必ず入ってきてしまう件 にて検討します。
（more がエコーバックしない件も別にすればよかった…）