Ticket #3509 (closed defect: fixed)

Opened 4 years ago

Last modified 3 years ago

man invocation not portable

Reported by: petere Owned by: andrew_b
Priority: major Milestone: 4.8.16
Component: mc-core Version: master
Keywords: Cc:
Blocked By: Blocking:
Branch state: merged Votes for changeset: committed-master

Description

The invocation of man in misc/ext.d/text.sh.in is not portable. BSD man commands do not take the -l option to show a man page by file name. (Instead, just give the file name, no option necessary.)

As a result, viewing man pages with mc does not work on these platforms.

Distributions are working around that:

man

Change History

comment:1 Changed 3 years ago by and

man option -l was used to ensure that last parameter interpreted as (local)file and not as page.

man -P cat file.name
No manual entry for file.name
$ man -P cat -l file.name
man: file.name: No such file or directory
$ man -P cat /file.name
man: /file.name: No such file or directory
No manual entry for /file.name

for man-db (which provide -l option) this option is not required at all
because man-db detects if last parameter is a file
and mc helper calls by full pathfilename always.

So let's get rid of not-at-all supported -l option

comment:2 Changed 3 years ago by and

  • Milestone changed from Future Releases to 4.8.16

comment:3 Changed 3 years ago by andrew_b

  • Status changed from new to accepted
  • Owner set to andrew_b
  • Branch state changed from no branch to on review

comment:4 Changed 3 years ago by andrew_b

  • Votes for changeset set to andrew_b
  • Branch state changed from on review to approved

comment:5 Changed 3 years ago by andrew_b

  • Status changed from accepted to testing
  • Votes for changeset changed from andrew_b to committed-master
  • Resolution set to fixed
  • Branch state changed from approved to merged

comment:6 Changed 3 years ago by andrew_b

  • Status changed from testing to closed
Note: See TracTickets for help on using tickets.