Merge lp:~kalikiana/ubuntu-ui-toolkit/tellMeAboutIt into lp:ubuntu-ui-toolkit

Proposed by Cris Dywan
Status: Merged
Approved by: Tim Peeters
Approved revision: 840
Merged at revision: 929
Proposed branch: lp:~kalikiana/ubuntu-ui-toolkit/tellMeAboutIt
Merge into: lp:ubuntu-ui-toolkit
Prerequisite: lp:~kalikiana/ubuntu-ui-toolkit/unit_x11_fixes
Diff against target: 76 lines (+20/-5)
5 files modified
modules/Ubuntu/Components/deployment.pri (+4/-1)
modules/Ubuntu/Layouts/deployment.pri (+4/-1)
modules/Ubuntu/Test/deployment.pri (+4/-1)
tests/unit/add_qmlmakecheck.pri (+4/-1)
tests/unit/runtest.sh (+4/-1)
To merge this branch: bzr merge lp:~kalikiana/ubuntu-ui-toolkit/tellMeAboutIt
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Approve
Tim Peeters Approve
Review via email: mp+199439@code.launchpad.net

Commit message

Suppress errors to work-around font error messages

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Gerry Boland (gerboland) wrote :

Shouldn't we have the Qt libs fixed to look for fonts in correct location?

Revision history for this message
Cris Dywan (kalikiana) wrote :

Yes. However as you may be aware this bug has been left unaddressed for months.

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Tim Peeters (tpeeters) wrote :

okay

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'modules/Ubuntu/Components/deployment.pri'
2--- modules/Ubuntu/Components/deployment.pri 2013-09-25 13:47:49 +0000
3+++ modules/Ubuntu/Components/deployment.pri 2014-01-20 16:27:44 +0000
4@@ -44,6 +44,9 @@
5
6 plugins_qmltypes.path = $$installPath
7 plugins_qmltypes.files = plugins.qmltypes
8-plugins_qmltypes.extra = $$[QT_INSTALL_BINS]/qmlplugindump -notrelocatable Ubuntu.Components 0.1 ../../ > $(INSTALL_ROOT)/$$installPath/plugins.qmltypes
9+# Silence spam on stderr due to fonts
10+# https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1256999
11+# https://bugreports.qt-project.org/browse/QTBUG-36243
12+plugins_qmltypes.extra = $$[QT_INSTALL_BINS]/qmlplugindump -notrelocatable Ubuntu.Components 0.1 ../../ 2>/dev/null > $(INSTALL_ROOT)/$$installPath/plugins.qmltypes
13
14 INSTALLS += qmldir_file qml_files js_files artwork_files listitems_files listitems_artwork_files popups_files picker_files plugins_qmltypes
15
16=== modified file 'modules/Ubuntu/Layouts/deployment.pri'
17--- modules/Ubuntu/Layouts/deployment.pri 2013-12-09 16:26:24 +0000
18+++ modules/Ubuntu/Layouts/deployment.pri 2014-01-20 16:27:44 +0000
19@@ -13,6 +13,9 @@
20
21 plugins_qmltypes.path = $$installPath
22 plugins_qmltypes.files = plugins.qmltypes
23-plugins_qmltypes.extra = $$[QT_INSTALL_BINS]/qmlplugindump -notrelocatable Ubuntu.Layouts 0.1 ../../ > $(INSTALL_ROOT)/$$installPath/plugins.qmltypes
24+# Silence spam on stderr due to fonts
25+# https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1256999
26+# https://bugreports.qt-project.org/browse/QTBUG-36243
27+plugins_qmltypes.extra = $$[QT_INSTALL_BINS]/qmlplugindump -notrelocatable Ubuntu.Layouts 0.1 ../../ 2>/dev/null > $(INSTALL_ROOT)/$$installPath/plugins.qmltypes
28
29 INSTALLS += qmldir_file plugins_qmltypes
30
31=== modified file 'modules/Ubuntu/Test/deployment.pri'
32--- modules/Ubuntu/Test/deployment.pri 2013-07-24 12:34:45 +0000
33+++ modules/Ubuntu/Test/deployment.pri 2014-01-20 16:27:44 +0000
34@@ -15,6 +15,9 @@
35
36 plugins_qmltypes.path = $$installPath
37 plugins_qmltypes.files = plugins.qmltypes
38-plugins_qmltypes.extra = $$[QT_INSTALL_BINS]/qmlplugindump -notrelocatable Ubuntu.Test 0.1 ../../ > $(INSTALL_ROOT)/$$installPath/plugins.qmltypes
39+# Silence spam on stderr due to fonts
40+# https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1256999
41+# https://bugreports.qt-project.org/browse/QTBUG-36243
42+plugins_qmltypes.extra = $$[QT_INSTALL_BINS]/qmlplugindump -notrelocatable Ubuntu.Test 0.1 ../../ 2>/dev/null > $(INSTALL_ROOT)/$$installPath/plugins.qmltypes
43
44 INSTALLS += qmldir_file plugins_qmltypes
45
46=== modified file 'tests/unit/add_qmlmakecheck.pri'
47--- tests/unit/add_qmlmakecheck.pri 2013-12-13 15:50:35 +0000
48+++ tests/unit/add_qmlmakecheck.pri 2014-01-20 16:27:44 +0000
49@@ -8,7 +8,10 @@
50 check.commands += ../../unit/runtest.sh $${TARGET} $${TEST} minimal;
51 }
52 check.commands += cd ../../..;
53-check.commands += qmlplugindump Ubuntu.Components 0.1 modules > plugins.qmltypes;
54+# Silence spam on stderr due to fonts
55+# https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1256999
56+# https://bugreports.qt-project.org/browse/QTBUG-36243
57+check.commands += qmlplugindump Ubuntu.Components 0.1 modules 2>/dev/null > plugins.qmltypes;
58 # Palette gets included in Qt 5.2 qmlplugindump even though it's qml
59 check.commands += BUILTINS=QQuick,QQml,U1db::,Palette python tests/qmlapicheck.py modules/Ubuntu/Components/qmldir modules/Ubuntu/Components/Colors/UbuntuColors.qml modules/Ubuntu/Components/*/qmldir plugins.qmltypes > components.api.new;
60 check.commands += diff -Fqml -u components.api components.api.new || exit 1; cd tests/unit
61
62=== modified file 'tests/unit/runtest.sh'
63--- tests/unit/runtest.sh 2013-12-09 14:02:08 +0000
64+++ tests/unit/runtest.sh 2014-01-20 16:27:44 +0000
65@@ -38,7 +38,10 @@
66 function execute_test_cmd {
67 echo "Executing $_CMD $_ARGS"
68 if [ $DISPLAY ]; then
69- QML2_IMPORT_PATH=../../../modules:$QML2_IMPORT_PATH UBUNTU_UI_TOOLKIT_THEMES_PATH=../../../modules $_CMD $_ARGS
70+ # https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1256999
71+ # https://bugreports.qt-project.org/browse/QTBUG-36243
72+ QML2_IMPORT_PATH=../../../modules:$QML2_IMPORT_PATH UBUNTU_UI_TOOLKIT_THEMES_PATH=../../../modules \
73+ $_CMD $_ARGS 2>&1 | grep -v 'QFontDatabase: Cannot find font directory'
74 else
75 echo "Skipped because no DISPLAY available"
76 fi

Subscribers

People subscribed via source and target branches

to status/vote changes: