
<!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><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></body></html>