Skip to content

Change robot_model from indigo-devel branch to jade-devel tag#15927

Merged
sloretz merged 1 commit intoros:masterfrom
sloretz:freeze_robot_model_jade
Sep 15, 2017
Merged

Change robot_model from indigo-devel branch to jade-devel tag#15927
sloretz merged 1 commit intoros:masterfrom
sloretz:freeze_robot_model_jade

Conversation

@sloretz
Copy link
Copy Markdown
Contributor

@sloretz sloretz commented Sep 15, 2017

The purpose of this is to not break rosinstall_generator --upstream-development for the EOL distro jade once the package joint_state_publisher is removed from the indigo-devel branch. This changes the version for the repository robot_model from the branch indigo-devel to a tag jade-devel. The tag is set to the state of the repo when the last jade sync happened.

See #15926
See ros/robot_model#195

@mikaelarguedas
Copy link
Copy Markdown
Member

looks good to me, thanks for the follow-up

@mikaelarguedas
Copy link
Copy Markdown
Member

Note that the job will fail because we dont allow modifications in eol distros, but it's ok to merge anyway

@sloretz
Copy link
Copy Markdown
Contributor Author

sloretz commented Sep 15, 2017

Note to future self, this failed because the source entry must be a branch and not a tag (thanks @dirk-thomas). I'll delete the jade-devel tag on robot_model and make it a branch.

ERROR: In file '''/home/travis/build/ros/rosdistro/jade/distribution.yaml''': Could not validate source entry for repo robot_model with error [[[Could not validate repository with url https://github.com/ros/robot_model.git and version jade-devel from entry at line '''5294''']]]

@sloretz
Copy link
Copy Markdown
Contributor Author

sloretz commented Sep 15, 2017

Failed with only the expected failure:

ERROR: In file '''/home/travis/build/ros/rosdistro/jade/distribution.yaml''': There is a change to a doc section of an EOLed distribution. Lines: [5283]

so I'm going to squash/merge this.

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.

2 participants