Bug 1396836

Summary:	staticlib in qt_lib_uitools.pri, but doesn't shipping static library
Product:	[Fedora] Fedora	Reporter:	Igor Gnatenko <ignatenko>
Component:	qt5-qttools	Assignee:	Rex Dieter <rdieter>
Status:	CLOSED RAWHIDE	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	rawhide	CC:	jgrulich, jreznik, me, rdieter, rnovacek, than
Target Milestone:	---	Keywords:	Reopened
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2016-11-21 12:53:03 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:

Description Igor Gnatenko 2016-11-20 19:22:47 UTC

$ grep "QT.uitools.module_config" /usr/lib64/qt5/mkspecs/modules/qt_lib_uitools.pri
QT.uitools.module_config = v2 staticlib


staticlib in module_config breaks usage of this library through qmake:
make[2]: *** No rule to make target '/usr/lib64/libQt5UiTools.a', needed by '../../lib/libPythonQt_QtAll.so.1.0.0'.  Stop.

Comment 1 Igor Gnatenko 2016-11-20 19:24:35 UTC

actually we ship static lib, but in different subpackage (qt5-qttools-static), but there is no hard requires from devel subpkg.

Comment 2 Igor Gnatenko 2016-11-20 19:29:47 UTC

also I'm wondering why not all headers are included into -devel subpkg...

Comment 3 Rex Dieter 2016-11-21 00:23:33 UTC

It's simply sub pkg, include qt5-qttools-static, if you need it.

Minor bug her is that we probably ought to include /usr/lib64/qt5/mkspecs/modules/qt_lib_uitools.pri
in -static too

Comment 4 Igor Gnatenko 2016-11-21 09:12:10 UTC

(In reply to Rex Dieter from comment #3)
> It's simply sub pkg, include qt5-qttools-static, if you need it.
> 
> Minor bug her is that we probably ought to include
> /usr/lib64/qt5/mkspecs/modules/qt_lib_uitools.pri
> in -static too
Well, if we ship that pri in -devel it is simply broken as qmake always tries to use static lib which is not present.

Comment 5 Rex Dieter 2016-11-21 12:47:37 UTC

Sure, that's a bug worth fixing (and I acknowledged that).

I'll work on moving it as previously threatened to -static subpkg

Comment 6 Rex Dieter 2016-11-21 12:53:03 UTC

%changelog
* Mon Nov 21 2016 Rex Dieter <rdieter> - 5.7.0-4
- -static: move qt_lib_uitools*.pri here (#1396836)