lestrrat-go / server-starter

Go port of start_server utility (Server::Starter)
MIT License
215 stars 21 forks source link

Not using `OptKillOldDelay` and `OptEnableAutoRestart` option? #15

Closed sioncojp closed 7 years ago

sioncojp commented 7 years ago

https://github.com/lestrrat/go-server-starter/blob/master/starter.go#L439..L448

I see that OptKillOldDelay and OptEnableAutoRestart doesn't use because there has used Environment variable.

What there should be use option value?

lestrrat commented 7 years ago

sorry, that makes no sense:

I see that OptKillOldDelay and OptEnableAutoRestart doesn't use because there has used Environment variable.

日本語でどうぞー

sioncojp commented 7 years ago

すみません。

--kill-old-delay=secondsオプションが使われてない(その代わり環境変数でoverwrite)ように見受けられたので、不要ではないかと思いました。

--enable-auto-restartオプションはUNIMPLEMENTEDと書かれてましたので解決しました。失念しており、申し訳ないです。

lestrrat commented 7 years ago

そのあたりは #11 でやった気がします。ただし、テストが進んで無くてマージしてない感じです。 もし確認してもらえるなら最高です!

sioncojp commented 7 years ago

ご回答ありがとうございます! 時間あるときに一度確認してみます〜。