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

ament_cmake_vendor_package: Replace 'git' dep with 'vcstool' #462

Merged
merged 1 commit into from Jun 16, 2023

Conversation

cottsay
Copy link
Contributor

@cottsay cottsay commented Jun 15, 2023

At some point in development, I switched from 'git' to 'vcstool' for fetching sources and forgot to update the manifest.

Should resolve the Rpr failure in ros2/rviz#995.

At some point in development, I switched from 'git' to 'vcstool' for
fetching sources and forgot to update the manifest.

Signed-off-by: Scott K Logan <logans@cottsay.net>
@cottsay cottsay added the bug Something isn't working label Jun 15, 2023
@cottsay cottsay self-assigned this Jun 15, 2023
@cottsay cottsay requested a review from mjeronimo as a code owner June 15, 2023 22:18
@cottsay
Copy link
Contributor Author

cottsay commented Jun 15, 2023

The Rpr job should cover the syntax validation for this manifest change. Rosdeps are ignored on ci.ros2.org, so I don't plan to run any jobs there.

@cottsay cottsay merged commit 448647d into rolling Jun 16, 2023
3 checks passed
@delete-merged-branch delete-merged-branch bot deleted the cottsay/ament_vendor_vcstool branch June 16, 2023 17:05
@cottsay
Copy link
Contributor Author

cottsay commented Jun 16, 2023

@Mergifyio backport humble

@mergify
Copy link

mergify bot commented Jun 16, 2023

backport humble

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request Jun 16, 2023
At some point in development, I switched from 'git' to 'vcstool' for
fetching sources and forgot to update the manifest.

Signed-off-by: Scott K Logan <logans@cottsay.net>
(cherry picked from commit 448647d)

# Conflicts:
#	ament_cmake_nose/package.xml
@cottsay
Copy link
Contributor Author

cottsay commented Jun 16, 2023

@Mergifyio backport iron

@mergify
Copy link

mergify bot commented Jun 16, 2023

backport iron

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request Jun 16, 2023
At some point in development, I switched from 'git' to 'vcstool' for
fetching sources and forgot to update the manifest.

Signed-off-by: Scott K Logan <logans@cottsay.net>
(cherry picked from commit 448647d)
cottsay added a commit that referenced this pull request Jun 20, 2023
…464)

At some point in development, I switched from 'git' to 'vcstool' for
fetching sources and forgot to update the manifest.

Signed-off-by: Scott K Logan <logans@cottsay.net>
(cherry picked from commit 448647d)

Co-authored-by: Scott K Logan <logans@cottsay.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants