-
Notifications
You must be signed in to change notification settings - Fork 107
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
change test runner to work on windows #14
Conversation
c275287
to
d65c274
Compare
Lgtm, but why was so much code removed in |
cc8cf4b
to
f042c2e
Compare
Ups, the copyright stuff was not intended to be in that commit. I updated the PR to exclude the copyright stuff. |
+1 |
foreach(arg ${ARG_UNPARSED_ARGUMENTS}) | ||
set(subcmd "${subcmd} \"${arg}\"") | ||
endforeach() | ||
set(cmd "${ament_copyright_BIN}" "--xunit-file" "${AMENT_TEST_RESULTS_DIR}/${PROJECT_NAME}/${ARG_TESTNAME}.xml") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No PYTHON_EXECUTABLE on this line? All the others have it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, it is an extension point which will generate an executable on Windows. And the command didn't use PYTHON_EXECUTABLE before this PR.
+1 |
1 similar comment
+1 |
change test runner to work on windows
change test runner to work on windows
Depends on ament/ament_cmake#7.
Connects to ament/ament_cmake#7