[Bug 225854] Merge Review: gperf

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxxxx>  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.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]