Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

expect not properly spawning Stockfish in github actions. #5185

Open
vondele opened this issue Apr 21, 2024 · 0 comments
Open

expect not properly spawning Stockfish in github actions. #5185

vondele opened this issue Apr 21, 2024 · 0 comments

Comments

@vondele
Copy link
Member

vondele commented Apr 21, 2024

CI actually doesn't run correctly, i.e. in the actions some tests are effectively not run. It is specific to running it in the actions the same test locally seems to work. In CI we have this debug output:

 spawn ./stockfish
parent: waiting for sync byte
parent: telling child to go ahead
parent: now unsynchronized from child
spawn: returns {3784}

expect: does "" (spawn_id exp3) match glob pattern "Stockfish"? no
expect: read eof
expect: set expect_out(spawn_id) "exp0"
expect: set expect_out(buffer) ""
instrumented testing failed on line 296

compared to locally:

spawn ./stockfish
parent: waiting for sync byte
parent: telling child to go ahead
parent: now unsynchronized from child
spawn: returns {16146}

expect: does "" (spawn_id exp4) match glob pattern "Stockfish"? no
Stockfish dev-20240413-c55ae376 by the Stockfish developers (see AUTHORS file)
expect: does "Stockfish dev-20240413-c55ae376 by the Stockfish developers (see AUTHORS file)\r\n" (spawn_id exp4) match glob pattern "Stockfish"? yes
expect: set expect_out(0,string) "Stockfish"
expect: set expect_out(spawn_id) "exp4"
expect: set expect_out(buffer) "Stockfish"
send: sending "uci\n" to { exp4 }
...

My interpretation of this is that in CI expect spawns stockfish but it (or whatever pipe that connects expect to stockfish) closes on startup, leading to eof. While locally there is no issue. An additional tests shows that expect can spawn things (e.g. using cat instead of stockfish works) also in CI.

Originally posted by @vondele in #5181 (comment)

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

No branches or pull requests

1 participant