Bug 2809 - qt4.inc defines packages for fonts that are only useful for qt4-embedded
Summary: qt4.inc defines packages for fonts that are only useful for qt4-embedded
Status: VERIFIED FIXED
Alias: None
Product: OE-Core
Classification: Build System, Metadata & Runtime
Component: graphics (show other bugs)
Version: 1.2.1
Hardware: x86 Multiple
: Medium normal
Target Milestone: 1.3 M3
Assignee: Paul Eggleton
QA Contact:
URL:
Whiteboard:
Depends on:
Blocks:
 
Reported: 2012-07-20 17:31 UTC by Marc Ferland
Modified: 2012-08-31 12:09 UTC (History)
4 users (show)

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Marc Ferland 2012-07-20 17:31:52 UTC
Hi,

The qt4.inc file defines some packages like ${QT_BASE_NAME}-fonts, ${QT_BASE_NAME}-fonts-ttf-vera, etc. These are only installed when doing an "embedded" type build. This means they are always empty for qt4-x11 builds.

I noticed this when trying to add qt4-fonts-ttf-vera to my own task-qt4.bb. The task failed with a missing dependency.

| Processing locale-base-en-us...
| Processing locale-base-en-gb...
| Processing task-core-boot...
| Processing task-core-apps-console...
| Processing pollpower...
| Processing task-core-ssh-dropbear...
| Processing ntpclient...
| Processing zypper...
| Processing task-core-tools-testapps...
| Processing task-core-x11-base...
| Processing task-core-apps-x11-core...
| Processing task-qt4-base...
| Processing task-base-extended...
| Processing rpm...
| Processing project3...
| error: Failed dependencies:
|       qt4-fonts-ttf-dejavu is needed by task-qt4-base-1.0-r2.all
|       qt4-fonts-ttf-vera is needed by task-qt4-base-1.0-r2.all

I also asked the qt-interest mailing list and they confirmed that these packages were only useful on embedded builds (http://lists.qt-project.org/pipermail/interest/2012-July/003062.html).

I think these packages should only be defined in qt4-embedded.inc instead of qt4.inc.
Comment 1 Paul Eggleton 2012-07-30 23:33:12 UTC
Patch sent for review:

http://patchwork.openembedded.org/patch/33325/
Comment 2 Paul Eggleton 2012-07-31 13:58:25 UTC
Fix merged in revision 07c93ba0ce2edbdd5766ece2c0e0f6099a7a5884.
Comment 3 Stoicescu Cornel 2012-08-31 12:09:00 UTC
Verified on 07c93ba0ce2edbdd5766ece2c0e0f6099a7a5884. The fonts packages are removed from qt4.inc and added to qt4-embeddd.inc