[Konsole-devel] Fwd: [konsole] src: Accessible interface for TerminalDisplay

[Frederik Gladhorn]

On 04/10/2012 04:37 AM, Kurt Hindenburg wrote:
>
>
> On Mon, Apr 9, 2012 at 5:38 PM, Jekyll Wu <adaptee at gmail.com
> <mailto:adaptee at gmail.com>> wrote:
>
>     -----BEGIN PGP SIGNED MESSAGE-----
>     Hash: SHA1
>
>     Hi:
>
>     I just realized that this commit increases the requirement for Qt
>     version to 4.8+, since it uses some Enums introduced in Qt 4.8.0, like
>     QAccessible::TextUpdated and QAccessible::TextInserted.
>
>     Now Konsole fails to compile on Debian Sid, which ships Qt-4.7.4
>     as its
>     newest version.
>
>     So questions:
>
>     1. Is this requirement too high? IIRC, even kdelibs hasn't increased
>     the requirement to Qt 4.8+ now.
>
>     2. Should we add explicit requirement for Qt 4.8+ in CMakeLists.txt ?
>
> If kdelibs doesn't require Qt 4.8, we can't.  Let's see what the
> official word is.  I haven't looked at the code yet, how hard would it
> be to #define it out?
>
> Kurt
>  
>  
No problem, I'll fix it. It's easy to ifdef out.
Sorry for the mess.

Frederik