[Bug 225809] Merge Review: gmp

[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.


https://bugzilla.redhat.com/show_bug.cgi?id=225809


Jason Tibbitts <tibbs at math.uh.edu> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |CLOSED
         Resolution|                            |RAWHIDE
               Flag|fedora-review?              |fedora-review+




--- Comment #29 from Jason Tibbitts <tibbs at math.uh.edu>  2010-01-06 21:17:52 EDT ---
I checked current CVS.  Things look good; the only thing I can see currently is
that I don't really understand the need for the conditional in the devel
scriptlets:

if [ -f %{_infodir}/gmp.info.gz ]; then
    /sbin/install-info %{_infodir}/gmp.info.gz %{_infodir}/dir || :
fi

I guess the idea is to protect against an --excludedocs install, but that's not
necessary as the recommended scriptlet (just the second line) handles that case
just fine. In any case, it's only four pointless lines and this has gone on
long enough, so I guess we're done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.