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=459535 Aurelien Bompard <gauret@xxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |gauret@xxxxxxx AssignedTo|nobody@xxxxxxxxxxxxxxxxx |gauret@xxxxxxx Flag| |fedora-review+ --- Comment #9 from Aurelien Bompard <gauret@xxxxxxx> 2009-01-10 08:34:29 EDT --- Since there's been no action for over a month, I'm taking over the review :) Review for release 3.fc10: * RPM name is OK * Source backup-manager-0.7.7.tar.gz is the same as upstream * rpmlint looks OK * File list looks OK * Works fine APPROVED Two small things though: - the actual license is GPLv2+ (see the head of the backup-manager file). Please fix the RPM tag. - in the comment you add to the conf file, please switch "Specific RPM" to "RPM-specific", this is better english. Since those changes are so trivial, no need to resubmit a package here, just do it right before or after importing. Thanks Guillaume. -- 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