Review Request 128739: Fix build with Qt < 5.6
Burkhard Lück
lueck at hube-lueck.de
Tue Aug 23 11:48:03 UTC 2016
> On Aug. 23, 2016, 11:27 vorm., Heiko Becker wrote:
> > ?[12:11] ?<?sandsmark?>? commit it if someone can test with the appropriate Qt versions
> > ...
> > ?[12:13] ?<?veqz?>? hmm, I have the right Qt versions. I can download the pat-
> > ...
> > ?[12:31] ?<?veqz?>? heirecka: I ended up applying to patch manually..., and it compiles fine now :)
Your patch breaks build with Qt 5.6.1 for me
I need this patch on to of yours to get it build:
index 4607966..abc50af 100644
--- a/src/core/guesslanguage.cpp
+++ b/src/core/guesslanguage.cpp
@@ -489,7 +489,7 @@ GuessLanguagePrivate::GuessLanguagePrivate()
case QChar::Script_WarangCiti:
names = getNames(QLocale::VarangKshitiScript);
break;
-#if QT_VERSION >= QT_VERSION_CHECK(5, 6, 0)
+#if QT_VERSION > QT_VERSION_CHECK(5, 6, 1)
case QChar::Script_Ahom:
names = getNames(QLocale::AhomScript);
break;
- Burkhard
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128739/#review98566
-----------------------------------------------------------
On Aug. 23, 2016, 11:28 vorm., Heiko Becker wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128739/
> -----------------------------------------------------------
>
> (Updated Aug. 23, 2016, 11:28 vorm.)
>
>
> Review request for KDE Frameworks and Martin Tobias Holmedahl Sandsmark.
>
>
> Repository: sonnet
>
>
> Description
> -------
>
> Also correct the other #ifdef because like QChar::Script_SignWriting
> 0e1f3aab11a4310595d8651c02ce5f9d60c93252 in qtbase.git introduced the
> other enums as well. That commit was first released with Qt 5.6
>
> BUG: 366789
> CCMAIL: martin.sandsmark at kde.org
>
>
> Diffs
> -----
>
> src/core/guesslanguage.cpp 5c5239fbe99c9a3ea95bd07dc4d0ec6582897379
>
> Diff: https://git.reviewboard.kde.org/r/128739/diff/
>
>
> Testing
> -------
>
> Haven't an old Qt around, tested building with 5.7. though.
>
>
> Thanks,
>
> Heiko Becker
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160823/0d6ebdab/attachment.html>
More information about the Kde-frameworks-devel
mailing list