Skip to content

Conversation

@arukuka
Copy link
Member

@arukuka arukuka commented Nov 21, 2017

fix #33

@arukuka
Copy link
Member Author

arukuka commented Nov 21, 2017

./official/official samples/sample-course.smrjky player/greedy 1 player/greedy 2 > out.json --stdinLogFile0 stdin0.txt --stdinLogFile1 stdin1.txt --stderrLogFile0 stderr0.txt --stderrLogFile1 stderr1.txt --pauseP0 "echo pause0 >> pause.txt" --pauseP1 "echo pause1 >> pause.txt" --resumeP0 "echo resume0 >> resume.txt" --resumeP1 "echo resume1 >> resume.txt"

上記コマンドで確認をしました。

@arukuka
Copy link
Member Author

arukuka commented Nov 21, 2017

前のバージョンと違うことを確認しようとしたのですが、
変更前も意図通りの動作になってますね…(mingw64環境です)。
Ubuntu 17.10 機でも確認しようと思います。

@arukuka
Copy link
Member Author

arukuka commented Nov 21, 2017

前バージョンでは意図した動作をせず、
今回のコミット( 72bd201 )で意図した動作になることを
Ubuntu 17.10 で確認しました。

ただ、実行したコマンドの標準出力がそのまま流れているため、
ゲームログを汚染していることに気づきましたので修正します。
これは、stdout を stderr にリダイレクトすることで対応しようと思います。

@arukuka
Copy link
Member Author

arukuka commented Nov 21, 2017

修整しました。
travis でテストを書いてないのであまり意味がないですが、
passしたら(コンパイルが通ったら)self-mergeしようと思います。

@arukuka arukuka merged commit 4d94e8e into master Nov 21, 2017
@arukuka arukuka deleted the launch_shell branch November 21, 2017 10:16
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.

[official] pause, resume commands are not invoked as we planned

2 participants