<html><head><meta http-equiv="content-type" content="text/html; charset=utf-8"></head><body dir="auto"><div dir="ltr">What does QED mean? And why do you insist on making even a short sentence hard to decipher?</div><div dir="ltr"><br><blockquote type="cite">On 21 Oct 2019, at 16:05, Rémi Denis-Courmont <remi@remlab.net> wrote:<br><br></blockquote></div><blockquote type="cite"><div dir="ltr"><title></title>So I'm the only one using it, QED.<br><br><div class="gmail_quote">Le 21 octobre 2019 16:35:55 GMT+03:00, Thomas Guillem <thomas@gllm.fr> a écrit :<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div><br></div><div>On Mon, Oct 21, 2019, at 15:33, Rémi Denis-Courmont wrote:<br></div><blockquote type="cite" id="qt"><div>Hi,<br></div><div><br></div><div>If you use asan, how come you missed the packetizer test leaking for several weeks? It's got nothing to do with my hardware.<br></div></blockquote><div><br></div><div>I don't use Asan for make check since some tests are failing.<br></div><div><br></div><div>Yes, that's not a good solution. I should fix them instead, but last time I checked, last failing tests were not VLC's fault.<br></div><div><br></div><blockquote type="cite" id="qt"><div><br></div><div class="qt-gmail_quote"><div>Le 21 octobre 2019 16:06:07 GMT+03:00, Thomas Guillem <thomas@gllm.fr> a écrit :<br></div><blockquote style="margin-top:0pt;margin-right:0pt;margin-bottom:0pt;margin-left:0.8ex;border-left-color:rgb(204, 204, 204);border-left-style:solid;border-left-width:1px;padding-left:1ex;" class="qt-gmail_quote"><div><br></div><div>On Mon, Oct 21, 2019, at 14:04, Rémi Denis-Courmont wrote:<br></div><blockquote type="cite" id="qt-qt"><div>Hi,<br></div><div><br></div><div>With all due respect, this fixation on Gitlab CI seems delusional to me. It took several weeks for anybody to report an obvious and universal memory leak in the unit tests. Because nobody uses the leak sanitizer on regular basis except me.<br></div><div><br></div><div>As for the video problem, it only occurs in smoke testing not in unit testing, so any CI is entirely helpless there.<br></div><div><br></div><div>It's rather ironic that I was the one to object against adding the sanitizers to the configure script, and I'm semingly the only one to use them systematically in the end.<br></div></blockquote><div><br></div><div>A lot of devs are using asan. Personally, I always use it when I work.<br></div><div><br></div><div>The bug you caught was not reproducible on my main workstation because I don't have a nvidia GPU and it happens with a specific user interaction. When I tested VDPAU an a secondary desktop, I forgot to enable ASAN on this specific case (because I didn't use my usual helper scripts).<br></div><div><br></div><div>Human errors like that will continue to happen. I think we can live with it. They are generally quickly fixed by the author.<br></div><div><br></div><div>I already had to fix your errors, like you already had to fix mine. In that case, either we fix it, assign a ticket or signal it on the mailing list. This process seems OK to me.<br></div><div><br></div><blockquote type="cite" id="qt-qt"><div><br></div><div>Besides premerge CI will fail in so many other ways that it's not even funny. It's great for an admin who has the power to fix the CI system or to overrule it. It sucks big time for everyone else who gets stuck whenever:<br></div><div>- one platform is down,<br></div><div>- contrib or sync fails due to network issue,<br></div><div>- build fails due to transient out-of-ressource situation,<br></div><div>- nondeterministic test case breaks pseudorandomly,<br></div><div>- system update introduces unrelated regressions (especially likely with leaksan).<br></div><div>And if it's gating, then it'll be miserable for people working in different continent or hobbyists, who'll have nobody to get live support from.<br></div><div><br></div><div><br></div><div>No, the only way to fix this is that  developers use sanitizers, which was the whole point of adding them to configure.<br></div><div><br></div><div class="qt-qt-gmail_quote"><div>Le 20 octobre 2019 21:01:25 GMT+03:00, Jean-Baptiste Kempf <jb@videolan.org> a écrit :<br></div><blockquote style="margin-top:0pt;margin-right:0pt;margin-bottom:0pt;margin-left:0.8ex;border-left-color:rgb(204, 204, 204);border-left-style:solid;border-left-width:1px;padding-left:1ex;" class="qt-qt-gmail_quote"><pre class="qt-qt-k9mail"><div>Yo,<br></div><div><br></div><div>On Sun, Oct 20, 2019, at 15:33, Rémi Denis-Courmont wrote:<br></div><blockquote style="margin-top:0pt;margin-right:0pt;margin-bottom:1ex;margin-left:0.8ex;border-left-color:rgb(114, 159, 207);border-left-style:solid;border-left-width:1px;padding-left:1ex;" class="qt-qt-gmail_quote"><div>The points are that some people ostensibly don't run the test suite, and/or <br></div><div>smoke tests and/or enable the ADDR and UB sanitisers, causing easily avoidable <br></div><div>regressions in master.<br></div></blockquote><div><br></div><div>Agreed.<br></div><div><br></div><div>Breaking the build or breaking tests should not happen anymore. This is not normal.<br></div><div><br></div><div>We will then move to gitlab to get the CI to run pre-merge, as soon as possible.<br></div><div>CI should catch all of those.<br></div></pre></blockquote></div><div><br></div><div>-- <br></div><div>Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté. <br></div><div>_______________________________________________<br></div><div>vlc-devel mailing list<br></div><div>To unsubscribe or modify your subscription options:<br></div><div>https://mailman.videolan.org/listinfo/vlc-devel<br></div></blockquote><div><br></div></blockquote></div><div><br></div><div>-- <br></div><div>Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté. <br></div><div>_______________________________________________<br></div><div>vlc-devel mailing list<br></div><div>To unsubscribe or modify your subscription options:<br></div><div>https://mailman.videolan.org/listinfo/vlc-devel<br></div></blockquote><div><br></div></blockquote></div><br>-- <br>Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.<span>_______________________________________________</span><br><span>vlc-devel mailing list</span><br><span>To unsubscribe or modify your subscription options:</span><br><span>https://mailman.videolan.org/listinfo/vlc-devel</span></div></blockquote></body></html>