Merge lp:~zeller-benjamin/ubuntu-ui-toolkit/trusty-builds into lp:ubuntu-ui-toolkit/staging

Proposed by Benjamin Zeller
Status: Needs review
Proposed branch: lp:~zeller-benjamin/ubuntu-ui-toolkit/trusty-builds
Merge into: lp:ubuntu-ui-toolkit/staging
Diff against target: 47 lines (+9/-12)
3 files modified
apicheck/apicheck.pro (+2/-4)
src/Ubuntu/UbuntuToolkit/menubar.cpp (+5/-5)
ubuntu-ui-toolkit-launcher/ubuntu-ui-toolkit-launcher.pro (+2/-3)
To merge this branch: bzr merge lp:~zeller-benjamin/ubuntu-ui-toolkit/trusty-builds
Reviewer Review Type Date Requested Status
ubuntu-sdk-build-bot continuous-integration Needs Fixing
Zoltan Balogh Pending
Review via email: mp+304851@code.launchpad.net

This proposal supersedes a proposal from 2016-09-03.

Commit message

- Some out of source build fixes
- Implicit casting of QPointer<QObject> to QObject* does not work on trusty

Description of the change

- Some out of source build fixes
- Implicit casting of QPointer<QObject> to QObject* does not work on trusty

To post a comment you must log in.
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Needs Fixing (continuous-integration)

Unmerged revisions

2090. By Benjamin Zeller

Implicit casting of QPointer<QObject> to QObject* does not work on trusty

2089. By Benjamin Zeller

Fix out of source builds

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'apicheck/apicheck.pro'
--- apicheck/apicheck.pro 2016-01-22 09:31:54 +0000
+++ apicheck/apicheck.pro 2016-09-03 13:07:00 +0000
@@ -4,7 +4,5 @@
4QT += core-private gui-private testlib quick-private4QT += core-private gui-private testlib quick-private
5CONFIG += no_keywords5CONFIG += no_keywords
6SOURCES += apicheck.cpp6SOURCES += apicheck.cpp
7installPath = $$[QT_INSTALL_LIBS]/ubuntu-ui-toolkit7target.path = $$[QT_INSTALL_LIBS]/ubuntu-ui-toolkit
8apicheck.path = $$installPath8INSTALLS += target
9apicheck.files = apicheck
10INSTALLS += $$TARGET
119
=== modified file 'src/Ubuntu/UbuntuToolkit/menubar.cpp'
--- src/Ubuntu/UbuntuToolkit/menubar.cpp 2016-08-25 14:06:06 +0000
+++ src/Ubuntu/UbuntuToolkit/menubar.cpp 2016-09-03 13:07:00 +0000
@@ -252,11 +252,11 @@
252 , m_target(target)252 , m_target(target)
253 , m_inserted(false)253 , m_inserted(false)
254{254{
255 connect(m_target, &Menu::visibleChanged, this, &PlatformMenuWrapper::updateVisible);255 connect(m_target.data(), &Menu::visibleChanged, this, &PlatformMenuWrapper::updateVisible);
256 connect(m_target, &Menu::textChanged, this, &PlatformMenuWrapper::updateText);256 connect(m_target.data(), &Menu::textChanged, this, &PlatformMenuWrapper::updateText);
257 connect(m_target, &Menu::enabledChanged, this, &PlatformMenuWrapper::updateEnabled);257 connect(m_target.data(), &Menu::enabledChanged, this, &PlatformMenuWrapper::updateEnabled);
258 connect(m_target, &Menu::iconSourceChanged, this, &PlatformMenuWrapper::updateIcon);258 connect(m_target.data(), &Menu::iconSourceChanged, this, &PlatformMenuWrapper::updateIcon);
259 connect(m_target, &Menu::iconNameChanged, this, &PlatformMenuWrapper::updateIcon);259 connect(m_target.data(), &Menu::iconNameChanged, this, &PlatformMenuWrapper::updateIcon);
260260
261 syncPlatformMenu();261 syncPlatformMenu();
262}262}
263263
=== modified file 'ubuntu-ui-toolkit-launcher/ubuntu-ui-toolkit-launcher.pro'
--- ubuntu-ui-toolkit-launcher/ubuntu-ui-toolkit-launcher.pro 2016-07-22 10:47:24 +0000
+++ ubuntu-ui-toolkit-launcher/ubuntu-ui-toolkit-launcher.pro 2016-09-03 13:07:00 +0000
@@ -12,6 +12,5 @@
12CONFIG += no_keywords c++1112CONFIG += no_keywords c++11
13SOURCES += launcher.cpp13SOURCES += launcher.cpp
14installPath = $$[QT_INSTALL_PREFIX]/bin14installPath = $$[QT_INSTALL_PREFIX]/bin
15launcher.path = $$installPath15target.path = $$installPath
16launcher.files = ubuntu-ui-toolkit-launcher16INSTALLS += target
17INSTALLS += launcher

Subscribers

People subscribed via source and target branches