[Bug 225237] Merge Review: acpid

bugzilla at redhat.com bugzilla at redhat.com
Thu Feb 8 02:02:11 UTC 2007


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 at tummy.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|kevin at tummy.com             |pknirsch at redhat.com
               Flag|fedora-review?              |fedora-review-




------- Additional Comments From kevin at tummy.com  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.




More information about the Fedora-package-review mailing list