Bug#991984: closed by Russ Allbery (Re: Bug#991984: Please document minimal environment variable needed for sensible-utils)

[Edward Little]

Please remove the following email address:  e.little...@gmail.com

On Sun, Sep 10, 2023 at 4:21 AM Bastien Roucariès  wrote:

> Le dimanche 10 septembre 2023, 04:33:06 UTC Debian Bug Tracking System a
> écrit :
> > This is an automatic notification regarding your Bug report
> > which was filed against the debian-policy package:
> >
> > #991984: Please document minimal environment variable needed for
> sensible-utils
> >
> > It has been closed by Russ Allbery .
> >
> > Their explanation is attached below along with your original report.
> > If this explanation is unsatisfactory and you have not received a
> > better one in a separate message then please contact Russ Allbery <
> r...@debian.org> by
> > replying to this email.
> >
> Seems sensible note that linux manpages mandate now some behavior for
> EDITOR, PAGER and VISUAL
>
> Bastien
>
>

Bug#991984: closed by Russ Allbery (Re: Bug#991984: Please document minimal environment variable needed for sensible-utils)

[Bastien Roucariès]

Le dimanche 10 septembre 2023, 04:33:06 UTC Debian Bug Tracking System a écrit :
> This is an automatic notification regarding your Bug report
> which was filed against the debian-policy package:
> 
> #991984: Please document minimal environment variable needed for 
> sensible-utils
> 
> It has been closed by Russ Allbery .
> 
> Their explanation is attached below along with your original report.
> If this explanation is unsatisfactory and you have not received a
> better one in a separate message then please contact Russ Allbery 
>  by
> replying to this email.
> 
Seems sensible note that linux manpages mandate now some behavior for EDITOR, 
PAGER and VISUAL

Bastien



signature.asc
Description: This is a digitally signed message part.

Bug#991984: Please document minimal environment variable needed for sensible-utils

[Russ Allbery]

"Bastien Roucariès"  writes:

> Dear Maintainer,

> For now $env -i sensible-utils, fail due to $HOME and $TERM not set.

> I am for now working around HOME not set in sensible-utils core, but
> posix [1] documentation does not document really the value that should
> be set for a correct behavior of program.

> Nevertheless:
> - we should expect that PATH is set to a sensible value (note that it depends
>   of the shell), but nevertheless not setting PATH is not really safe
> - HOME may not be set. If set the value may be incorrect (su -)
> - TERM may not be set. If set the value may not be correct
> - USER may not be set. If set the value may be incorrect (su -)

> So I will like to have a footnote saying that
> sensible-pager/sensible-editor etc, should test if they work under env
> -i, and if they do not work fallback to return 126 (according to shell
> documentation Command invoked cannot execute), thus allowing
> sensible-utils to fallback to vi that is safe and tested under env -i

I think it is a fine idea to attempt to support those situations in
sensible-editor and sensible-pager and document them there, but I'm not
sure what Policy change you're asking for here or why this would be a
matter for Policy.

Policy does not mandate any specific behavior for sensible-editor and
sensible-pager other than that they will implement the EDITOR and PAGER
environment variable checking for you.  I think that's best left to the
maintainer of those programs to decide.

We also don't expect that the editor or pager invoked following the rules
in Policy 11.4 (and, by extension, sensible-editor and sensible-pager
themselves) will work in unusual situations, such as ones without standard
environment variables set.  We can't: the user is free to set EDITOR and
PAGER to anything they chose, including programs not in Debian.  So you
can't really expect any particular behavior from whatever EDITOR or PAGER
is set to.  Maybe it will fail with a helpful error code, maybe it will
start and not work but not exit, maybe something else entirely will
happen.  This is really outside of our control.

-- 
Russ Allbery (r...@debian.org)  

Bug#991984: Please document minimal environment variable needed for sensible-utils

[Bastien Roucariès]

Package: debian-policy
Version: 4.5.1.0
Severity: important
Control: block 991982 by -1
Control: block 987675 by -1


Dear Maintainer,

For now $env -i sensible-utils, fail due to $HOME and $TERM not set.

I am for now working around HOME not set in sensible-utils core, but posix [1]
documentation does not document really the value that should be set for a
correct behavior of program.

Nevertheless:
- we should expect that PATH is set to a sensible value (note that it depends
of the shell), but nevertheless not setting PATH is not really safe
- HOME may not be set. If set the value may be incorrect (su -)
- TERM may not be set. If set the value may not be correct
- USER may not be set. If set the value may be incorrect (su -)

So I will like to have a footnote saying that sensible-pager/sensible-editor
etc, should test if they work under env -i, and if they do not work fallback to
return 126 (according to shell documentation Command invoked cannot execute),
thus allowing sensible-utils to fallback to vi that is safe and tested under
env -i

Bastien




[1] https://pubs.opengroup.org/onlinepubs/9699919799/