docs: correct auto-update platform support (7.1)#753
Conversation
The Automatic Updater is documented as Windows-only, but in 7.1 it also downloads and installs updates on macOS (Sparkle) and on Linux AppImage builds (in-app AppImage updater). Only Linux distribution packages are notify-only. Clarify the Windows flow as a silent download followed by a user-confirmed "Restart required" prompt. Source: src/gui/updater/updater.cpp, appimageupdater.cpp, sparkleupdater_mac.mm, ocupdater.cpp. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Signed-off-by: Thomas Müller <1005065+DeepDiver1975@users.noreply.github.com>
Review —
|
dj4oC
left a comment
There was a problem hiding this comment.
7.1 counterpart of #754 — same Automatic Updater platform-support rewrite.
Findings
Diff matches the master PR (#754) in substance. Not independently re-verified against owncloud/client source (out of scope here).
Verdict
Approved
🤖 Automated review by Claude Code
Generated by Claude Code
Summary
The Automatic Updater page documents auto-update as Windows-only, but that is inaccurate for client 7.1:
src/gui/updater/sparkleupdater_mac.mm).src/gui/updater/appimageupdater.cpp)..deb/.rpm) are notify-only (PassiveUpdateNotifierinocupdater.cpp).This PR rewrites the Introduction, macOS, and Linux sections accordingly, and clarifies the Windows flow as a silent download followed by a user-confirmed Restart required prompt (
WindowsUpdater/UpdateDownloadedWidget).Source of truth
owncloud/client@7.1:src/gui/updater/{updater,appimageupdater,ocupdater}.cpp,sparkleupdater_mac.mm.🤖 Generated with Claude Code