Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: mcpp - Alternative C/C++ preprocessor https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=237373 mtasaka@xxxxxxxxxxxxxxxxxxx changed: What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+ ------- Additional Comments From mtasaka@xxxxxxxxxxxxxxxxxxx 2007-05-09 12:08 EST ------- Well, * mcpp packaging/performance is okay * For pre-reviewing: - for referencer, actually Deji Akingunola is a good packager and the spec file seems no problem at a glance (I didn't check in detail) - for ddccontrol, according to your comment the spec file seems to have many problem. I guess more problem may be found.... (for this package, I have not seen the spec file yet) (In reply to comment #16) > Writing review is a hard task than I expected! Perhaps. While I have reviewed about 140 packages and have submitted about 40 review requests, reviewing srpm was very difficult for me first. Okay!! ------------------------------------------------------ This package (mcpp) is APPROVED by me ------------------------------------------------------ Please follow: http://fedoraproject.org/wiki/PackageMaintainers/Join from "Get a Fedora Account". I will receive a mail which notices that you need a sponsor and then I will sponsor you. NOTE: devel repository is currently under the way of the merge of Core and Extras and you may meet some trouble. If you have some trouble, please let me know. -- 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