Bug 1241543
| Summary: | Use man instead of groff for man pages | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Ville Skyttä <ville.skytta> | ||||||
| Component: | less | Assignee: | Viktor Jancik <vjancik> | ||||||
| Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> | ||||||
| Severity: | unspecified | Docs Contact: | |||||||
| Priority: | unspecified | ||||||||
| Version: | 23 | CC: | hhorak, vjancik | ||||||
| Target Milestone: | --- | Keywords: | Reopened | ||||||
| Target Release: | --- | ||||||||
| Hardware: | All | ||||||||
| OS: | All | ||||||||
| Whiteboard: | |||||||||
| Fixed In Version: | 471-5.fc22 | Doc Type: | Bug Fix | ||||||
| Doc Text: | Story Points: | --- | |||||||
| Clone Of: | Environment: | ||||||||
| Last Closed: | 2015-09-04 19:45:48 UTC | Type: | --- | ||||||
| Regression: | --- | Mount Type: | --- | ||||||
| Documentation: | --- | CRM: | |||||||
| Verified Versions: | Category: | --- | |||||||
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||||
| Cloudforms Team: | --- | Target Upstream Version: | |||||||
| Embargoed: | |||||||||
| Attachments: |
|
||||||||
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.
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 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. less-479-2.fc23 has been submitted as an update to Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2015-14867 less-471-5.fc22 has been submitted as an update to Fedora 22. https://bodhi.fedoraproject.org/updates/FEDORA-2015-14870 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 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 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. 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. |
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.