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: acpid https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225237 kevin@xxxxxxxxx changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|kevin@xxxxxxxxx |pknirsch@xxxxxxxxxx Flag|fedora-review? |fedora-review- ------- Additional Comments From kevin@xxxxxxxxx 2007-02-07 21:01 EST ------- Thanks for the quick fixes and reply... 1. good. ok. 2. good. ok. 3.1 good. ok. 3.2 I suppose so. ok. 3.3 good. ok. 3.4 Not sure why it's complaining about that 0755 init script. ;( good on the prereqs. 3.5 good. ok. 4.1 Humm...I haven't seen that used for /var/log/FOO files before... I guess there aren't many packages that make such files. Looking at for example vixie-cron, it just has no mention of the log file at all. I guess the question is: do we want this package to remove the /var/log/acpid log when the package is removed? Or should it linger around? We should probibly decide what behavior is desired and make all the packages that create /var/log files do the same thing... In fact I am hard pressed to find another package that does things the way this one does. Do you have any example? scrollkeeper appears to ghost the log file, but yum, x.org, vixie-cron just don't claim anything for their log file (ie, it's unowned). 4.2 good. ok. 4.3 good. ok. Setting it back to you to comment on the 4.1 issue. When you do, please assign it back to me and change the fedora-review flag back to ? Thanks again for the quick fixes... -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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