Skip to content

Update the catkin release test to use melodic.#510

Merged
nuclearsandwich merged 1 commit intomasterfrom
test-on-melodic
Jan 23, 2019
Merged

Update the catkin release test to use melodic.#510
nuclearsandwich merged 1 commit intomasterfrom
test-on-melodic

Conversation

@nuclearsandwich
Copy link
Copy Markdown
Contributor

With ros-infrastructure/rosdep#647 rosdistros
that are no longer supported don't have rosdep updates fetched.

The system tests relied on groovy updates from rosdep. I've moved the
system test to melodic. I chose melodic rather than indigo or kinetic
because indigo will enter end-of-support later this year and Kinetic is
currently supported on only one Debian/Ubuntu distro. Melodic is still
supported on Bionic and Stretch.

Fixes #509 (hopefully)

With ros-infrastructure/rosdep#647 rosdistros
that are no longer supported don't have rosdep updates fetched.

The system tests relied on groovy updates from rosdep. I've moved the
system test to melodic. I chose melodic rather than indigo or kinetic
because indigo will enter end-of-support later this year and Kinetic is
currently supported on only one Debian/Ubuntu distro. Melodic is still
supported on Bionic and Stretch.
@cottsay
Copy link
Copy Markdown
Member

cottsay commented Jan 23, 2019

Would it be a good idea to add --include-eol-distros to the rosdep update line in the .travis.yml as well, so this doesn't need to happen in the future?

@nuclearsandwich
Copy link
Copy Markdown
Contributor Author

Would it be a good idea to add --include-eol-distros to the rosdep update line in the .travis.yml as well, so this doesn't need to happen in the future?

It'd be curious to see if that's sufficient. But I think the issue is happening in the rosdep updates triggered by bloom itself. I'm not sure off the top of my head if those are done via API or a subprocess.

@nuclearsandwich nuclearsandwich merged commit f079968 into master Jan 23, 2019
@nuclearsandwich nuclearsandwich deleted the test-on-melodic branch January 23, 2019 20:22
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