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=682414 Jussi Lehtola <jussi.lehtola@xxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+ --- Comment #12 from Jussi Lehtola <jussi.lehtola@xxxxxx> 2011-03-11 12:56:58 EST --- reptyr.src: W: strange-permission reptyr-snapshot.sh 0755L reptyr.src: W: invalid-url Source0: reptyr-20110311git919fff7.tar.bz2 3 packages and 0 specfiles checked; 0 errors, 2 warnings. You can fix the permission warning by setting 644 (you can always run the script with sh script.sh). ** I highly recommend making the comment more verbose by changing # Source0 generated with Source99 to something of the form # Upstream has not released stable tarballs yet, so we use git snapshots. # The script in Source99 is used to generate a git snapshot tarball. which is IMHO more in line with http://fedoraproject.org/wiki/Packaging/SourceURL#Using_Revision_Control ** Looks like the blockers have been fixed. This package has been APPROVED -- 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. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review