Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Merge Review: texinfo https://bugzilla.redhat.com/show_bug.cgi?id=226488 pertusus@xxxxxxx changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |pertusus@xxxxxxx ------- Additional Comments From pertusus@xxxxxxx 2008-01-11 17:34 EST ------- I don't understand exactly why the __spec_install_post is redefined. It seems to me that brp-compress leaves dir as is. I have tried to remove the %define and things seems to be right. Prereq should be Requires(.). At least I think so, although this is not obvious with trigger/Requires(.) interactions. The manual gzip are bad. Of course they could be there for the comment above, that is avoiding gziping the dir file, but couldn't it be done more cleanly? Why are the xsl and dtd files removed? in the install-info scriptlets, install-info should figure out whether .gz is needed or not. Having the dir file %config is not very nice, given that it is not in sysconfdir. However I can't see any other way. I suggest using %defattr(-,root,root,-) instead of %defattr(-,root,root) There is a build dependency loop texinfo -> ncurses-devel -> gpm-devel -> install-info. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug, or are watching the QA contact. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review