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: at https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225288 ------- Additional Comments From tibbs@xxxxxxxxxxx 2007-02-20 10:50 EST ------- Let me offer some comments on the rpmlint output. W: at strange-permission atd.init 0775 W: at strange-permission test.pl 0755 Generally these aren't worth bothering with, but having a file group writable in your checkout could be problematic. I don't see that, but I think my umask doesn't allow it. Someone should try to understand where this is coming from. E: at non-readable /etc/pam.d/atd 0640 I think this is OK, albeit different from what most other packages do. (They use 0644). E: at non-standard-dir-perm /var/spool/at/spool 0700 E: at non-standard-dir-perm /var/spool/at 0700 E: at non-readable /etc/at.deny 0600 E: at non-readable /var/spool/at/.SEQ 0600 These are necessitated by security. W: at hidden-file-or-dir /var/spool/at/.SEQ That's just the file that at uses; it's OK for it to be hidden. E: at setuid-binary /usr/bin/at root 04755 E: at non-standard-executable-perm /usr/bin/at 04755 These are necessary. W: at dangerous-command-in-%post chown I'm not really sure why these are here as opposed to just being part of %files. Perhaps rpm would keep creating .SEQ.rpmnew files endlessly otherwise? If so then I think it's OK. W: at service-default-enabled /etc/rc.d/init.d/atd It's allowable for a service to be on by default, especially in the case of a daemon that everyone expects to be there. -- 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