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 mastahnke@xxxxxxxxx 2007-03-03 23:39 EST ------- >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. I see atd.init and test.pl at 0755 permissions in the source. It looks like those two files are not in upstream and added to make this package. So basically, the question is can they have better perms? I think the answer is yes. I am also still seeing a difference between debian's upstream sum and the source used for this package. I am not sure why that is. When I unpack both soruces and run a diff, I see no difference. >From the package review guidelines: - MUST: The sources used to build the package must match the upstream source, as provided in the spec URL. Reviewers should use md5sum for this task. -- 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