changedetection: remove msttcorefonts to fix install hang - #16319
Merged
michelroegl-brunner merged 4 commits intoAug 7, 2026
Merged
Conversation
apt-get install msttcorefonts triggers a debconf EULA prompt with no tty to answer it, so the "Installing Font Packages" step can hang indefinitely (reported stuck 15+ min). apache-tika-install.sh and onlyoffice-install.sh already work around this by preseeding the answer with debconf-set-selections before the install; apply the same fix here.
build.func fetches the install script from the upstream hardcoded URL, so simply running ct/changedetection.sh from our fork uses the unfixed upstream install script. Override the curl command after sourcing build.func to intercept that specific URL and redirect it to our fork's fix-changedetection-msttcorefonts-eula branch, which has the EULA preseed for msttcorefonts. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
msttcorefonts downloads fonts from SourceForge at install time; debconf EULA pre-selection only bypasses the interactive prompt, not the network download. fonts-liberation is already in the list and provides metric-compatible Arial/Times/Courier replacements. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Remove the interim curl-override workaround from ct/changedetection.sh that was routing installs to our fork. The actual fix (removing msttcorefonts from install/changedetection-install.sh) is already in place; the redirect is no longer needed and shouldn't be in the upstream PR. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Contributor
Author
|
Verified functionality (with the removed dependency) WAD via fresh install. Tested the ChangeDetector installation, web page tracking, properly received notifications, etc. |
MickLesk
approved these changes
Aug 7, 2026
michelroegl-brunner
approved these changes
Aug 7, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
✍️ Description
Remove
msttcorefontsfrom the changedetection font package list to fix an indefinite install hang on headless LXC environments.msttcorefontsrequires downloading fonts from SourceForge at install time, which frequently stalls or times out in a headless container with no interactive TTY. Pre-seeding the debconf EULA bypasses the interactive prompt but not the SourceForge download.fonts-liberation(already in the list) provides metric-compatible Arial/Times/Courier New replacements and is sufficient for Playwright/Browserless rendering.AI assistance: Claude Sonnet 4.6, medium reasoning level.
🔗 Related Issue
Fixes #
✅ Prerequisites (X in brackets)
🤖 AI Assistance (X in brackets)
AGENTS.mdand.github/agents/pve-script-creator.agent.mdas guidance, and the output has been reviewed and corrected to match those guidelines.🛠️ Type of Change (X in brackets)
README,AppName.md,CONTRIBUTING.md, or other docs.