-
Notifications
You must be signed in to change notification settings - Fork 145
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
Adds packaging for Fedora 27 #59
Conversation
Signed-off-by: Eli Uriegas <[email protected]>
@@ -0,0 +1,17 @@ | |||
FROM fedora:27 | |||
RUN dnf -y upgrade | |||
RUN dnf install -y @development-tools fedora-packager |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we follow best-practice here, and combine dnf -y upgrade
with the dnf install
lines? (I guess we don't use caching, so probably doesn't make a big difference)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah these aren't cached when we build packages since we build on freshly spawned slaves in the Jenkins setup, WDYT @andrewhsu?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's just a nit, definitely no blocker for this PR
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM assuming the builds work 😜
Can confirm I was able to successfully build packages after the merge of docker-library/official-images#3708 |
Please also PR the docs when this is available. Would do it myself, but I don't know if F24 is still supported (and thus might need to be removed). |
Fedora 27 has hit GA! https://fedoramagazine.org/announcing-fedora-27/
Relies on: docker-library/official-images#3708
Relates to:
NOTE: The next release this is eligible for is
17.12.0-ce
Signed-off-by: Eli Uriegas [email protected]