gsettings-qt fails test_reset() property count on Qt 5.7.1

Bug #1624251 reported by Timo Jyrinki
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
gsettings-qt (Ubuntu)
Invalid
Undecided
Albert Astals Cid
qtdeclarative-opensource-src (Ubuntu)
Fix Released
Undecided
Timo Jyrinki

Bug Description

--
FAIL! : GSettings::test_reset() property count
   Actual (): 0
   Expected (): 1
   Loc: [/«BUILDDIR»/gsettings-qt-0.1+16.04.20160329/tests/tst_GSettings.qml(102)]

(process:7207): GLib-GIO-WARNING **: g_settings_set_value: value for key 'test-enum' in schema 'com.canonical.gsettings.Test' is outside of valid range
--

(https://launchpadlibrarian.net/284597323/buildlog_ubuntu-yakkety-amd64.gsettings-qt_0.1+16.04.20160329-0ubuntu2~5_BUILDING.txt.gz)

More information about Qt 5.7 at https://wiki.ubuntu.com/Touch/QtTesting

Tags: qt5.7
Revision history for this message
Timo Jyrinki (timo-jyrinki) wrote :

Still fails with Qt 5.7.1, testable from the PPA.

summary: - gsettings-qt fails a test on Qt 5.7
+ gsettings-qt fails a test on Qt 5.7.1
summary: - gsettings-qt fails a test on Qt 5.7.1
+ gsettings-qt fails test_reset() property count on Qt 5.7.1
Revision history for this message
Albert Astals Cid (aacid) wrote :

It's a regression in Qt, i'm working on it

Changed in gsettings-qt (Ubuntu):
assignee: nobody → Albert Astals Cid (aacid)
status: New → In Progress
Revision history for this message
Albert Astals Cid (aacid) wrote :

Patch upstream at
 https://codereview.qt-project.org/#/c/177422

Let's see what they say

Changed in gsettings-qt (Ubuntu):
status: In Progress → Invalid
Changed in qtdeclarative-opensource-src (Ubuntu):
assignee: nobody → Albert Astals Cid (aacid)
Changed in gsettings-qt (Ubuntu):
assignee: Albert Astals Cid (aacid) → nobody
Changed in qtdeclarative-opensource-src (Ubuntu):
status: New → In Progress
Revision history for this message
Albert Astals Cid (aacid) wrote :

It's merged upstream. Timo your turn to merge back to our Qt.

Changed in gsettings-qt (Ubuntu):
assignee: nobody → Albert Astals Cid (aacid)
Changed in qtdeclarative-opensource-src (Ubuntu):
assignee: Albert Astals Cid (aacid) → Timo Jyrinki (timo-jyrinki)
Revision history for this message
Timo Jyrinki (timo-jyrinki) wrote :

Fantastic! Confirming fix in 1985 PPA (https://wiki.ubuntu.com/Touch/QtTesting).

Changed in qtdeclarative-opensource-src (Ubuntu):
status: In Progress → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package qtdeclarative-opensource-src - 5.7.1-1ubuntu1~7

---------------
qtdeclarative-opensource-src (5.7.1-1ubuntu1~7) zesty; urgency=medium

  [ Dmitry Shachnev ]
  * Merge 5.6.1-7 from unstable.

  [ Timo Jyrinki ]
  * New upstream release.
  * Drop patches now in upstream.
  * Rebase patches to apply to 5.7:
    - debian/patches/Fix-visibility-of-properties-in-value-types.patch
    - debian/patches/disable_failing_tests.patch
    - debian/patches/fix_binary_location_for_tests.patch
  * Rebase QML cache patch for 5.7 and add arm64 support.
  * Update symbols and mark private ones for 5.7.0 and 5.7.1.
  * debian/patches/Fix-SignalSpy-with-QQmlPropertyMap-signals.patch:
    - Backport from 5.8 (LP: #1624251)

 -- Timo Jyrinki <email address hidden> Wed, 07 Sep 2016 13:12:08 +0000

Changed in qtdeclarative-opensource-src (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.