Bug 1396836 - staticlib in qt_lib_uitools.pri, but doesn't shipping static library
Summary: staticlib in qt_lib_uitools.pri, but doesn't shipping static library
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: qt5-qttools
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Rex Dieter
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-11-20 19:22 UTC by Igor Gnatenko
Modified: 2016-11-21 12:53 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-11-21 12:53:03 UTC
Type: Bug


Attachments (Terms of Use)

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)


Note You need to log in before you can comment on or make changes to this bug.