Bug 993686
Summary: | bochs possibly affected by F-20 unversioned docdir change | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Ville Skyttä <ville.skytta> |
Component: | bochs | Assignee: | Chris Lalancette <clalancette> |
Status: | CLOSED RAWHIDE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | unspecified | Docs Contact: | |
Priority: | unspecified | ||
Version: | rawhide | CC: | clalancette, gwync, virt-maint |
Target Milestone: | --- | Keywords: | Reopened |
Target Release: | --- | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
URL: | http://fedoraproject.org/wiki/Changes/UnversionedDocdirs | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2013-08-13 20:07:29 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: | |||
Bug Depends On: | |||
Bug Blocks: | 993551 |
Description
Ville Skyttä
2013-08-06 11:39:55 UTC
Looks like a false positive. (In reply to Jon Ciesla from comment #1) > Looks like a false positive. But isn't, which would have been easy to catch by looking at the grep output as suggested: $ grep -A 2 -B 2 -E "(/doc|_docdir|_defaultdocdir).+version" bochs.spec # Fix up some man page paths. sed -i \ -e 's|/usr/local/share/doc/bochs/|%{_docdir}/%{name}-%{version}/|' \ -e 's|/usr/local/share/|%{_datadir}/|' \ doc/man/*.* I saw the grep output, but I'm not sure what needs to be fixed here. I built this in mock for rawhide, and I get an unversioned docdir, and the paths referenced in the man pages match that. What's the problem. Indeed, the man pages have /usr/share/doc/bochs/ hardcoded. That means that the mentioned sed does nothing to them (it looks for /usr/local/...) and that at the moment the path is correct for Rawhide and incorrect in earlier distros. I suggest dropping this line from the sed altogether: -e 's|/usr/local/share/doc/bochs/|%{_docdir}/%{name}-%{version}/|' \ ...or adjusting it to match /usr/share/doc/bochs/ and to modify to %{_pkgdocdir} (+ adding its conditional definition) if you want to fix this for earlier distros. Dropped. Thanks! BTW the other sed line that was removed in 2.6.2-4 in addition to the one I suggested was not a noop: $ grep -r usr/local/share doc/man/* doc/man/bochs-dlx.1:/usr/local/share/bochs/dlxlinux/bochsrc.txt or run bochs(1) doc/man/bochsrc.5:"share" directory which is typically /usr/local/share/bochs on UNIX Right you are, fixed. |