-
Notifications
You must be signed in to change notification settings - Fork 100
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
Windows messages when atomic type is unsupported #145
Conversation
Signed-off-by: Shane Loretz <[email protected]>
Signed-off-by: Shane Loretz <[email protected]>
Signed-off-by: Shane Loretz <[email protected]>
Signed-off-by: Shane Loretz <[email protected]>
Signed-off-by: Shane Loretz <[email protected]>
Signed-off-by: Shane Loretz <[email protected]>
Not sure what I was thinking with the |
Signed-off-by: Shane Loretz <[email protected]>
ab9ea67 removed the |
Only |
This PR uses a default value if
ROS_PACKAGE_NAME
is not set, which appears to be the case forrclpy
.Also it adds a compile time message when an atomic type is not supported in addition to the existing runtime error.(Edit, second part makes no sense. See comment below)From https://ci.ros2.org/job/ci_windows/6471/consoleText