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

Remove superfluous items which were moved to the calibre mod #276

Merged
merged 4 commits into from
Oct 18, 2023

Conversation

drizuid
Copy link
Member

@drizuid drizuid commented Oct 17, 2023

linuxserver.io


  • I have read the contributing guideline and understand that I have made the correct modifications

Description:

while working on some issues in lazylibrarian with missing packages, it was determined that the missing packages are required for the calibre mod. In the case of CW, these were being installed on the container which did not appear to need them without the calibre mod. a paired PR (linuxserver/docker-mods#774) adds these packages into the mod so that it works consistently across both LL and CW.

Benefits of this PR and context:

Remove the packages which aren't needed for the base install. speeds up install and reduces size.

How Has This Been Tested?

tested locally for base functionality without mod, added mod PR to confirm functionality

@drizuid drizuid requested a review from a team October 17, 2023 14:22
@drizuid
Copy link
Member Author

drizuid commented Oct 17, 2023

do not merge prior to linuxserver/docker-mods#774 being merged

@LinuxServer-CI
Copy link
Contributor

I am a bot, here are the test results for this PR:
https://ci-tests.linuxserver.io/lspipepr/calibre-web/0.6.20-pkg-c841649a-dev-28ea5a0f6511911ab035e4f2bf11cd2af0f30459-pr-276/index.html
https://ci-tests.linuxserver.io/lspipepr/calibre-web/0.6.20-pkg-c841649a-dev-28ea5a0f6511911ab035e4f2bf11cd2af0f30459-pr-276/shellcheck-result.xml

Tag Passed
amd64-0.6.20-pkg-c841649a-dev-28ea5a0f6511911ab035e4f2bf11cd2af0f30459-pr-276
arm64v8-0.6.20-pkg-c841649a-dev-28ea5a0f6511911ab035e4f2bf11cd2af0f30459-pr-276

@drizuid drizuid merged commit 297ab51 into master Oct 18, 2023
6 checks passed
@drizuid drizuid deleted the remove-superfluous-items branch October 18, 2023 01:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants