[Bug 226488] Merge Review: texinfo

[bugzilla at redhat.com]

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 at free.fr changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |pertusus at free.fr




------- Additional Comments From pertusus at free.fr  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.