Bug 487075
Summary: | silent ABI change in qt breaks po2xml (renamed XML parser features) | ||||||
---|---|---|---|---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Hans Ulrich Niedermann <rhbugs> | ||||
Component: | qt | Assignee: | Than Ngo <than> | ||||
Status: | CLOSED RAWHIDE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> | ||||
Severity: | medium | Docs Contact: | |||||
Priority: | low | ||||||
Version: | rawhide | CC: | arbiter, jreznik, kevin, ltinkl, rdieter, than, tuxbrewr | ||||
Target Milestone: | --- | ||||||
Target Release: | --- | ||||||
Hardware: | All | ||||||
OS: | Linux | ||||||
Whiteboard: | |||||||
Fixed In Version: | Doc Type: | Bug Fix | |||||
Doc Text: | Story Points: | --- | |||||
Clone Of: | Environment: | ||||||
Last Closed: | 2009-03-03 14:02:43 UTC | Type: | --- | ||||
Regression: | --- | Mount Type: | --- | ||||
Documentation: | --- | CRM: | |||||
Verified Versions: | Category: | --- | |||||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||
Cloudforms Team: | --- | Target Upstream Version: | |||||
Embargoed: | |||||||
Bug Depends On: | |||||||
Bug Blocks: | 486162, 487078 | ||||||
Attachments: |
|
Description
Hans Ulrich Niedermann
2009-02-24 01:06:39 UTC
I'd blame Qt for breaking backwards compatibility there. They should be converting trolltech.com URLs to qtsoftware.com URLs automatically, it's pretty easy to do. Something like: feature.replace(QRegExp(QLatin1String("^http://trolltech.com/")), "http://qtsoftware.com/"); or, probably more efficient: if (feature.startsWith(QLatin1String("http://trolltech.com/")) feature.replace(0, 21, "http://qtsoftware.com/"); at an appropriate place in Qt should fix it. For the record (and while I still have the file open in my editor)... That would affect the methods QXmlSimpleReader::feature() QXmlSimpleReader::setFeature() QXmlSimpleReader::hasFeature() in the file qt-x11-opensource-src-4.5.0-rc1/src/xml/sax/qxml.cpp Created attachment 333412 [details]
UNTESTED patch trying to fix the issue
I could not actually test this patch yet, as qt is taking ages to compile. However, someone else might want to so something with this in the mean time...
At the very least, someone with knowledge of C++, qt's string API and how to create and prevent memory leaks and similar fun things while using the two should take a good look at the patch before it is included in any packages or even upstream qt.
Working on a newer qt snapshot (20090224), and it looks fixed upstream. ok, qt-4.5.0-0.8 should fix this, building now... should land soon in a friendly rawhide mirror near you. boo, the -0.8 build failed on the buildsys, more snapshot breakage. I'll try to fix that asap. woo, qt-4.5.0-1 (final) released today, and than heroically built it for rawhide. |