On Tue, Dec 07, 2021 at 07:51:31PM +0100, Richard Ulmer wrote:
> Hi Ingo and Theo,
> thanks for sharing your opinions and thanks for the very thorough
> response, Ingo! I learned a few things while reading it and appreciate
> your suggested patch. I especially like the introduced bullet points and
> find they make the man page easier to read.
> 
> "Theo de Raadt" <[email protected]> wrote:
> > Ingo Schwarze <[email protected]> wrote:
> > 
> > > >> I'd much prefer to have
> > > >> the actions explained in the lesskey(1) man page.
> > > 
> > > No way.  Copying half of the less(1) manual to the lesskey(1) manual
> > > would result in a maintenance nightmare.
> > 
> > I agree.  This is not the first time one has to read two related pages
> > to gain understanding, rather than reading one monster combined or
> > duplicated page -- which can muddle up other learning patterns.
> 
> I understand your concerns and think I might had made up my mind a
> little to quickly. Let me take one step back: I thought something was
> missing from the man page, because I initially thought, that lesskey was
> meant to provide ways to extend less. I thought, that it would allow
> me to bind shell scripts to keys or something similar. Now that I'm a
> little more familiar with lesskey, I can see that this assumption was
> wrong.
> 
> lesskey is only meant to change the default key bindings. If I
> understood this earlier on, it would probably have saved me the
> confusion. Maybe the purpose of lesskey could be clarified a bit more in
> the man page. What do you think?
> 
> Greetings,
> Richard Ulmer
> 
> Index: lesskey.1
> ===================================================================
> RCS file: /cvs/src/usr.bin/less/lesskey.1,v
> retrieving revision 1.17
> diff -u -p -u -r1.17 lesskey.1
> --- lesskey.1   7 Dec 2021 13:26:49 -0000       1.17
> +++ lesskey.1   7 Dec 2021 18:49:26 -0000
> @@ -27,7 +27,7 @@
>  .Os
>  .Sh NAME
>  .Nm lesskey
> -.Nd specify key bindings for less
> +.Nd customize key bindings for less
>  .Sh SYNOPSIS
>  .Nm lesskey
>  .Oo Fl o Ar output
> @@ -37,8 +37,8 @@
>  .Fl V | -version
>  .Sh DESCRIPTION
>  .Nm
> -is used to specify a set of key bindings to be used by
> -.Xr less 1 .
> +is used to change the default key bindings of
> +.Xr less 1 to match personal preference.
>  The input file is a text file which describes the key bindings.
>  If the input file is
>  .Sq - ,
> 

hi.

i committed your diff, with one adjustment: i omitted the "to match
personal preference" text since i didn;t like how it sounded, nor felt
that it added anything.

thanks for the diff,
jmc

Reply via email to