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

Add all Ubuntu EOL distros back to boxturtle to old release template #637

Merged

Conversation

mikaelarguedas
Copy link
Contributor

Modify the old_release_set.Dockerfile.em template to behave as expected for all distros listed in the ROS snapshot repository.

Original motivation for this PR is to allow to reuse this template for the generation of the official docker images.

Related to osrf/docker_images#273 and osrf/docker_templates#65

Arguably this does not necessarily belong in the ros_buildfarm package as no buildfarm will be ran against these distros. If so a copy of this snippet can be hosted on https://github.com/osrf/docker_templates but would result in duplicating maintenance effort

@dirk-thomas
Copy link
Member

Can you elaborate why this is necessary / useful? Those old distributions are partially even using rosbuild instead of catkin. Is there any particular distribution you are trying to / actively run a buildfarm for?

@mikaelarguedas
Copy link
Contributor Author

Is there any particular distribution you are trying to / actively run a buildfarm for?

As per original description I am not and dont know of anyone who would try to run a buildfarm for these distributions:

The only reason for submitting it here is to avoid duplicating and maintaining the same template in the osrf/docker_templates repository (docker_templates tries to reuse as much as possible the snippets maintained in ros_buildfarm).
The change itself is needed to be able to expand this template for all ubuntu distros having packages in the snapshot.ros.org repository.

As mentioned in original description, as this change is not really buildfarm related, I'm happy to consider another place to store such template if it is preferred

@dirk-thomas
Copy link
Member

If the plan is to build Docker images for all EOL distros and reuse this snippet that sounds fine with me.

Can you please add a comment above the condition mentioning the Docker images. Otherwise I would be worried this get truncated in the near future since the use is not obvious.

@mikaelarguedas
Copy link
Contributor Author

Can you please add a comment above the condition mentioning the Docker images. Otherwise I would be worried this get truncated in the near future since the use is not obvious.

Comment added in a3a8ea3

@dirk-thomas
Copy link
Member

Thanks.

@dirk-thomas dirk-thomas merged commit 0ca4e94 into ros-infrastructure:master Jun 6, 2019
@mikaelarguedas mikaelarguedas deleted the ubuntu_eol_distros branch June 6, 2019 23:36
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