[Bug 487901] Review Request: zikula - web-based CMS

bugzilla at redhat.com bugzilla at redhat.com
Sat Mar 7 07:01:05 UTC 2009


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 <herlo1 at 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.




More information about the Fedora-package-review mailing list