rng2cpp.cpp build fail with qt5.5
Treeve Jelbert
treeve at scarlet.be
Fri May 29 08:35:48 BST 2015
On Fri, 29 May 2015 01:03:41 +0200, Friedrich W. H. Kossebau wrote:
> Am Donnerstag, 28. Mai 2015, 22:44:04 schrieb Friedrich W. H.
> Kossebau:
>> Okay. Now that I understand completely why it is needed, no
>> objections from
>> my side (in version which keeps the #if QT_VERSION < 0x040800
>> branches):
>> #include <QtGlobal>
>> #include <QSharedPointer>
>> #include <QStringList>
>> +#include <QSet>
>> +#include <QVector>
>>
>> #if QT_VERSION < 0x040800
>> # define assert(cond, what) (Q_ASSERT_X(cond, "",
>> qPrintable(QString(what))))
>> #else
>> -# define assert(cond, what) (Q_ASSERT_X(cond, "",
>> qPrintable(what)))
>> +# define assert(cond, what) Q_ASSERT_X(cond, "",
>> qPrintable(what))
>> #endif
>>
>> Treeve, do you have push rights for the Calligra repo, or shall I
>> commit for
>> you?
>
> Simply commited the fix meanwhile to the "frameworks" branch.
> Hopefully you
> can confirm it builds also for you now without your local patch.
>
frameworks builds for me still, with that local patch.
(I have some more patches related to qt5.5 + gcc-5.1, which I will send
separately.
> Looking forward for more patches from you, and thanks again for this
> one :)
>
> Future patches ideally you would send in via
> https://git.reviewboard.kde.org/
> , it makes handling and commenting the patches easier for us.
>
I do not have a reviewboard account.
Regards, Treeve
> Cheers
> Friedrich
> _______________________________________________
> calligra-devel mailing list
> calligra-devel at kde.org
> https://mail.kde.org/mailman/listinfo/calligra-devel
More information about the calligra-devel
mailing list