Ticket #3509 (closed defect: fixed)
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:
- https://github.com/Homebrew/homebrew/pull/42176
- https://svnweb.freebsd.org/ports/head/misc/mc/files/patch-misc__ext.d__text.sh.in?view=markup&pathrev=382039
man
Change History
comment:3 Changed 9 years ago by andrew_b
- Owner set to andrew_b
- Status changed from new to accepted
- Branch state changed from no branch to on review
Branch: 3509_man_not_portable
changeset:7dc7eb3d14b95ce1f57a56bb95bbcdc43be7483f
comment:4 Changed 9 years ago by andrew_b
- Votes for changeset set to andrew_b
- Branch state changed from on review to approved
comment:5 Changed 9 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
Merged to master: [7dc7eb3d14b95ce1f57a56bb95bbcdc43be7483f].
Note: See
TracTickets for help on using
tickets.
man option -l was used to ensure that last parameter interpreted as (local)file and not as page.
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