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=487901





--- Comment #6 from Clint Savage <her...@gmail.com>  2009-03-07 02:01:02 EDT ---
I'm pretty happy with the results in the latest rpm/spec you provided.  rpmlint
is clean.  However, might I suggest you put the selinux changes into a %pre
section?  It makes complete sense as the selinux context and parameter should
be set prior to adding files which will get the context.

That's the last gripe I have.  Otherwise, everything works from install to
activation with a database.

Cheers,

Clint

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

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

Reply via email to