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: dialog https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225693 bugzilla@xxxxxxxxxx changed: What |Removed |Added ---------------------------------------------------------------------------- Priority|normal |medium pertusus@xxxxxxx changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |NEEDINFO Flag| |needinfo? ------- Additional Comments From pertusus@xxxxxxx 2007-06-01 04:26 EST ------- The API shouldn't be platform dependent, that's why it is wrong to have configure conditionals in header files, at least platform specific conditionals should be avoided as much as possible. It shows up especially in multilib situations since headers should be the same for both arches. This is a suitable bug report to report such issue, it is suitable to report all the package issues, even those that require talking with upstream. The resolution of those issues may not be a blocker for the merge review, but at least they must be discussed. Regarding the name, as Thomas said 1.1-20070409 (for example) is the full upstream version. I don't know exactly how it should translate in version and release, but in any case there shouldn't be the svn string in the release. -- 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