[vlc-devel] breaking build

[Rémi Denis-Courmont]

Hi,

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.

As for the video problem, it only occurs in smoke testing not in unit testing, so any CI is entirely helpless there.

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.

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:
- one platform is down,
- contrib or sync fails due to network issue,
- build fails due to transient out-of-ressource situation,
- nondeterministic test case breaks pseudorandomly,
- system update introduces unrelated regressions (especially likely with leaksan).
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.


No, the only way to fix this is that  developers use sanitizers, which was the whole point of adding them to configure.

Le 20 octobre 2019 21:01:25 GMT+03:00, Jean-Baptiste Kempf <jb at videolan.org> a écrit :
>Yo,
>
>On Sun, Oct 20, 2019, at 15:33, Rémi Denis-Courmont wrote:
>> The points are that some people ostensibly don't run the test suite,
>and/or 
>> smoke tests and/or enable the ADDR and UB sanitisers, causing easily
>avoidable 
>> regressions in master.
>
>Agreed.
>
>Breaking the build or breaking tests should not happen anymore. This is
>not normal.
>
>We will then move to gitlab to get the CI to run pre-merge, as soon as
>possible.
>CI should catch all of those.
>
>-- 
>Jean-Baptiste Kempf -  President
>+33 672 704 734
>_______________________________________________
>vlc-devel mailing list
>To unsubscribe or modify your subscription options:
>https://mailman.videolan.org/listinfo/vlc-devel

-- 
Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20191021/b648fbd7/attachment.html>