On Tue, 2006-07-04 at 09:58 +0100, Tim Jackson wrote: > Christopher Stone wrote: > > > On 7/3/06, Tim Jackson <lists@xxxxxxxxxx> wrote: > >> https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=185423 > >> http://www.timj.co.uk/linux/specs/php-pear-PEAR-Command-Packaging.spec > >> http://www.timj.co.uk/linux/srpms/php-pear-PEAR-Command-Packaging-0.1.1-1.src.rpm > > > I do not like the way patches are handled in this case. I have > > reworked the spec file here: > > http://tkmame.retrogames.com/fedora-extras/php-pear-PEAR-Command-Packaging.spec > > Looks fine to me. I agree that's more readable. I disagree on this: .. Requires(post): php-pear Requires(pre): php-pear .. %post pear install --nodeps --soft --force --register-only %{xmldir}/PEAR_Command_Packaging.xml >/dev/null ||: .. %postun if [ "$1" -eq "0" ]; then pear uninstall --nodeps --ignore-errors --register-only PEAR_Command_Packaging >/dev/null ||: fi Also c.f. https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=197561 Ralf -- Fedora-packaging mailing list Fedora-packaging@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/fedora-packaging