-
Notifications
You must be signed in to change notification settings - Fork 160
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
Improve ros2 doctor on Windows #631
Conversation
* Print an useful error message when failing to import rosdistro. * Print a clearer error message when failing to find either a loopback or no loopback interface on Windows. * Print "latest version" instead of "required version". * Print the corrent name of the python executable according to the platform. Signed-off-by: Ivan Santiago Paunovic <ivanpauno@ekumenlabs.com>
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.
A couple of comments in here.
Signed-off-by: Ivan Santiago Paunovic <ivanpauno@ekumenlabs.com> Co-authored-by: Chris Lalancette <clalancette@openrobotics.org>
Signed-off-by: Ivan Santiago Paunovic <ivanpauno@ekumenlabs.com> Co-authored-by: Christophe Bedard <bedard.christophe@gmail.com>
Signed-off-by: Ivan Santiago Paunovic <ivanpauno@ekumenlabs.com>
Signed-off-by: Ivan Santiago Paunovic <ivanpauno@ekumenlabs.com>
Signed-off-by: Ivan Santiago Paunovic <ivanpauno@ekumenlabs.com>
Signed-off-by: Ivan Santiago Paunovic <ivanpauno@ekumenlabs.com>
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.
This looks a lot simpler :) LGTM !
@ros-pull-request-builder retest this please |
@Mergifyio backport galactic |
Improve ros2 doctor on Windows: * Print a more useful error message when failing to load plugin * Print a clearer error message when failing to find either a loopback or no loopback interface on Windows. * Print "latest version" instead of "required version". * Print the corrent name of the python executable according to the platform. Signed-off-by: Ivan Santiago Paunovic <ivanpauno@ekumenlabs.com> Co-authored-by: Chris Lalancette <clalancette@openrobotics.org> Co-authored-by: Christophe Bedard <bedard.christophe@gmail.com> (cherry picked from commit 3803d77)
Command
|
Improve ros2 doctor on Windows: * Print a more useful error message when failing to load plugin * Print a clearer error message when failing to find either a loopback or no loopback interface on Windows. * Print "latest version" instead of "required version". * Print the corrent name of the python executable according to the platform. Signed-off-by: Ivan Santiago Paunovic <ivanpauno@ekumenlabs.com> Co-authored-by: Chris Lalancette <clalancette@openrobotics.org> Co-authored-by: Christophe Bedard <bedard.christophe@gmail.com> (cherry picked from commit 3803d77) Co-authored-by: Ivan Santiago Paunovic <ivanpauno@ekumenlabs.com>
The output of
ros2 doctor
on Windows was confusing.See the comment I wrote in the Galactic testing matric for more context.
rosdistro
python package is not usually installed on Windows, so having a nicer message when not found is helpful.I updated "latest version" to "required version" because the warning shows that for that ros distribution there's a newer version of the package available in rosdistro, it's not really a required version.