Skip to content
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

[roslaunch] --ros-args option fails in noetic #1971

Closed
xaver-k opened this issue Jun 12, 2020 · 1 comment · Fixed by #1973
Closed

[roslaunch] --ros-args option fails in noetic #1971

xaver-k opened this issue Jun 12, 2020 · 1 comment · Fixed by #1973
Labels

Comments

@xaver-k
Copy link
Contributor

xaver-k commented Jun 12, 2020

Short description

Running roslaunch with the option --ros-args under ROS noetic fails.

Expected behavior

Print the arguments of the launch file like under ROS melodic.
As an example, getting the arguments of roslaunch's own example launch-file via

docker run --rm -it ros:melodic-robot roslaunch roslaunch example.launch --ros-args

gives

Optional Arguments:
  commandline_true_arg (default "true"): undocumented

Acutual behavior

Running any of the below commands

docker run --rm -it ros:noetic-robot roslaunch roslaunch example.launch --ros-args
docker run --rm -it ros:noetic-robot roslaunch --ros-args roslaunch example.launch
docker run --rm -it ros:noetic-robot roslaunch /opt/ros/noetic/share/roslaunch/resources/example.launch --ros-args
docker run --rm -it ros:noetic-robot roslaunch --ros-args /opt/ros/noetic/share/roslaunch/resources/example.launch

Gives

Traceback (most recent call last):
  File "/opt/ros/noetic/lib/python3/dist-packages/roslaunch/__init__.py", line 275, in main
    import arg_dump as roslaunch_arg_dump
ModuleNotFoundError: No module named 'arg_dump'

Thoughts

I suspect that

import arg_dump as roslaunch_arg_dump

should be

from . import arg_dump as roslaunch_arg_dump 

because of the switch to python 3.

@dirk-thomas
Copy link
Member

Please create a pull request with the proposed change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants