-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Improved Docker pkgbuild #824
Improved Docker pkgbuild #824
Conversation
26ed9d5
to
ceb8b9f
Compare
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.
Mostly LGTM, minor notes only I think
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.
No particular changes necessary, but need some discussion.
Also, have a look at https://docs.docker.com/samples/library/buildpack-deps/
81fd4c1
to
7be4a85
Compare
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
Changes
Cleans up the 3 main Docker package builds (Debian, Fedora, and CentOS), and adds a 4th Ubuntu one, to use a reproducible Docker+Script build method, similar to how jellyfin/jellyfin-android works. The built Docker image is thus reusable, and can be kept around by passing the
-k
/--keep-artifacts
argument first tobuild
, thus allowing one to skip most of the environment setup when rebuilding if desired. This also fixes up a couple problems with the Fedora (and CentOS) builds as originally suggested in #806.Issues
N/A