Skip to content

Commit 00a60f7

Browse files
jnpkrnrh-atomic-bot
authored andcommitted
cli: reconcile how permissions are effectively checked vs. suggestion
Nowhere to be found that UID-0 user has to go under "root" login name, making the message confusing for cases it is not the case, or when there are more such names pertaining that key condition. The original message would only be credible, if that was indeed how the check was constructed, but for obvious reasons, it is not like that. Current wording is a courtesy of Neal Gompa, there were more variants considered before (e.g. "[fully] privileged user", GH#1509). Signed-off-by: Jan Pokorný <jpokorny@redhat.com> Closes: rpm-software-management#1549 Approved by: Conan-Kudo
1 parent c20dcab commit 00a60f7

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

dnf/cli/cli.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -809,7 +809,8 @@ def _process_demands(self):
809809

810810
if demands.root_user:
811811
if not dnf.util.am_i_root():
812-
raise dnf.exceptions.Error(_('This command has to be run under the root user.'))
812+
raise dnf.exceptions.Error(
813+
_('This command has to be run with superuser privileges.'))
813814

814815
if demands.changelogs:
815816
for repo in repos.iter_enabled():

0 commit comments

Comments
 (0)