-
Notifications
You must be signed in to change notification settings - Fork 12
Refactor Dockerfiles #39
Conversation
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.
Additionally there was a new dependency added for rkmpp which is missing here too.
|
Sorry to interrupt but can i help with testing? I need this merge to be done cause of Issue #10674 AMD VAAPI transcoding. What do i need to do? I know a bit about docker but i didnt understand wich files are needed within the dockerfile to build the new image correctly so i can test it - especially the vulkan api catched my eye ^^ Thx and thx for your work on jelly <3 |
This PR is ready to merge I don't have to make any other change live checked everything 5 times we now have to wait for the team review and merge and if feel free to ask me some questions ! |
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
Gtk... So is it enough to dl your repo and "execute" the right dockerfile to have the newest jellyfin up n running? |
Of course no problem I've tested the image and I didn't have any problem, say to me if it fix your problem |

Rework Dockerfiles
Can be closed #38