-
-
Notifications
You must be signed in to change notification settings - Fork 597
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
New Script: calibre-server #960
Conversation
…e tag case, and add newline in success message
ct/calibre-server.sh
Outdated
# Potentially re-run the official calibre script to ensure most recent version | ||
# to keep logic consistent with how other scripts handle updates. | ||
msg_info "Updating Calibre (latest)" | ||
wget -nv -O- https://download.calibre-ebook.com/linux-installer.sh | sh /dev/stdin &>/dev/null |
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.
If possible check if there is a newer version wich needs to be installed. Have a look at the snipeit.sh file for reference.
hey @michelroegl-brunner , thank you for taking a look through my PR. I had actually set this up as a draft and wasn't expecting anyone to review. I will update with your recommended changes when I get the chance! |
@ThisIsJeron Any progress on this so far? |
without check because latest
✍️ Description
Create a new script that adds calibre-server as an LXC
🛠️ Type of Change
Please check the relevant options:
✅ Prerequisites
The following steps must be completed for the pull request to be considered:
📋 Additional Information (optional)
Provide any extra context or screenshots about the feature or fix here.