[Bug 188180] Review Request: qt4: Qt GUI toolkit

[bugzilla at redhat.com]

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: qt4: Qt GUI toolkit


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=188180





------- Additional Comments From laurent.rineau__fc_extra at normalesup.org  2006-05-15 09:57 EST -------
Actually, libQtAssistant.a search assistant in the path. The following patch 
is required:
--- qt-x11-opensource-src-4.1.2/tools/assistant/lib/qassistantclient.cpp~      
2006-03-20 14:04:38.000000000 +0100
+++ qt-x11-opensource-src-4.1.2/tools/assistant/lib/qassistantclient.cpp       
2006-05-15 15:42:07.000000000 +0200
@@ -170,7 +170,7 @@
     else {
         QFileInfo fi( path );
         if ( fi.isDir() )
-            assistantCommand = path + "/assistant";
+            assistantCommand = path + "/assistant4";
         else
             assistantCommand = path;
     }

One problem: assistant should perhaps go to qt4 (not -devel), or a -assistant 
sub package (I would prefere the latter), because of libQtAssistant.

As regard the -L%{_libdir}/mysql, sorry for that: I had not noticed that 
-lmysql was not in that .pc file.

As regards the necessity to have a -doc subpackage, du(1) shows:
$ du -s /usr/lib64/qt4/{examples,demos,lib,plugins,include,doc}/. | sort -n
2636    /usr/lib64/qt4/plugins/.
4392    /usr/lib64/qt4/demos/.
13596   /usr/lib64/qt4/examples/.
13828   /usr/lib64/qt4/lib/.
15892   /usr/lib64/qt4/include/.
61756   /usr/lib64/qt4/doc/.

It seems mandatory.


-- 
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.