fix: propagate run_doctor exit code in all launchers

All three launchers (deb, RPM, AppImage) used bare `exit` after
`run_doctor`, discarding the return code. Now uses `exit $?` so
non-zero exit codes from doctor diagnostics propagate correctly.

Also adds joekale-pp to README contributors for RPM doctor support.

Co-Authored-By: Claude <claude@anthropic.com>
This commit is contained in:
aaddrick
2026-03-19 08:03:35 -04:00
parent dcb7b39393
commit 3f7a3c2328
4 changed files with 4 additions and 3 deletions

View File

@@ -180,6 +180,7 @@ Special thanks to:
- **[noctuum](https://github.com/noctuum)** for the `CLAUDE_MENU_BAR` env var with configurable menu bar visibility and boolean alias support
- **[typedrat](https://github.com/typedrat)** for the NixOS flake integration with build.sh, node-pty derivation, and CI auto-update
- **[cbonnissent](https://github.com/cbonnissent)** for reverse-engineering the Cowork VM guest RPC protocol and fixing the KVM startup blocker
- **[joekale-pp](https://github.com/joekale-pp)** for adding `--doctor` support to the RPM launcher
## Sponsorship

View File

@@ -78,7 +78,7 @@ source "$appdir/usr/lib/claude-desktop/launcher-common.sh"
if [[ "${1:-}" == '--doctor' ]]; then
electron_path="$appdir/usr/lib/node_modules/electron/dist/electron"
run_doctor "$electron_path"
exit
exit $?
fi
# Setup logging and environment

View File

@@ -98,7 +98,7 @@ source "/usr/lib/$package_name/launcher-common.sh"
if [[ "\${1:-}" == '--doctor' ]]; then
local_electron_path="/usr/lib/$package_name/node_modules/electron/dist/electron"
run_doctor "\$local_electron_path"
exit
exit \$?
fi
# Setup logging and environment

View File

@@ -83,7 +83,7 @@ source "/usr/lib/$package_name/launcher-common.sh"
if [[ "\${1:-}" == '--doctor' ]]; then
local_electron_path="/usr/lib/$package_name/node_modules/electron/dist/electron"
run_doctor "\$local_electron_path"
exit
exit \$?
fi
# Setup logging and environment