Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Merge Review: kdevelop https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225967 rdieter@xxxxxxxxxxxx changed: What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+ ------- Additional Comments From rdieter@xxxxxxxxxxxx 2007-03-08 11:42 EST ------- 4. SHOULD: package c_cpp_references-2.0.2 separately, imo. (ie, why build and upgrade users for this static item for *every* kdevelop rebuild?). 5. SHOULD: omit configure flags: --with-qt-libraries=$QTDIR/lib \ --with-qt-includes=$QTDIR/include \ --with-extra-libs=%{_libdir} I don't think they're needed (anymore). the only MUST item, is relatively minor, and I'll trust you to adjust that post-review, so APPROVED. -- 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