[Pkg-shadow-devel] Bug#1064940: Bug#1064940: vipw's manpage does not document ~/.selected_editor

[Serge E. Hallyn]

Thanks.  This is new to me - I see that debian/rules sets that as the
default.

Note that vipw will not use sensible-editor, or ask you what you want
for it, if VISUAL or EDITOR is set in your environment.  Would you
mind adding that to your manpage patch?

thanks,
-serge

On Wed, Feb 28, 2024 at 12:07:50AM +0000, Toni via Pkg-shadow-devel wrote:
> Package: passwd
> Version: 1:4.13+dfsg1-1+b1
> Severity: minor
> Tags: patch
> 
> 
> Hi,
> 
> the 'vipw' program uses a file that isn't documented. The attached patch
> should fill this gap, although I haven't tried to build the package with
> it.
> 
> 
> Enjoy,
> Toni
> 
> 
> 
> -- System Information:
> Debian Release: 12.5
>   APT prefers stable
>   APT policy: (990, 'stable'), (500, 'oldstable')
> Architecture: amd64 (x86_64)
> Foreign Architectures: i386
> 
> Kernel: Linux 6.1.0-18-amd64 (SMP w/12 CPU threads; PREEMPT)
> Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.utf8 (charmap=UTF-8), LANGUAGE=en_US:en
> Shell: /bin/sh linked to /usr/bin/dash
> Init: systemd (via /run/systemd/system)
> LSM: AppArmor: enabled
> 
> Versions of packages passwd depends on:
> ii  libaudit1       1:3.0.9-1
> ii  libc6           2.36-9+deb12u4
> ii  libcrypt1       1:4.4.33-2
> ii  libpam-modules  1.5.2-6+deb12u1
> ii  libpam0g        1.5.2-6+deb12u1
> ii  libselinux1     3.4-1+b6
> ii  libsemanage2    3.4-1+b5
> 
> Versions of packages passwd recommends:
> ii  sensible-utils  0.0.17+nmu1
> 
> passwd suggests no packages.
> 
> -- no debconf information

> diff --git a/man/vipw.8.xml b/man/vipw.8.xml
> index 4caddcb..9775af7 100644
> --- a/man/vipw.8.xml
> +++ b/man/vipw.8.xml
> @@ -77,6 +77,11 @@
>        <citerefentry><refentrytitle>vi</refentrytitle>
>        <manvolnum>1</manvolnum></citerefentry>.
>      </para>
> +    <para>
> +        On the first run, this program asks you for an editor and stores your
> +        selection in ~/.selected_editor. If the editor mentioned therein does
> +        not exist on your system, the program will fall back to using 'nano'.
> +    </para>
>    </refsect1>
>  
>    <refsect1 id='options'>
> @@ -210,6 +215,9 @@
>        <citerefentry>
>  	<refentrytitle>gshadow</refentrytitle><manvolnum>5</manvolnum>
>        </citerefentry>
> +      <citerefentry>
> +          <refentrytitle>~/.selected_editor</refentrytitle><manvolnum>5</manvolnum>
> +      </citerefentry>
>        <citerefentry condition="tcb">
>  	<refentrytitle>login.defs</refentrytitle><manvolnum>5</manvolnum>
>        </citerefentry>,

> _______________________________________________
> Pkg-shadow-devel mailing list
> Pkg-shadow-devel at alioth-lists.debian.net
> https://alioth-lists.debian.net/cgi-bin/mailman/listinfo/pkg-shadow-devel