<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Le dim. 12 mai 2019 à 07:32, Nyall Dawson <<a href="mailto:nyall.dawson@gmail.com">nyall.dawson@gmail.com</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Fri, 10 May 2019 at 18:31, René-Luc Dhont <<a href="mailto:rldhont@gmail.com" target="_blank" rel="noreferrer">rldhont@gmail.com</a>> wrote:<br>
><br>
> Hi devs,<br>
><br>
> I am still waiting for a review about fixing 2 issues in printing for<br>
> QGIS and QGIS Server 3.4 LTR. <a href="https://github.com/qgis/QGIS/pull/9692" rel="noreferrer noreferrer" target="_blank">https://github.com/qgis/QGIS/pull/9692</a><br>
><br>
> This PR wil fix an issue <a href="https://issues.qgis.org/issues/19500" rel="noreferrer noreferrer" target="_blank">https://issues.qgis.org/issues/19500</a> `Layout<br>
> export - raster divided into tiles, edges evident in pdf/svg`. This<br>
> issue has been identifed in version 3.2 and has been fixed in 3.6.<br>
> This PR will also fix a regression in QGIS Server `Printing selection in<br>
> PDF and SVG`. The selection is already printed in image output.<br>
><br>
> The next LTR will be released in 10 mounths. QGIS Server 3.6, which<br>
> already have the fix, is not available on Debian Stretch because it<br>
> requires Qt 5.9 and Debian Buster will be freezed in 8 mounths. So the<br>
> QGIS server regression will not be available since the end of 2019.<br>
><br>
> I will appreciate a decision about this fix. Do you think QGIS Server<br>
> users have to wait for the next LTR or Debian Buster release ?<br>
<br>
I'm still uncomfortable with the potential risk this has of breaking<br>
layout export in 3.4 for desktop users. So I'm not going to approve<br>
the backport, but if another dev is willing to take on this risk and<br>
approve the backport then I won't stand in the way.<br>
<br>
Nyall<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">I agree with Nyall here unfortunately. </div><div dir="auto"><br></div><div dir="auto">Backporting this could be acceptable at the condition of having a wide integration test coverage and extensive real testing of the fixed branches.</div><div dir="auto"><br></div><div dir="auto">I'd be pleased to test things here but use cases are so numerous that I fail to see how to do this efficiently. </div><div dir="auto">I hope we'll one day be able to run those smoke tests as discussed at La Coruna. </div><div dir="auto">Best regards</div><div dir="auto"><br></div><div dir="auto">Régis</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
><br>
> Reagrds,<br>
> René-Luc<br>
><br>
> Le 19/04/2019 à 09:34, René-Luc Dhont a écrit :<br>
> > Hi devs,<br>
> ><br>
> > I have cherry-picked the changes<br>
> ><br>
> > I have tested it without regressions or issues.<br>
> ><br>
> > Now I need some review to merge and fix 2 issues.<br>
> > <a href="https://github.com/qgis/QGIS/pull/9692" rel="noreferrer noreferrer" target="_blank">https://github.com/qgis/QGIS/pull/9692</a><br>
> ><br>
> > Regards,<br>
> > René-Luc<br>
> ><br>
> > Le 17/04/2019 à 14:18, René-Luc Dhont a écrit :<br>
> >> Hi devs,<br>
> >><br>
> >> Thanks Paolo for your agreement.<br>
> >><br>
> >> I'd like to fix the QGIS Sever regression before friday and the<br>
> >> version 3.4.7 released.<br>
> >><br>
> >> So I'd like to have other point of view  about backporting :<br>
> >> [needs-docs][layouts] Add checkbox to disable raster tiling for<br>
> >> PDF/SVG exports <a href="https://github.com/qgis/QGIS/pull/9016" rel="noreferrer noreferrer" target="_blank">https://github.com/qgis/QGIS/pull/9016</a><br>
> >> and finalizing Fix regression server print selection pdf release 3.4<br>
> >> <a href="https://github.com/qgis/QGIS/pull/9692" rel="noreferrer noreferrer" target="_blank">https://github.com/qgis/QGIS/pull/9692</a><br>
> >><br>
> >> Thanks.<br>
> >> René-Luc<br>
> >><br>
> >> Le 12/04/2019 à 12:06, Paolo Cavallini a écrit :<br>
> >>> Hi René,<br>
> >>><br>
> >>> On 12/04/19 12:05, René-Luc Dhont wrote:<br>
> >>><br>
> >>>> I would like to know if I can backport all the commits made by<br>
> >>>> Nyall to<br>
> >>>> fix issue 19500 even if it updates the User Interface ?<br>
> >>> +1 from my side, but obviously Nyall has the final word on this.<br>
> >>> Thanks.<br>
> >>><br>
> >><br>
> ><br>
><br>
> _______________________________________________<br>
> QGIS-Developer mailing list<br>
> <a href="mailto:QGIS-Developer@lists.osgeo.org" target="_blank" rel="noreferrer">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><br>
_______________________________________________<br>
QGIS-Developer mailing list<br>
<a href="mailto:QGIS-Developer@lists.osgeo.org" target="_blank" rel="noreferrer">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>