Bug 1241543 - Use man instead of groff for man pages
Summary: Use man instead of groff for man pages
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: less
Version: 23
Hardware: All
OS: All
unspecified
unspecified
Target Milestone: ---
Assignee: Viktor Jancik
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-07-09 12:51 UTC by Ville Skyttä
Modified: 2015-09-24 08:26 UTC (History)
2 users (show)

Fixed In Version: 471-5.fc22
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-09-04 19:45:48 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
Use man instead of groff for man pages (997 bytes, patch)
2015-07-09 12:51 UTC, Ville Skyttä
no flags Details | Diff
less-479-nonascii-changes (1.57 KB, patch)
2015-08-22 00:37 UTC, Viktor Jancik
no flags Details | Diff

Description Ville Skyttä 2015-07-09 12:51:22 UTC
The groff implementation in lesspipe.sh did not do the right thing
with non-ASCII nor terminal width.

With this change, the dependency on groff-base should maybe be changed
to %{_bindir}/man. But then again other tools possibly used by
lesspipe.sh are not dependencies either so maybe the groff-base
dependency should be just dropped?

The attachment is a "git am"able patch.

Comment 1 Ville Skyttä 2015-07-09 12:51:25 UTC
Created attachment 1050261 [details]
Use man instead of groff for man pages

The groff implementation did not do the right thing with non-ASCII nor
terminal width.

Comment 2 Jan Kurik 2015-07-15 13:19:34 UTC
This bug appears to have been reported against 'rawhide' during the Fedora 23 development cycle.
Changing version to '23'.

(As we did not run this process for some time, it could affect also pre-Fedora 23 development
cycle bugs. We are very sorry. It will help us with cleanup during Fedora 23 End Of Life. Thank you.)

More information and reason for this action is here:
https://fedoraproject.org/wiki/BugZappers/HouseKeeping/Fedora23

Comment 3 Viktor Jancik 2015-08-22 00:37:36 UTC
Created attachment 1065764 [details]
less-479-nonascii-changes

The problem with displaying special characters in rpmlint.1 was that groff -Tascii dropped all special characters, however changing it to -Tutf8 didn't work either, because dispite 'file rpmlint.1' claiming it's encoded in UTF8, it is actually encoded in latin1 (ISO 8859-1).

Using -Tlatin1 however, broke other man pages which were properly encoded in UTF8.

So as a workaround to having to deal with various encodings, man -P cat -l was used, as it appears to have a more sophisticated encoding discovery routine, and converts to utf8 from various encodings, before displaying the man page.

An additional Requires to man-db was therefore added.

Note: The results from 'less rpmlint.1.gz' and 'less rpmlint.1' still differ because of a man bug I discovered in the process:
https://bugzilla.redhat.com/show_bug.cgi?id=1255930

I will push the changes to Rawhide as soon as an unrelated less issue gets resolved.

Comment 4 Fedora Update System 2015-09-02 12:42:16 UTC
less-479-2.fc23 has been submitted as an update to Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2015-14867

Comment 5 Fedora Update System 2015-09-02 13:05:49 UTC
less-471-5.fc22 has been submitted as an update to Fedora 22. https://bodhi.fedoraproject.org/updates/FEDORA-2015-14870

Comment 6 Fedora Update System 2015-09-03 09:52:38 UTC
less-479-2.fc23 has been pushed to the Fedora 23 testing repository. If problems still persist, please make note of it in this bug report.\nIf you want to test the update, you can install it with \n su -c 'yum --enablerepo=updates-testing update less'. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-14867

Comment 7 Fedora Update System 2015-09-03 19:20:51 UTC
less-471-5.fc22 has been pushed to the Fedora 22 testing repository. If problems still persist, please make note of it in this bug report.\nIf you want to test the update, you can install it with \n su -c 'yum --enablerepo=updates-testing update less'. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-14870

Comment 8 Fedora Update System 2015-09-04 19:45:46 UTC
less-479-2.fc23 has been pushed to the Fedora 23 stable repository. If problems still persist, please make note of it in this bug report.

Comment 9 Fedora Update System 2015-09-24 08:26:38 UTC
less-471-5.fc22 has been pushed to the Fedora 22 stable repository. If problems still persist, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.