<!DOCTYPE html><html><head><title></title><style type="text/css">p.MsoNormal,p.MsoNoSpacing{margin:0}</style></head><body><div>Hello,<br></div><div><br></div><div>I asked Sergio to remove the old module mainly for maintenance purpose (so that we won't have to debug 2 modules). I didn't think about the compatibility issue.<br></div><div><br></div><div>This will cause an issue mainly on Linux if the user or the distribution doesn't have librist.<br></div><div><br></div><div>I'm OK for both choices, but I still prefer removing the old one. What do you think?<br></div><div><br></div><div>On Tue, Mar 30, 2021, at 16:34, Sergio M. Ammirata, Ph.D. wrote:<br></div><blockquote type="cite" id="qt" style="text-align:left;direction:ltr;"><div><br></div><div>Confirmed, you can safely ignore/skip the meson upgrade patch. <br></div><div><br></div><div>Here is the pipeline test:<br></div><div><a href="https://code.videolan.org/sammirata/vlc/-/pipelines/83119">https://code.videolan.org/sammirata/vlc/-/pipelines/83119</a><br></div><div><br></div><div>Regards,<br></div><div><br></div><div>Sergio<br></div><div><br></div><div>On Tue, 2021-03-30 at 09:03 -0400, Sergio M. Ammirata, Ph.D. wrote:<br></div><blockquote type="cite" style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0.8ex;border-left-color:rgb(114, 159, 207);border-left-style:solid;border-left-width:2px;padding-left:1ex;"><div><br></div><div>I can remove the requirement for the meson upgrade. It is only needed when the internal mbedtls subproject is used.<br></div><div><br></div><div>Sergio<br></div><div><br></div><div>On Tue, 2021-03-30 at 15:52 +0300, Rémi Denis-Courmont wrote:<br></div><blockquote type="cite" style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0.8ex;border-left-color:rgb(114, 159, 207);border-left-style:solid;border-left-width:2px;padding-left:1ex;"><pre>Le tiistaina 30. maaliskuuta 2021, 14.54.19 EEST Konstantin Pavlov a écrit :<br></pre><pre><br></pre><pre>On Tue, Mar 30, 2021 at 01:04:49PM +0300, Rémi Denis-Courmont wrote:<br></pre><pre><br></pre><pre>Le maanantaina 29. maaliskuuta 2021, 19.54.22 EEST Sergio M. Ammirata,<br></pre><pre>Ph.D. a> <br></pre><pre>écrit :<br></pre><pre><br></pre><pre>Hello Thomas,<br></pre><pre>I have fixed every item using your and Alexandre's<br></pre><pre>suggestions in the comments of the commits. Accordingly, I<br></pre><pre>have prepared and attached the final 4 new patches.<br></pre><pre>This time, I used this forked repo:<br></pre><pre><a href="https://code.videolan.org/rist/vlc/-/tree/librist">https://code.videolan.org/rist/vlc/-/tree/librist</a><br></pre><pre><br></pre><pre>I don't think we can add *new* requirements for *existing* features within<br></pre><pre>a *stable* release branch.<br></pre><pre><br></pre><pre>-1 on upgrading Meson<br></pre><pre><br></pre><pre>We do it regularly for meson and other tools to fix various issues, so not<br></pre><pre>sure why it's a problem now.<br></pre><pre><br></pre><pre>It's fine in the unstable branch(es). But people should be able to keep a <br></pre><pre>stable build environment for the stable branch, and some actually do that.<br></pre><pre><br></pre><pre>We've had no shortage of regressions and build issues due to changing the <br></pre><pre>stable branch build environments.<br></pre><pre><br></pre><pre><br></pre><pre><br></pre><pre>-1 on requiring a new library to support existing RIST use cases<br></pre><pre><br></pre><pre>Can we keep older modules with a lower priority instead of removing them?<br></pre><pre>AFAICT it wouldnt be different to e.g. dav1d inclusion in around 3.0.5<br></pre><pre>then.<br></pre><pre><br></pre><pre>I guess so.<br></pre><pre><br></pre><pre><br></pre></blockquote><pre>_______________________________________________<br></pre><pre>vlc-devel mailing list<br></pre><pre>To unsubscribe or modify your subscription options:<br></pre><pre><a href="https://mailman.videolan.org/listinfo/vlc-devel">https://mailman.videolan.org/listinfo/vlc-devel</a><br></pre></blockquote><div>_______________________________________________<br></div><div>vlc-devel mailing list<br></div><div>To unsubscribe or modify your subscription options:<br></div><div><a href="https://mailman.videolan.org/listinfo/vlc-devel">https://mailman.videolan.org/listinfo/vlc-devel</a><br></div></blockquote><div><br></div></body></html>