Skip to content

feat: add sonarr to easy install and uninstall script - #474

Open
chrismin13 wants to merge 2 commits into
Servarr:masterfrom
chrismin13:patch-1
Open

chrismin13 wants to merge 2 commits into
Servarr:masterfrom
chrismin13:patch-1

Conversation

@chrismin13

Copy link
Copy Markdown

Hello, this is a PR to add Sonarr to the easy install script, as it was missing. Tested on Ubuntu Server 26.04

Copilot AI balanced review requested due to automatic review settings October 3, 2026 15:16

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@chrismin13 chrismin13 changed the title feat: add sonarr to easy install script feat: add sonarr to easy install and uninstall script Oct 3, 2026
@chrismin13

Copy link
Copy Markdown
Author

also added it to the uninstall script!

@bakerboy448

Copy link
Copy Markdown
Contributor

Sonarr intentionally is not part of the Servarr Script as Sonarr maintains their own scripts.

Had you did any search you would have seen this before wasting your time

@chrismin13

Copy link
Copy Markdown
Author

https://wiki.servarr.com/en/sonarr Then how come there is a servarr wiki entry that links to this very script and includes manual setup instructions as a backup?

@bakerboy448

Copy link
Copy Markdown
Contributor

Thanks for testing this. Sonarr is left out of the community script on purpose: it has its own official installer, maintained in the Sonarr repo (distribution/debian/install.sh), forked from this script and kept in step with Sonarr's download endpoints. Adding Sonarr here gives two installers that will drift apart.

The real bug is that the wiki's Sonarr Linux page sends people to the community script for "Easy Install". I'm opening a PR to point that at Sonarr's official script instead.

The uninstall-script change (adding sonarr to the services list) is harmless and useful on its own, since the official script has no uninstaller. If you want to cut this PR down to just that, I'm fine with it.

@bakerboy448

Copy link
Copy Markdown
Contributor

Suggest closing the installer half: Sonarr's own script covers this and #475 points the wiki at it. The uninstall change (adding sonarr to services) is useful on its own PR: Sonarr ships no uninstaller, and this one reads User/Group/paths from the systemd unit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants