bug#63221: 30.0.50; [PATCH] Eshell should get user (and group) IDs in a connection-aware fashion

[Eli Zaretskii <eliz@gnu.org>]

> Date: Tue, 2 May 2023 11:36:16 -0700
> Cc: 63221@debbugs.gnu.org
> From: Jim Porter <jporterbugs@gmail.com>
> 
> > This terse sentence needs to be explained in the rest of the doc
> > string, because, unlike "If non-nil", "When" does not explain itself.
> > The doc string should explain how to specify "when".  It should also
> > explain the different supported values.
> 
> Thanks, fixed.
> 
> >> +  :tag "Query"
> >> +  :type '(choice (const :tag "Never" nil)
> >> +                 (const :tag "Always" t)
> >> +                 (const :tag "When root" root)))
> > 
> > Also, the default value is not one of the possible optional values.
> 
> I changed how this works so now the widget inherits from 'radio' instead 
> of 'lazy', and I think it should work better overall now.
> 
> > Same comment to the other similar defcustoms where you changed a
> > boolean option to something else: their doc strings are now
> > obfuscated.
> 
> Fixed.
> 
> >> +(defun eshell-interactive-query-p (value)
> >> +  "Return non-nil if a command should query the user according to VALUE.
> >> +If VALUE is `root', return non-nil when evaluated as root (see
> >> +`file-user-uid').  Otherwise, simply return VALUE."
> > 
> > You assume here that "evaluated as root" explains itself?  I wouldn't
> > rely on that.
> 
> Also fixed.

Thanks, LGTM.