[Bug 226351] Merge Review: qt
bugzilla at redhat.com
bugzilla at redhat.com
Wed Feb 7 17:34:56 UTC 2007
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: qt
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226351
rdieter at math.unl.edu changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |NEEDINFO
Flag|fedora-review? |needinfo?(than at redhat.com),
| |fedora-review-
------- Additional Comments From rdieter at math.unl.edu 2007-02-07 12:34 EST -------
Guidelines-wise, looks good, only a few relatively small SHOULD (optional) items
to consider:
* icons. SHOULD try to follow icon-spec here, and put items in/under
%_datadir/icons/hicolor/ somewhere instead of using %_datadir/pixmaps
* .desktop files: using namespace/vendor of qt-* *and* using '3' appended to
.desktop file names is redundant. Consider removing the '3' from the .desktop
file names.
* consider changing qtdir from %{_libdir}/qt-3.3 to just %_libdir/qt3. It would
then be consistent with qt4 packaging. This is a big change that may induce
pain in other packages that currently hard-code the old location, so it may not
be worth it.
* -devel-docs subpkg SHOULD:
1. should simply be -doc (corrolary: and have no dependency on -devel)
2. assistant should be included here.
* SHOULD avoid some of the multilib ickiness, and necessity to use
/etc/ld.so.conf.d/qt*, by simply using
./configure -libdir %{_libdir}
ie, putting qt's libs into %_libdir directly.
qt4 has been using these latter 2 items with success.
--
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.
More information about the Fedora-package-review
mailing list