262939 2011-01-12 12:52:48 +0000 Kateviewinternal does not check for availability of Qt accessibility module. 2011-05-02 19:41:37 +0000 1 1 2 Applications kate part unspecified Compiled Sources All RESOLVED FIXED NOR normal --- 0 aanisimov kwrite-bugs-null thiago vapier 4.7.0 0 oldest_to_newest 1072211 0 aanisimov 2011-01-12 12:52:48 +0000 Version: 4.6 (using KDE 4.5.95) OS: All Subj. I have attached a patch that makes kateviewinternal disable its support for accessibility on systems that do not use it. Reproducible: Always Steps to Reproduce: Compile Qt with -no-accessibility and then compile kdelibs. Actual Results: Compilation of kdelibs fails. Expected Results: Kateviewinternal should disable its support for accessibility. 1072212 1 55906 aanisimov 2011-01-12 12:54:44 +0000 Created attachment 55906 Patch that disables kateviewinternal's support for accessibility on systems that do not use it. 1090228 2 vapier 2011-02-22 05:42:29 +0000 blah, i just wrote the same patch. wish i searched first ;). i wonder if the QT_NO_ACCESSIBILITY check should be after the local includes though ... that way the QT headers are included to provide the define. 1107266 3 aanisimov 2011-04-12 19:03:46 +0000 Hi again! Guys, you already have a patch for this bug. Why not fix it? 1113760 4 thiago 2011-05-01 23:59:08 +0000 Less-than-full Qt is not supported. Simply enable accessibility support in Qt and the problem is gone. 1113773 5 vapier 2011-05-02 00:23:40 +0000 the code already has proper #ifdef checks in places, and everything else in KDE that ive used builds fine. and someone already has posted a trivial patch. all you have to do is merge it. 1114135 6 dhaumann 2011-05-02 19:41:37 +0000 Git commit cb20a83c3806698a35a0ec7a9d24f4dfebe1736b by Dominik Haumann. Committed on 02/05/2011 at 19:50. Pushed by dhaumann into branch 'master'. check for availability of Qt accessibility module Thanks to Artem Anisimov for the patch & sorry for the delay! BUG: 262939 FIXED-IN: 4.7.0 M +2 -0 part/view/kateviewaccessible.h M +6 -0 part/view/kateviewinternal.cpp http://commits.kde.org/kate/cb20a83c3806698a35a0ec7a9d24f4dfebe1736b 55906 2011-01-12 12:54:44 +0000 2011-01-12 12:54:44 +0000 Patch that disables kateviewinternal's support for accessibility on systems that do not use it. patch text/plain 1639 aanisimov ZGlmZiAtdXIga2RlbGlicy00LjUuOTYub3JpZy9rYXRlL3ZpZXcva2F0ZXZpZXdhY2Nlc3NpYmxl Lmgga2RlbGlicy00LjUuOTUva2F0ZS92aWV3L2thdGV2aWV3YWNjZXNzaWJsZS5oCi0tLSBrZGVs aWJzLTQuNS45Ni5vcmlnL2thdGUvdmlldy9rYXRldmlld2FjY2Vzc2libGUuaAkyMDEwLTExLTEx IDE3OjQxOjI4LjAwMDAwMDAwMCArMDMwMAorKysga2RlbGlicy00LjUuOTUva2F0ZS92aWV3L2th dGV2aWV3YWNjZXNzaWJsZS5oCTIwMTEtMDEtMTEgMTE6Mzc6MTUuMjg4ODEwNTA1ICswMzAwCkBA IC0xOSw2ICsxOSw3IEBACiAKICNpZm5kZWYgX0tBVEVfVklFV19BQ0NFU1NJQkxFXwogI2RlZmlu ZSBfS0FURV9WSUVXX0FDQ0VTU0lCTEVfCisjaWZuZGVmIFFUX05PX0FDQ0VTU0lCSUxJVFkKIAog I2luY2x1ZGUgImthdGV2aWV3aW50ZXJuYWwuaCIKICNpbmNsdWRlICJrYXRldGV4dGN1cnNvci5o IgpAQCAtMzE4LDMgKzMxOSw0IEBACiB9CiAKICNlbmRpZgorI2VuZGlmCmRpZmYgLXVyIGtkZWxp YnMtNC41Ljk2Lm9yaWcva2F0ZS92aWV3L2thdGV2aWV3aW50ZXJuYWwuY3BwIGtkZWxpYnMtNC41 Ljk1L2thdGUvdmlldy9rYXRldmlld2ludGVybmFsLmNwcAotLS0ga2RlbGlicy00LjUuOTYub3Jp Zy9rYXRlL3ZpZXcva2F0ZXZpZXdpbnRlcm5hbC5jcHAJMjAxMC0xMi0wMyAxNzo1NzowNS4wMDAw MDAwMDAgKzAzMDAKKysrIGtkZWxpYnMtNC41Ljk1L2thdGUvdmlldy9rYXRldmlld2ludGVybmFs LmNwcAkyMDExLTAxLTExIDExOjM2OjUxLjk2ODgxMTQ4MSArMDMwMApAQCAtMjE2LDcgKzIxNiw5 IEBACiAgIGNvbm5lY3QoIG1fdmlldywgU0lHTkFMKCBzZWxlY3Rpb25DaGFuZ2VkKEtUZXh0RWRp dG9yOjpWaWV3KikgKSwKICAgICAgICAgICAgICB0aGlzLCBTTE9UKCB2aWV3U2VsZWN0aW9uQ2hh bmdlZCgpICkgKTsKIAorI2lmbmRlZiBRVF9OT19BQ0NFU1NJQklMSVRZCiAgIFFBY2Nlc3NpYmxl OjppbnN0YWxsRmFjdG9yeShhY2Nlc3NpYmxlSW50ZXJmYWNlRmFjdG9yeSk7CisjZW5kaWYKIAog ICAvLyB1cGRhdGUgaXMgY2FsbGVkIGluIEthdGVWaWV3LCBhZnRlciBjb25zdHJ1Y3Rpb24gYW5k IGxheW91dCBpcyBvdmVyCiAgIC8vIGJ1dCBiZWZvcmUgYW55IG90aGVyIGthdGV2aWV3aW50ZXJu YWwgY2FsbApAQCAtMjI0LDcgKzIyNiw5IEBACiAKIEthdGVWaWV3SW50ZXJuYWw6On5LYXRlVmll d0ludGVybmFsICgpCiB7CisjaWZuZGVmIFFUX05PX0FDQ0VTU0lCSUxJVFkKICAgUUFjY2Vzc2li bGU6OnJlbW92ZUZhY3RvcnkoYWNjZXNzaWJsZUludGVyZmFjZUZhY3RvcnkpOworI2VuZGlmCiAK ICAgLy8ga2lsbCBwcmVlZGl0IHJhbmdlcwogICBkZWxldGUgbV9pbVByZWVkaXRSYW5nZTsKQEAg LTY0Niw3ICs2NTAsOSBAQAogCiAgIG1fbWFkZVZpc2libGUgPSAhZm9yY2U7CiAKKyNpZm5kZWYg UVRfTk9fQUNDRVNTSUJJTElUWQogICBRQWNjZXNzaWJsZTo6dXBkYXRlQWNjZXNzaWJpbGl0eSgg dGhpcywgS2F0ZUN1cnNvckFjY2Vzc2libGU6OkNoaWxkSWQsIFFBY2Nlc3NpYmxlOjpGb2N1cyAp OworI2VuZGlmCiB9CiAKIHZvaWQgS2F0ZVZpZXdJbnRlcm5hbDo6c2xvdFJlZ2lvblZpc2liaWxp dHlDaGFuZ2VkQXQodW5zaWduZWQgaW50LGJvb2wgY2xlYXJfY2FjaGUpCg==