<div>Hello, </div><div>I defined the macro  QT_ENDL appropriately,  and made the changes locally. Should I push them?</div><br><div>Thanks, </div><div>Anjani</div><br><div class="gmail_quote_attribution">On Jun 16 2021, at 12:35 pm, Anjani Kumar <anjanik012@gmail.com> wrote:</div><blockquote><div>Hello,</div><br><div>This should be fine I guess, <a href="https://invent.kde.org/graphics/digikam/-/commit/0a1fd2c60ec38707f0e2dfeabf6ebb45a01f9382" title="https://invent.kde.org/graphics/digikam/-/commit/0a1fd2c60ec38707f0e2dfeabf6ebb45a01f9382">https://invent.kde.org/graphics/digikam/-/commit/0a1fd2c60ec38707f0e2dfeabf6ebb45a01f9382</a>. Let me know.</div><br><div>Thanks,</div><div>Anjani</div><br><div class="gmail_quote_attribution">On Jun 16 2021, at 11:23 am, Gilles Caulier <caulier.gilles@gmail.com> wrote:</div><blockquote><div><div>Hi,</div><div><br></div><div>Using digikam_globasl.h with a preprocessor definition as QT_ENDL depending of Qt version can be a solution.</div><div><br></div><div><div>Gilles</div></div><br></div><br><div class="gmail_quote"><div class="gmail_attr"><div>Le mer. 16 juin 2021 à 07:41, Maik Qualmann <<a href="mailto:metzpinguin@gmail.com" title="mailto:metzpinguin@gmail.com">metzpinguin@gmail.com</a>> a écrit :</div></div><blockquote><div>It's all correct with the Qt version check. My only thought was whether we</div><div>could find another option for Qt::endl, as there would be a lot of Qt version</div><div>checks.</div><br><div>Maik</div><br><div>Am Dienstag, 15. Juni 2021, 23:09:28 CEST schrieb Anjani Kumar:</div><div>> What should I do? Keep doing this or not?</div><div>></div><div>> Thanks</div><div>> Anjani</div><div>></div><div>> On Jun 16 2021, at 2:23 am, Anjani Kumar <<a href="mailto:anjanik012@gmail.com" title="mailto:anjanik012@gmail.com">anjanik012@gmail.com</a>> wrote:</div><div>> > Something like this</div><div>> ></div><div>> ></div><div>> > #if (QT_VERSION >= QT_VERSION_CHECK(5, 14, 0))</div><div>> ></div><div>> > QStringList tagsList = hierarchy.split(QLatin1Char('/'),</div><div>> > Qt::SkipEmptyParts);</div><div>> ></div><div>> > #else</div><div>> ></div><div>> > QStringList tagsList = hierarchy.split(QLatin1Char('/'),</div><div>> > QString::SkipEmptyParts);</div><div>> ></div><div>> > #endif</div><div>> > I haven't pushed it yet though.</div><div>> ></div><div>> > On Jun 16 2021, at 2:19 am, Anjani Kumar <<a href="mailto:anjanik012@gmail.com" title="mailto:anjanik012@gmail.com">anjanik012@gmail.com</a>> wrote:</div><div>> > > I already started this pre-processor branching. Not for Qt::endl but I</div><div>> > > have done it for QString::split() which is also present from Qt 5.14.> ></div><div>> > > On Jun 16 2021, at 2:16 am, Maik Qualmann <<a href="mailto:metzpinguin@gmail.com" title="mailto:metzpinguin@gmail.com">metzpinguin@gmail.com</a>> wrote:</div><div>> > > > But we already have a problem with Qt::endl, this only exists with</div><div>> > > > Qt-5.14. We should find another way of doing this than with</div><div>> > > > pre-processeur branching. Maybe define a macro.</div><div>> > > ></div><div>> > > > Maik</div><div>> > > ></div><div>> > > > Am Dienstag, 15. Juni 2021, 13:55:53 CEST schrieb Gilles Caulier:</div><div>> > > > > Hi,</div><div>> > > > ></div><div>> > > > > No. We need to be compatible at least with Qt 5.12 LTS.</div><div>> > > > > Please make pre-processeur branching in source code by checking Qt</div><div>> > > > > version,</div><div>> > > > > as it already do in other place in digiKam.</div><div>> > > > ></div><div>> > > > > exemple :</div><div>> > > > > <a href="https://invent.kde.org/graphics/digikam/-/blob/master/core/app/date/" title="https://invent.kde.org/graphics/digikam/-/blob/master/core/app/date/">https://invent.kde.org/graphics/digikam/-/blob/master/core/app/date/</a></div><div>> > > > > ddatepic ker.cpp#L547</div><div>> > > > ></div><div>> > > > > Best</div><div>> > > > ></div><div>> > > > > Gilles Caulier</div><div>> > > > ></div><div>> > > > > Le mar. 15 juin 2021 à 13:52, Anjani Kumar <<a href="mailto:anjanik012@gmail.com" title="mailto:anjanik012@gmail.com">anjanik012@gmail.com</a>> a</div><div>écrit :</div><div>> > > > > > The recent changes to</div><div>> > > > > > <a href="https://invent.kde.org/graphics/digikam/-/tree/gsoc21-qt6-port" title="https://invent.kde.org/graphics/digikam/-/tree/gsoc21-qt6-port">https://invent.kde.org/graphics/digikam/-/tree/gsoc21-qt6-port</a></div><div>> > > > > > makes it</div><div>> > > > > > compile only on newer Qt versions. So I it should be best Bump Qt</div><div>> > > > > > version</div><div>> > > > > > to 5.15.2. Should I make the change?</div><div>> > > > > ></div><div>> > > > > > Thanks,</div><div>> > > > > > Anjani</div></blockquote></div></blockquote></blockquote>