Merge lp:~zeller-benjamin/qtcreator-plugin-ubuntu/scopes_should_run_again into lp:qtcreator-plugin-ubuntu

Proposed by Benjamin Zeller
Status: Merged
Approved by: Zoltan Balogh
Approved revision: 426
Merged at revision: 431
Proposed branch: lp:~zeller-benjamin/qtcreator-plugin-ubuntu/scopes_should_run_again
Merge into: lp:qtcreator-plugin-ubuntu
Prerequisite: lp:~zeller-benjamin/qtcreator-plugin-ubuntu/lp1476647
Diff against target: 43 lines (+17/-2)
2 files modified
share/qtcreator/ubuntu/qml/publishpage.qml (+16/-1)
src/ubuntu/ubuntupackagestep.cpp (+1/-1)
To merge this branch: bzr merge lp:~zeller-benjamin/qtcreator-plugin-ubuntu/scopes_should_run_again
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Approve
Zoltan Balogh Approve
Review via email: mp+278810@code.launchpad.net

Commit message

- Make scopes run again
- Fix Bug lp:1427133 "Build and validate click package button disabled
without explanation"

Description of the change

- Make scopes run again
- Fix Bug lp:1427133 "Build and validate click package button disabled
without explanation"

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

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'share/qtcreator/ubuntu/qml/publishpage.qml'
2--- share/qtcreator/ubuntu/qml/publishpage.qml 2015-11-23 10:34:50 +0000
3+++ share/qtcreator/ubuntu/qml/publishpage.qml 2015-11-27 13:26:12 +0000
4@@ -104,11 +104,26 @@
5 }
6 }
7
8+ Row {
9+ id: warningsRow
10+ anchors.top: topRow.bottom
11+ anchors.horizontalCenter: parent.horizontalCenter
12+ anchors.topMargin: units.gu(2)
13+ spacing: units.gu(2)
14+ visible: !publishModel.canBuild
15+ Icon {
16+ source: "qrc:/ubuntu/images/warning.png"
17+ }
18+ Label {
19+ text: "The selected Kit does not support building click packages. Please make sure to select a Ubuntu Kit."
20+ }
21+ }
22+
23 ScrollableView {
24 id: scrollView
25 clip: true
26
27- anchors.top: topRow.bottom
28+ anchors.top: warningsRow.bottom
29 anchors.left: parent.left
30 anchors.right: parent.right
31 anchors.bottom: parent.bottom
32
33=== modified file 'src/ubuntu/ubuntupackagestep.cpp'
34--- src/ubuntu/ubuntupackagestep.cpp 2015-09-09 11:45:25 +0000
35+++ src/ubuntu/ubuntupackagestep.cpp 2015-11-27 13:26:12 +0000
36@@ -628,7 +628,7 @@
37 continue;
38 }
39
40- QString defaultSubCmd = QStringLiteral("/usr/lib/%1/unity-scopes/scoperunner '' %S ").arg(static_cast<ClickToolChain*>(tc)->gnutriplet());
41+ QString defaultSubCmd = QStringLiteral("/usr/lib/%1/unity-scopes/scoperunner %R %S ").arg(static_cast<ClickToolChain*>(tc)->gnutriplet());
42 QString commTemplate = QStringLiteral("./%S scope %1 %C")
43 .arg(manifest.name()+QStringLiteral("_")+hook.appId); //tell our script the appid
44

Subscribers

People subscribed via source and target branches