Skip to content

Conversation

@mattn
Copy link
Contributor

@mattn mattn commented Mar 17, 2015

No description provided.

@mattn
Copy link
Contributor Author

mattn commented Mar 17, 2015

Ah, I notice all of signals doesn't work on windows. So please wait.

@mattn
Copy link
Contributor Author

mattn commented Mar 17, 2015

Hmm, for terminating process on windows, should implement like below.

https://github.com/mattn/goemon/blob/master/proc_windows.go#L48-L60

Let's separate PRs. So lease for you. Please take a look.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 93.02% when pulling f738282 on mattn:windows into cde00ec on Songmu:master.

@Songmu
Copy link
Owner

Songmu commented Mar 17, 2015

pull requestありがとうございます!

このパッチでとりあえずwindowsでビルドはできるようになりますが、プロセスハンドリングの方法がwindows特有のものがあるので、取り込んだとしてもwindowsではちゃんと動かないという理解であってますでしょうか?

@mattn
Copy link
Contributor Author

mattn commented Mar 17, 2015

はい。なのでいま迷ってるんです。goemon の様に interrupt はやれない事ないので

  • INT だけ生かして SIGINT は goemon の interrupt と同じ様に扱う(他は実行不可)
  • そもそもWindowsは外部からシグナルを送れるという仕組みがないので全て殺す

のどちらにしようかなーと考えている所です。
現状は、Process.Signal(x)しても syscall.EWINDOWS が返ってるはずです。(エラーは見てない)

@Songmu
Copy link
Owner

Songmu commented Mar 18, 2015

了解です。では、このp-rはとりあえず取り込ませてもらうので、後続のp-rをお待ちしています。

Songmu added a commit that referenced this pull request Mar 18, 2015
@Songmu Songmu merged commit a3e8030 into Songmu:master Mar 18, 2015
@mattn mattn deleted the windows branch March 18, 2015 09:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants