Skip to content
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

GUI CLI integration missing non-zero exit codes #9225

Open
deeplow opened this issue May 13, 2024 · 0 comments
Open

GUI CLI integration missing non-zero exit codes #9225

deeplow opened this issue May 13, 2024 · 0 comments
Labels
affects-4.2 This issue affects Qubes OS 4.2. C: manager/widget needs diagnosis Requires technical diagnosis from developer. Replace with "diagnosed" or remove if otherwise closed. P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. T: bug Type: bug report. A problem or defect resulting in unintended behavior in something that exists.

Comments

@deeplow
Copy link

deeplow commented May 13, 2024

How to file a helpful issue

Qubes OS release

4.2

Brief summary

Calling the GUI updater from the CLI always exits with zero even if some update failed.

Steps to reproduce

  1. qubes-update-gui --restart
  2. If any update failed or the user exited without restarting qubes, confirm that exit code is still zero

Expected behavior

Have failures consistent with CLI updater counterpart: after any failure, the exit code would be non-zero.

Extra caveat: what to do when user bailed on updates / restarts? (here are my takes)

(If possible, the above exit codes are different to disambiguate)

Actual behavior

The GUI updater always exits with zero

@deeplow deeplow added P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. T: bug Type: bug report. A problem or defect resulting in unintended behavior in something that exists. labels May 13, 2024
@andrewdavidwong andrewdavidwong added C: manager/widget needs diagnosis Requires technical diagnosis from developer. Replace with "diagnosed" or remove if otherwise closed. affects-4.2 This issue affects Qubes OS 4.2. labels May 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects-4.2 This issue affects Qubes OS 4.2. C: manager/widget needs diagnosis Requires technical diagnosis from developer. Replace with "diagnosed" or remove if otherwise closed. P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. T: bug Type: bug report. A problem or defect resulting in unintended behavior in something that exists.
Projects
None yet
Development

No branches or pull requests

2 participants