Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix script error when there is no default ros installation and the ro… #113

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

SylvainPastor
Copy link

…s_path argument is used:

source ./build_ros2_workspace.bash --ros_path /home/sylvain/work/ros/foxy/install/setup.bash
./build_ros2_workspace.bash: line 101: /opt/ros/foxy/setup.bash: No such file or directory

2 consecutive calls to the "source":
- 1st line 91: source $ros_path (ROS2 environment from arg)
- 2nd line 101: source /opt/ros/$ROS_DISTRO/setup.bash
In this second case, if there is no ROS installation, the script fails.

…s_path argument is used:

  source ./build_ros2_workspace.bash --ros_path /home/sylvain/work/ros/foxy/install/setup.bash
  ./build_ros2_workspace.bash: line 101: /opt/ros/foxy/setup.bash: No such file or directory

  2 consecutive calls to the "source":
    - 1st line 91:  source $ros_path (ROS2 environment from arg)
    - 2nd line 101: source /opt/ros/$ROS_DISTRO/setup.bash
      In this second case, if there is no ROS installation, the script fails.
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant