Skip to content

Renaming debian builds #642

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

Merged
merged 1 commit into from
Jun 3, 2019
Merged

Renaming debian builds #642

merged 1 commit into from
Jun 3, 2019

Conversation

yassingh
Copy link
Contributor

Replacing underscore with dash to make debian build name similar to other os name.

I should have pushed this change in #638
But I caught this discrepancy in build names later.

@yassingh yassingh requested a review from basepi May 31, 2019 13:51
Copy link
Collaborator

@basepi basepi left a comment

Choose a reason for hiding this comment

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

My brain is trying to tell me there was an issue around underscores vs dashes in Debian. But I can't for the life of me find it -- as far as I can tell we've been using underscores since we first had Debian package support. Maybe I was using underscores for the RPMs and that was the issue? I'm not sure.

Will this affect Debian's willingness to upgrade from an _ to a - version? Have you tested the upgrade path?

@fossam
Copy link
Contributor

fossam commented Jun 1, 2019

Debian didn't like underscores in their version strings (https://www.debian.org/doc/debian-policy/ch-controlfields.html#version). We had this in #527. But issue here is different because it's about package file names. Naming standard for this aspect is at https://www.debian.org/doc/manuals/debian-reference/ch02.en.html#_debian_package_file_names .

@yassingh
Copy link
Contributor Author

yassingh commented Jun 3, 2019

+1 with @fossam
There was an issue because of using underscore in "version" string of debian packages.
I hope that @basepi is also referring to this issue only.

Moreover, even the current debian builds are named in the same way as I am proposing ( debian build names are also using dashes here https://github.com/hubblestack/hubble/releases/tag/v2.4.7 ).

Earlier I used to rename the debain builds while uploading them.
With this PR, the debian builds will be renamed as soon as they are created inside the docker container.
So, this path change has already been tested by all of us.

Copy link
Collaborator

@basepi basepi left a comment

Choose a reason for hiding this comment

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

Ah, understood. Thanks for the explanation!

@basepi basepi merged commit bc3e316 into hubblestack:3.0 Jun 3, 2019
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.

3 participants