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

Remove outdated hack for ament_lint #411

Closed
wants to merge 2 commits into from
Closed

Conversation

tfoote
Copy link
Contributor

@tfoote tfoote commented Aug 31, 2022

This is causing CI to fail because the referenced patch has been merged and the branch deleted.

ament/ament_lint#268

This is causing CI to fail because the referenced patch has been merged and the branch deleted.

Signed-off-by: Tully Foote <tfoote@osrfoundation.org>
Need ros-tooling/action-ros-ci#430

Signed-of-by: Tully Foote <tfoote@osrfoundation.org>
@tfoote
Copy link
Contributor Author

tfoote commented Aug 31, 2022

Also added a necessary action-ros-ci upgrade for ros-tooling/action-ros-ci#430

Run ros-tooling/action-ros-ci@0.0.19
Invoking "bash -c 'rosdep update'
Invoking "bash -c 'curl --fail --silent --show-error --location 'https://raw.githubusercontent.com/ros2/ros2/foxy/ros2.repos' | vcs import --force --recursive src/'
Invoking "bash -c 'curl --fail --silent --show-error --location 'https://raw.githubusercontent.com/ros2/rmw_cyclonedds/3be1c[9](https://github.com/ros2/rmw_cyclonedds/runs/8119818947?check_suite_focus=true#step:6:10)d21ccd98e8e93ff8788dbf93e93a331622/.github/resources/local.repos' | vcs import --force --recursive src/'
Invoking "bash -c 'find "/Users/runner/work/rmw_cyclonedds/rmw_cyclonedds/ros_ws" -type d -and -name "rmw_cyclonedds" | xargs rm -rf'
Invoking "bash -c 'vcs import --force --recursive src/ < package.repo'
Invoking "bash -c 'diff --new-line-format="" --unchanged-line-format="" <(colcon list -p) <(colcon list --packages-up-to rmw_cyclonedds_cpp rmw_implementation -p) | xargs rm -rf'
Invoking "bash -c 'DEBIAN_FRONTEND=noninteractive RTI_NC_LICENSE_ACCEPTED=yes rosdep install -r --from-paths src --ignore-src --rosdistro foxy -y || true'
Error: Unable to process command '::add-path::/Users/runner/work/rmw_cyclonedds/rmw_cyclonedds/ros_ws/install/bin' successfully.
Error: The `add-path` command is disabled. Please upgrade to using Environment Files or opt into unsecure command execution by setting the `ACTIONS_ALLOW_UNSECURE_COMMANDS` environment variable to `true`. For more information see: https://github.blog/changelog/2020-10-01-github-actions-deprecating-set-env-and-add-path-commands/
Invoking "bash -c 'colcon build --event-handlers console_cohesion+ 			--packages-up-to rmw_cyclonedds_cpp rmw_implementation 			 			--cmake-args  --symlink-install'
Invoking "bash -c 'colcon lcov-result --initial'
Invoking "bash -c 'colcon test --event-handlers console_cohesion+ 			--pytest-with-coverage --return-code-on-test-failure 			--packages-select rmw_cyclonedds_cpp rmw_implementation 			'
Error: The process '/bin/bash' failed with exit code 1

@tfoote
Copy link
Contributor Author

tfoote commented Sep 1, 2022

Replaced by #414

@tfoote tfoote closed this Sep 1, 2022
@tfoote tfoote deleted the tfoote-fix-foxy-ci branch September 1, 2022 18:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant