<!doctype html><html><head><title></title><style type="text/css">p.MsoNormal,p.MsoNoSpacing{margin:0}
p.MsoNormal,p.MsoNoSpacing{margin:0}</style></head><body>Hi,<br><br>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><br><div class="gmail_quote">Le 21 octobre 2019 16:06:07 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 14:04, Rémi Denis-Courmont wrote:<br></div><blockquote id="qt" type="cite"><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 id="qt" type="cite"><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-gmail_quote"><div>Le 20 octobre 2019 21:01:25 GMT+03:00, Jean-Baptiste Kempf <jb@videolan.org> a écrit :<br></div><blockquote class="qt-gmail_quote" 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;"><pre class="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 class="qt-gmail_quote" 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;"><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><br>-- <br>Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.</body></html>