Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: asterisk - The Open Source PBX https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=178922 matthias@xxxxxxxxxxxx changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |matthias@xxxxxxxxxxxx ------- Additional Comments From matthias@xxxxxxxxxxxx 2006-03-29 07:12 EST ------- A few comments : - Asterisk shouldn't run as root by default IMHO. If it's a QoS related decision, it should be up to the user/admin to switch to using root. - I've had to replace /var/run with a sub-dir to have asterisk run as non-root - I've had to add some further ugly replacing to fix lib vs. lib64 issues. - The logrotate file should be %config(noreplace) in case people edit it. - I'm not totally convinced about the need for some of the sub-packages. For instance oss, alsa, curl and sounds-default. - Why is the manually installed asterisk.h file needed? My current spec and other files : http://svn.rpmforge.net/svn/trunk/rpms/asterisk/ -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is. -- fedora-extras-list mailing list fedora-extras-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/fedora-extras-list