Bug 380435 - Latest kirigami 2 from master breaks discover & new systemsettings sidebar view in Qt 5.7.1
Summary: Latest kirigami 2 from master breaks discover & new systemsettings sidebar vi...
Status: RESOLVED FIXED
Alias: None
Product: frameworks-kirigami
Classification: Frameworks and Libraries
Component: general (show other bugs)
Version: Master
Platform: Neon Linux
: NOR major
Target Milestone: Not decided
Assignee: Marco Martin
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-06-01 18:06 UTC by Rik Mills
Modified: 2017-06-02 11:31 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
breakage (248.49 KB, image/png)
2017-06-01 18:06 UTC, Rik Mills
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Rik Mills 2017-06-01 18:06:06 UTC
Created attachment 105825 [details]
breakage

Latest kirigami 2 from master breaks discover & new systemsettings sidebar view in Qt 5.7.1

This is evident on both Neon and Kubuntu CI builds.

1st bad commit appears on bisection to be:

https://cgit.kde.org/kirigami.git/commit/?id=0d0755f3d402270a809652b74b656a0ba3e5742c

No compilation warnings about any incompatibility with Qt 5.7 are shown, if this is deliberate.
Comment 1 Rik Mills 2017-06-01 18:20:07 UTC
Please also see the paste below of the console output shown in the attached screenshot

https://paste.kde.org/phpjhvzj7
Comment 2 Aleix Pol 2017-06-02 11:31:07 UTC
Git commit 655bdb21b9185268b87a1ca60a0bf0ec1a563104 by Aleix Pol.
Committed on 02/06/2017 at 11:29.
Pushed by apol into branch 'master'.

Fix issue in Qt 5.7

Summary:
qml parser doesn't like having namespaced object types in the declaration
of a property. Use QtObject instead.

Test Plan:
Tests pass, applied it on my neon machine (sudo vim /usr...) and
it seemed to work

Reviewers: #kirigami, jriddell, davidedmundson

Reviewed By: davidedmundson

Subscribers: plasma-devel

Tags: #kirigami

Differential Revision: https://phabricator.kde.org/D6065

M  +1    -1    src/styles/org.kde.desktop.plasma/Units.qml
M  +1    -1    src/styles/org.kde.desktop/Units.qml

https://commits.kde.org/kirigami/655bdb21b9185268b87a1ca60a0bf0ec1a563104