-
Notifications
You must be signed in to change notification settings - Fork 561
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
re-enable runall tests under CTest #120
Comments
From [email protected] on October 08, 2009 11:20:23 in r226 :
|
From [email protected] on March 06, 2014 13:44:01 To get started on this, see all of the "issue #120" comments in suite/tests/CMakeLists.txt. Owner: --- |
This was referenced Nov 28, 2014
derekbruening
added a commit
that referenced
this issue
Feb 20, 2018
Changes the libutil nudge code to use NtCreateThreadEx on Win8+. For a test, enables the client.nudge_test test, part of i#120: enable .runall tests on Windows. Here I added a new app win32.infloop to parallel linux.infloop, rather than launching calc.exe or notepad.exe like we did in the past (not yet tackling systemwide injection here). It has a MessageBox with a title containing the pid, allowing us to use tools/closewnd with a unique target name for a clean and race-free exit. win32.infloop also has a 3-minute timeout to avoid leaving stale processes behind in case of issues closing it externally. Revamped runall.cmake to work on Windows, using "ping" to sleep, updating nudge and close commands, removing stale pid files, etc. Ported suite code to use run_in_bg on Windows as well as Linux, but with the pidfile coming from drrun. Fixes #1309 Fixes #1432 Issue: #120, #1835
derekbruening
added a commit
that referenced
this issue
Feb 20, 2018
Changes the libutil nudge code to use NtCreateThreadEx on Win8+. For a test, enables the client.nudge_test test, part of i#120: enable .runall tests on Windows. Here I added a new app win32.infloop to parallel linux.infloop, rather than launching calc.exe or notepad.exe like we did in the past (not yet tackling systemwide injection here). It has a MessageBox with a title containing the pid, allowing us to use tools/closewnd with a unique target name for a clean and race-free exit. win32.infloop also has a 3-minute timeout to avoid leaving stale processes behind in case of issues closing it externally. Revamped runall.cmake to work on Windows, using "ping" to sleep, updating nudge and close commands, removing stale pid files, etc. Ported suite code to use run_in_bg on Windows as well as Linux, but with the pidfile coming from drrun. Fixes #1309 Fixes #1432 Issue: #120, #1835
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
From [email protected] on April 12, 2009 23:12:23
splitting from issue #65 This case covers writing a CMake script to replace the old runall shell script.
Now that we have 64-bit tools ( issue #28 ) we should be able to have both
32-bit and 64-bit runall tests.
Xref issue #111 where we want to support runall tests in parallel.
Original issue: http://code.google.com/p/dynamorio/issues/detail?id=120
The text was updated successfully, but these errors were encountered: