[Bug 1620173] Re: Qt5 5.6 regression: conversion from QVariant to QString is broken
** Changed in: qtbase-opensource-src (Ubuntu) Assignee: Timo Jyrinki (timo-jyrinki) => (unassigned) -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1620173 Title: Qt5 5.6 regression: conversion from QVariant to QString is broken To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/qtbase-opensource-src/+bug/1620173/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1620173] Re: Qt5 5.6 regression: conversion from QVariant to QString is broken
Unfortunately fixing this would be causing true regressions in at least ktexteditor and pyqt5: https://requests.ci- train.ubuntu.com/static/britney/landing-1920/yakkety/excuses.html https://objectstorage.prodstack4-5.canonical.com/v1/AUTH_77e2ada1e7a84929a74ba3b87153c0ac/autopkgtest-yakkety-ci-train-ppa-service-landing-1920/yakkety/amd64/k/ktexteditor/20160914_111856@/log.gz https://objectstorage.prodstack4-5.canonical.com/v1/AUTH_77e2ada1e7a84929a74ba3b87153c0ac/autopkgtest-yakkety-ci-train-ppa-service-landing-1920/yakkety/amd64/p/pyqt5/20160914_140056@/log.gz I'm not sure if rebuilding qtxmlpatterns would be enough or what other kind of problems there could be lurking. Since 16.10 release is coming closer, I think the set of these three patches (that remove a symbol among else) is too risky. The fix would eventually be in Qt 5.7 in Ubuntu 17.04, unless of course a less invasive fix could be had for 16.10. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1620173 Title: Qt5 5.6 regression: conversion from QVariant to QString is broken To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/qtbase-opensource-src/+bug/1620173/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1620173] Re: Qt5 5.6 regression: conversion from QVariant to QString is broken
Also https://codereview.qt-project.org/#/c/139057/ is needed. Next time, full debdiff with working and rebased patches please :) -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1620173 Title: Qt5 5.6 regression: conversion from QVariant to QString is broken To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/qtbase-opensource-src/+bug/1620173/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1620173] Re: Qt5 5.6 regression: conversion from QVariant to QString is broken
Timo, yes sorry, you've got the proper link :) -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1620173 Title: Qt5 5.6 regression: conversion from QVariant to QString is broken To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/qtbase-opensource-src/+bug/1620173/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1620173] Re: Qt5 5.6 regression: conversion from QVariant to QString is broken
The patches do not apply to 5.6 as is, although it doesn't look too complicated failing. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1620173 Title: Qt5 5.6 regression: conversion from QVariant to QString is broken To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/qtbase-opensource-src/+bug/1620173/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1620173] Re: Qt5 5.6 regression: conversion from QVariant to QString is broken
I think you meant https://codereview.qt-project.org/#/c/104737/32/ for patch 1 :D -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1620173 Title: Qt5 5.6 regression: conversion from QVariant to QString is broken To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/qtbase-opensource-src/+bug/1620173/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1620173] Re: Qt5 5.6 regression: conversion from QVariant to QString is broken
Thanks for the report and finding both of the commits! ** Changed in: qtbase-opensource-src (Ubuntu) Assignee: (unassigned) => Timo Jyrinki (timo-jyrinki) ** Changed in: qtbase-opensource-src (Ubuntu) Status: New => Confirmed -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1620173 Title: Qt5 5.6 regression: conversion from QVariant to QString is broken To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/qtbase-opensource-src/+bug/1620173/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1620173] Re: Qt5 5.6 regression: conversion from QVariant to QString is broken
Ok, while trying to create a debdiff (with the help of Simon Q on IRC, thanks :) ), I realized the above is effectively part 2, and part 1 is this commit: Part 1: https://codereview.qt-project.org/#/c/127924/12//ALL,unified Part 2: https://codereview.qt-project.org/#/c/127924/12//ALL,unified (same as the .patch link above) This fixes QTBUG-44039, QTBUG-43885, and QTBUG-47192. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1620173 Title: Qt5 5.6 regression: conversion from QVariant to QString is broken To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/qtbase-opensource-src/+bug/1620173/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1620173] Re: Qt5 5.6 regression: conversion from QVariant to QString is broken
** Description changed: A regression was introduced in Qt5 5.5 - fixed in 5.7 - whereas conversion from QVariant to QString is broken for double and float values (you can see more details on this here: https://bugreports.qt.io/browse/QTBUG-47192). The patch fixing this issue is the following: http://code.qt.io/cgit/qt/qtbase.git/patch/src/corelib/tools/qlocale.h?id=726fed0d67013cbfac7921d3d4613ca83406fb0f Would it be possible to apply this patch against the package (based on - Qt5 5.6.1) Ubuntu 16.10 will October? It's a pretty ugly bug, which - needs addressing. Hopefully the Qt devs can eventually merge this patch - onto their 5.6 LTS branch. + Qt5 5.6.1) Ubuntu 16.10 will ship in October? It's a pretty ugly bug, + which needs addressing. Hopefully the Qt devs can eventually merge this + patch onto their 5.6 LTS branch. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1620173 Title: Qt5 5.6 regression: conversion from QVariant to QString is broken To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/qtbase-opensource-src/+bug/1620173/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
