Use snapshot of rosdep list on melodic #457
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
After ros/rosdistro#39174 was merged, rosdep key
python-omniorb
cannot be resolved on melodic.This makes an error on
rosdep install
without-r
option:https://app.travis-ci.com/github/start-jsk/jsk_apc/jobs/615633458
This PR fixes that error by using snapshot of rosdep list on melodic as well as kinetic and under.
Result:
https://app.travis-ci.com/github/start-jsk/jsk_apc/jobs/616059276
Of course, adding
-r
option torosdep install
is another solution, but I think it is good time to use snapshot because melodic is EOL and no longer maintained on rosdistro:ros/rosdistro#39223 (comment)
Minor fixes