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