Bug 986871 - [PATCH] Drop version from _docdir_fmt
[PATCH] Drop version from _docdir_fmt
Status: CLOSED RAWHIDE
Product: Fedora
Classification: Fedora
Component: redhat-rpm-config (Show other bugs)
rawhide
Unspecified Unspecified
unspecified Severity unspecified
: ---
: ---
Assigned To: Panu Matilainen
Fedora Extras Quality Assurance
: Patch
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2013-07-22 05:53 EDT by Ville Skyttä
Modified: 2013-07-24 19:58 EDT (History)
4 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
: 1112976 (view as bug list)
Environment:
Last Closed: 2013-07-24 19:58:46 EDT
Type: Bug
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)
Drop version from _docdir_fmt (453 bytes, patch)
2013-07-22 05:53 EDT, Ville Skyttä
no flags Details | Diff

  None (edit)
Description Ville Skyttä 2013-07-22 05:53:05 EDT
Created attachment 776833 [details]
Drop version from _docdir_fmt

Attached is a patch that drops package version from _docdir_fmt, thus making doc dirs unversioned per the now accepted F-20 feature.

https://lists.fedoraproject.org/pipermail/devel/2013-July/185633.html
https://fedorahosted.org/fesco/ticket/1138

Ok if I push and build this for devel? And would you like it conditionalized on 0%{?fedora} >= 20?
Comment 1 Miloslav Trmač 2013-07-22 07:22:29 EDT
Please note this needs to be applied before the F20 mass rebuild starts; see https://fedorahosted.org/rel-eng/ticket/5655 for related coordination.
Comment 2 Kevin Fenzi 2013-07-24 16:13:33 EDT
Scratch build with this and the libtool change in it. 

Please scream if you see anything wrong with it. 

I will push this out later tonight if not: 

http://koji.fedoraproject.org/koji/taskinfo?taskID=5653042
Comment 3 Ville Skyttä 2013-07-24 16:43:27 EDT
Thanks, unfortunately I have no time to test/scream any more tonight, but per discussion on the packaging list I suggest adding _pkgdocdir to macros in addition to _docdir_fmt, making the block of these vars look like:

%_defaultdocdir		%{_prefix}/share/doc
%_pkgdocdir		%{_docdir}/%{name}
%_docdir_fmt		%%{NAME}
Comment 4 Kevin Fenzi 2013-07-24 16:48:53 EDT
ok. I folded that change in too. Thanks.
Comment 5 Kevin Fenzi 2013-07-24 19:58:46 EDT
ok. Should be live in rawhide now. 

Please reopen if you see any issues.

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