Skip to content

Explain why uninstall script needs admin password#1131

Closed
William-Laverty wants to merge 1 commit intoapple:mainfrom
William-Laverty:docs-explain-sudo-requirement
Closed

Explain why uninstall script needs admin password#1131
William-Laverty wants to merge 1 commit intoapple:mainfrom
William-Laverty:docs-explain-sudo-requirement

Conversation

@William-Laverty
Copy link
Contributor

Summary

Fixes #1111

Adds an informative message to the uninstall script explaining why an administrator password is required before prompting for it.

Changes

Added a single line before the first sudo command:

This script requires an administrator password to remove the application files from system directories.

Before

% uninstall-container.sh -d
Password:

After

% uninstall-container.sh -d
This script requires an administrator password to remove the application files from system directories.
Password:

Fixes #1111

Add an informative message before the first sudo command to explain
that administrator privileges are required to remove application files
from system directories (/usr/local).
@jglogan jglogan self-requested a review February 2, 2026 17:45
@jglogan
Copy link
Contributor

jglogan commented Feb 2, 2026

@William-Laverty What is this one for when you already filed #1118?

See the comments about about commit signing in that PR.

@jglogan
Copy link
Contributor

jglogan commented Feb 2, 2026

With #1118 merged and your #1130 for enhancing the message further, is this one needed or can you close it?

@William-Laverty
Copy link
Contributor Author

Closing as redundant - #1118 was merged and #1130 covers the enhanced messaging. Thanks for the review!

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.

[Request]: Explain why uninstall-container.sh needs your password.

2 participants