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

Revert "xacro: 1.14.11-1 in 'noetic/distribution.yaml' [bloom]" #32139

Closed
wants to merge 1 commit into from

Conversation

sloretz
Copy link
Contributor

@sloretz sloretz commented Feb 11, 2022

Reverts #31860

@rhaschke While options are being discussed in ros/xacro#300, what do you think about reverting to the previous release? I can do a sync over the weekend.

@sloretz sloretz self-assigned this Feb 11, 2022
@github-actions github-actions bot added the noetic Issue/PR is for the ROS 1 Noetic distribution label Feb 11, 2022
@rhaschke
Copy link
Contributor

Reverting to 1.14.10 might be the quickest bet. However, I plan to implement and test a fixup tomorrow. So, if this still suffices for a sync, I suggest to wait, because 1.14.11 includes a fixup for a regression introduced already in 1.14.10 😢

@sloretz
Copy link
Contributor Author

sloretz commented Feb 11, 2022

However, I plan to implement and test a fixup tomorrow. So, if this still suffices for a sync, I suggest to wait, because 1.14.11 includes a fixup for a regression introduced already in 1.14.10

Got it. I'll start a sync freeze now in anticipation of the xacro release tomorrow.

@sloretz sloretz closed this Feb 11, 2022
@sloretz sloretz deleted the revert-31860-bloom-xacro-1 branch February 11, 2022 22:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
noetic Issue/PR is for the ROS 1 Noetic distribution
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants