
<!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>You asked why the ambience is bad and I answered. I don't want to hear excuses. If you don't like the answer that does not change.<br><br>If you think the GSoC ambience is more worthy than the vlc-devel ambience, that is your subjective opinion. That does not change the situation that the ambience is shit because of demotivate from UI brokeness (like in 0.9) and from you picking on people who don't agree with your mandatory review hypocritical nonsense.<br><br><div class="gmail_quote">Le 24 septembre 2019 09:28:45 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>On Tue, Sep 24, 2019, at 08:07, Rémi Denis-Courmont wrote:<br></div><blockquote id="qt" type="cite"><div>Hi,<br></div><div><br></div><div>No, people do not want mandatory review. That's the suggestion of mandatory review that brought this shitty ambience in the first place, and the UI brokeness. <br></div></blockquote><div><br></div><div>We had a great GSOC. Students were very happy to work on the QT part.<br></div><div><br></div><blockquote id="qt" type="cite"><div><br></div><div>That and you picking on and to a lesser extent on François.<br></div></blockquote><div><br></div><div>I speak to Francois regularly, we don't have any problems at all, we are not picking each other. We complete and help each other on different part of VLC.<br></div><div><br></div><blockquote id="qt" type="cite"><div>Besides, none of the committers, except me, have abode by the week-long wait implied by mandatory review. Also nobody addressed the problem of lack of reviewers in some areas (or time).<br></div></blockquote><div><br></div><div>It's very easy to maintain lot of branches with git. The problem with the ML is that we can forget easily to push a past branch. This will be solved with gitlab.<br></div><div><br></div><blockquote id="qt" type="cite"><div><br></div><div>And *you* do not follow mandatory review at all. It was obvious that wrecking the main UI would demotivate existing devs, rebute potential new ones and take months (and counting) to fix - exactly like in 0.9 development. Two people objected (not counting packagers even). <br></div></blockquote><div><br></div><div><div>I don't remember people complaining before I pushed. I remember asking that I needed to push this Qt work and that it would break a lot of things (that I listed by creating track tickets). Francois complained after, I don't know any packagers that complained to me.<br></div><div><br></div><div>And it was not my decision to push that work, it comes from VideoLan board. I was just the soldier here. I told them that I didn't like breaking the master, it was dully noted. Having said that, I don't regret it anymore since it allowed to move forward and work with external contributors (like GSOC students) on that part.<br></div><div><br></div></div><blockquote id="qt" type="cite"><div>And yet you merged it. That's not ho mandatory review works. that's how you ruin the working ambience.<br></div></blockquote><div><br></div><div>I know that the open source world can be really tough, specially with junior developers. That is why I always try to be nice with everyone, I don't hesitate to respond to private query and help them. I don't think I'm ruining the ambience at all.<br></div><div><br></div><div>And it works, few external contributors told me or other developers that it was very nice working with me (and I don't talk about clients).<br></div><div><br></div><div>Thomas<br></div><div><br></div><blockquote id="qt" type="cite"><div><br></div><div class="qt-gmail_quote"><div>Le 24 septembre 2019 01:32:36 GMT+03:00, Thomas Guillem <thomas@gllm.fr> 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><br></div><div><br></div><div>On Mon, Sep 23, 2019, at 22:57, 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>Le maanantaina 23. syyskuuta 2019, 23.26.20 EEST Thomas Guillem a écrit :<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(173, 127, 168);border-left-style:solid;border-left-width:1px;padding-left:1ex;"><div>On Mon, Sep 23, 2019, at 18:20, 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(138, 226, 52);border-left-style:solid;border-left-width:1px;padding-left:1ex;"><div>So me in public and everybody else privately. Same difference. That's<br></div><div>still<br></div></blockquote><div>I remember telling François and steve on this ML.<br></div></blockquote><div>All I see is asking François if he can send patches for review in the future. <br></div><div>Noting from Steve at all.<br></div></blockquote><div><br></div><div>I speak to Steve regularly, and we do video conf so I might have tel him this way. To be honest I don't remember. <br></div><div><br></div><div>> <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>How convenient to remember something that did not happen (sources: vlc-devel <br></div><div>archives).<br></div><div><br></div><div>Well, if that's it, then I remember that the committee did not approve any <br></div><div>review mandate. So I am going to flatly ignore all your comments on mandatory <br></div><div>reviews until the commitee or the GA passes a resolution on the topic<br></div></blockquote><div><br></div><div>Why so much tension ? Could we work together again ? It worked quite well in the past few months, every one were respecting each others and each others reviews. It can't work if you ignore me or if I ignore you since we are both main VLC developers... <br></div><div><br></div><div>I know for a fact that most people want mandatory review. I think it will be voted during the next VDD. In the meantime, I would really love that all core commits go through the ML. I really apprecited your extra effort recently when you went throught the ML. And I always try to send good reviews. <br></div><div><br></div><div><br></div><div>> <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>-- <br></div><div>Реми Дёни-Курмон<br></div><div><a href="http://www.remlab.net/">http://www.remlab.net/</a><hr>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><hr>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></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>