<div dir="ltr"><div>Hi Belgacem,</div><div><br></div><div>It would be very good to have tests indeed !</div><div>If you have them ready, it should be straightforward to enable these on the qt 5.14 build.</div><div><br></div><div>Best regards</div><div>Matthias<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Aug 27, 2020 at 3:13 PM BELGACEM NEDJIMA <<a href="mailto:gb_nedjima@esi.dz">gb_nedjima@esi.dz</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="auto"><div>Hi Matthias,<div dir="auto">Yes, I wanted to run the CI tests on a version >= Qt5.10 because if I disable shadow rendering for <=Qt5.9, it will bypass all the tests and may add some bugs into the codebase.</div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Aug 27, 2020, 1:45 PM Matthias Kuhn <<a href="mailto:matthias@opengis.ch" target="_blank">matthias@opengis.ch</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>Hi Peter,</div><div><br></div><div>I don't see a problem to conditionally disable shadows for Qt <= 5.10, we also have other conditions depending on the Qt version in other parts of the codebase.<br></div><div>Is there a hidden question how to go about testing this in CI? If yes, I'd say the best approach would indeed be to enable tests for Qt 5.14.</div><div><br></div><div>Best regards<br></div><div>Matthias<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Aug 27, 2020 at 1:58 PM Peter Petrik <<a href="mailto:peter.petrik@lutraconsulting.co.uk" rel="noreferrer" target="_blank">peter.petrik@lutraconsulting.co.uk</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr">Hi, <div><br></div><div>CI on travis runs with qt 5.9 and for <a href="https://github.com/qgis/QGIS/pull/38448" rel="noreferrer" target="_blank">https://github.com/qgis/QGIS/pull/38448</a> we need to have qt 5.10+. In 5.9 the 3d shadow rendering does not work correctly and since Windows, Mac and latest Linux distributions run on newer qt's I do not see a reason to spend considerable time to fix the 3d shadows for qt 5.9. I was thinking to disable this feature for builds with qt 5.9.I see the qt5.14 build in the repository, but that one is without tests. </div><div><br></div><div>I am wondering how to best approach this? </div><div><br></div><div>Thanks,</div><div>Peter</div><div></div></div></div>
_______________________________________________<br>
QGIS-Developer mailing list<br>
<a href="mailto:QGIS-Developer@lists.osgeo.org" rel="noreferrer" target="_blank">QGIS-Developer@lists.osgeo.org</a><br>
List info: <a href="https://lists.osgeo.org/mailman/listinfo/qgis-developer" rel="noreferrer noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/qgis-developer</a><br>
Unsubscribe: <a href="https://lists.osgeo.org/mailman/listinfo/qgis-developer" rel="noreferrer noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/qgis-developer</a></blockquote></div>
_______________________________________________<br>
QGIS-Developer mailing list<br>
<a href="mailto:QGIS-Developer@lists.osgeo.org" rel="noreferrer" target="_blank">QGIS-Developer@lists.osgeo.org</a><br>
List info: <a href="https://lists.osgeo.org/mailman/listinfo/qgis-developer" rel="noreferrer noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/qgis-developer</a><br>
Unsubscribe: <a href="https://lists.osgeo.org/mailman/listinfo/qgis-developer" rel="noreferrer noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/qgis-developer</a></blockquote></div></div></div>
</blockquote></div>