Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=462521 Mary Ellen Foster <mefoster@xxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-review+ --- Comment #14 from Mary Ellen Foster <mefoster@xxxxxxxxx> 2009-02-24 03:35:54 EDT --- Looks good. Just one tiny rpmlint warning from the SRPM: simplyhtml.src: W: strange-permission simplyhtml.sh 0755 A file that you listed to include in your package has strange permissions. Usually, a file should have 0644 permissions. Since you have the %attr line in the %files section, I don't think that the source file needs any special permissions itself. This is APPROVED (but please change the permissions on that one source file before importing, just for neatness. :) ) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review