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: pinfo https://bugzilla.redhat.com/show_bug.cgi?id=226297 panemade@xxxxxxxxx changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |CLOSED Resolution| |RAWHIDE Flag|fedora-review? |fedora-review+ ------- Additional Comments From panemade@xxxxxxxxx 2008-01-15 04:31 EST ------- If you think its ok then I don't mind. I just want to make sure it follows exactly whats written on packaging guidelines page. In SPEC its, %post /sbin/install-info %{_infodir}/pinfo.info.gz %{_infodir}/dir : %preun if [ $1 = 0 ]; then /sbin/install-info --delete %{_infodir}/pinfo.info.gz %{_infodir}/dir fi : suggested one is %post /sbin/install-info %{_infodir}/%{name}.info %{_infodir}/dir || : %preun if [ $1 = 0 ]; then /sbin/install-info --delete %{_infodir}/%{name}.info %{_infodir}/dir || : fi I think rest packaging looks Ok. -- 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