Skip to content

Commit

Permalink
Fix ninja_handle_control_c_rebuild for windows. The stderr on windows…
Browse files Browse the repository at this point in the history
… doesn't match other platforms, but it should be sufficient to verify that build.ninja file still exists
  • Loading branch information
bdbaddog committed Nov 4, 2021
1 parent d187236 commit f845a94
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions test/ninja/ninja_handle_control_c_rebuild.py
Original file line number Diff line number Diff line change
Expand Up @@ -67,8 +67,11 @@
# run ninja independently
program = test.workpath('run_ninja_env.bat') if IS_WINDOWS else ninja_bin
test.run(program=program, stdout=None, stderr=None, status=1)
test.must_contain_all(
test.stderr(), "ninja: error: rebuilding 'build.ninja': interrupted by user")

if not IS_WINDOWS:
error_msg = "ninja: error: rebuilding 'build.ninja': interrupted by user"
test.must_contain_all(
test.stderr(), error_msg )

# Verify that Rebuilding build.ninja and sending control-c to ninja doesn't remove build.ninja
test.must_exist('build.ninja')
Expand Down

0 comments on commit f845a94

Please sign in to comment.