[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#638182: less: please be more verbose about lesspipe usage in login script

Package: less
Version: 444-1
Severity: wishlist

Hi there!

The discussion started on the debian-devel@ mailing list:


On Tue, 16 Aug 2011 12:38:43 +0200, Adam Borowski wrote:
> On Tue, Aug 16, 2011 at 11:50:49AM +0200, Luca Capello wrote:
>> On Tue, 16 Aug 2011 10:08:45 +0200, Alexander Reichle-Schmehl wrote:
>> > FWIW, if you the following in your bashrc:
>> >
>> > if [ -f /usr/bin/lesspipe ]; then
>> I would use the -x expression instead ;-)
>> And I would also say that this test should be included in the
>> lesspipe(1) manpage...

Here we are.

> /etc/skel/.profile has it, commented out:

On sid it has not.  But /etc/skel/.bashrc:23 does ;-)

However, the lesspipe manpage says:

	Just put  one of  the following two  commands in  your login
	script (e.g.  ~/.bash_profile):

	  eval "$(lessfile)"


	  eval "$(lesspipe)"

I think the manpage should:

1) document the full snippet above as the one to be put in login script,
   i.e. with the test as well.

2) be consistent with what Debian ships in /etc/skel/ files, i.e. if the
   snippet is in .bashrc, the manpage should reflect that and not
   suggest .bash_profile.

> # make less more friendly for non-text input files, see lesspipe(1)
> #[ -x /usr/bin/lesspipe ] && eval "$(SHELL=/bin/sh lesspipe)"

I do not understand why we need to specify SHELL, however.

Thx, bye,
Gismo / Luca

-- System Information:
Debian Release: wheezy/sid
  APT prefers unstable
  APT policy: (990, 'unstable'), (1, 'experimental')
Architecture: amd64 (x86_64)

Kernel: Linux 3.0.0-1-amd64 (SMP w/2 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages less depends on:
ii  debianutils                   4.0.2      Miscellaneous utilities specific t
ii  libc6                         2.13-16    Embedded GNU C Library: Shared lib
ii  libncurses5                   5.9-1      shared libraries for terminal hand

less recommends no packages.

less suggests no packages.

-- no debconf information

Attachment: pgpR88y7faERu.pgp
Description: PGP signature

Reply to: