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=525005 Jason Tibbitts <tibbs@xxxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+ --- Comment #16 from Jason Tibbitts <tibbs@xxxxxxxxxxx> 2009-10-15 00:43:59 EDT --- OK, I've finally managed to find some time. It looks like, in the interim, someone else has offered to sponsor you. Is that correct? If so, it doesn't look as though that's happened yet and I guess this review will need to sit until that's finished. In any case, I'd want to see the Ivan review get finished up as well before sponsoring you myself. Anyway: The License: tag is correct now. The license file(s) are included as documentation in main package. rpmlint complaints are OK. And in addition you've given the patch a descriptive name. So at this point I'd approve the package. It would still be nice to have the upstream status of that patch, but it's not mandatory. Since I'm apt to run out of time again, I'll go ahead and approve this and you'll be able to make a CVS request and check it in once you've been sponsored. 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. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review