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

Move debian-specific release templates to a subdirectory #717

Merged
merged 2 commits into from
Jan 28, 2020

Conversation

cottsay
Copy link
Member

@cottsay cottsay commented Jan 23, 2020

Each of these templates includes a debian-specific approach that doesn't apply to RPMs. The templates remaining in the templates/release directory don't appear to be debian-specific and should be usable for RPMs as well.

In some locations (such as release_job.py) it will be necessary to select a template based on the packaging format of the target OS. Organizing the templates in subdirectories like this will give us an easy way to select the template that serves a specific purpose for a given packaging format. This is why some of the template file names were also renamed to be more neutral.

Since only the template files are moving, there shouldn't be any problem with the deployed Jenkins jobs getting out-of-sync with the current master of ros_buildfarm in regard to this change.

Each of these templates includes a debian-specific approach that doesn't
apply to RPMs. The templates remaining in the `templates/release`
directory don't appear to be debian-specific and should be usable for
RPMs as well.

In some locations (such as `release_job.py`) it will be necessary to
select a template based on the packaging format of the target OS.
Organizing the templates in subdirectories like this will give us an
easy way to select the template that serves a specific purpose for a
given packaging format. This is why some of the template file names were
also renamed to be more neutral.
@dirk-thomas
Copy link
Member

Please keep the order of the nodes in dot since changing it might affect the rendering.

Copy link
Member

@dirk-thomas dirk-thomas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comparing the old / new images they look the same.

@cottsay cottsay merged commit 4d88f57 into master Jan 28, 2020
@cottsay cottsay deleted the templates_release_deb branch January 28, 2020 06:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants