[Bug 225854] Merge Review: gperf
bugzilla at redhat.com
bugzilla at redhat.com
Mon Jan 19 22:28:29 UTC 2009
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=225854
--- Comment #3 from Robert Scheck <redhat-bugzilla at linuxnetz.de> 2009-01-19 17:28:26 EDT ---
Roman, thank you for taking action. I think, I've forgotten the suggestion
of preserving timestamps before and to avoid the usage of %makeinstall, thus
we recommend packagers to use instead:
make DESTDIR=$RPM_BUILD_ROOT install INSTALL='install -p'
Regarding the documentation I'm not really happy. We've currently much more
documentation rather rest of gperf inside of the package.
We've multiple options: Creating -docs subpackage and moving everything out
there OR just kill the huge *.ps from %doc (*.ps vs. *.pdf seems to be a bit
redundant and *.pdf is usually better searchable) - last of it was accepted
and got told to be useful in Freenode #fedora-devel, #fedora-de by several
packagers.
Choose what you like as packager and let me know. I can deal with both or
even a better option - afterwards we should be (hopefully) fine with review.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
More information about the Fedora-package-review
mailing list