<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">2013/11/11 Thomas Petazzoni <span dir="ltr"><<a href="mailto:thomas.petazzoni@free-electrons.com" target="_blank">thomas.petazzoni@free-electrons.com</a>></span><br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Dear Samuel Martin,<br>
<div class="im"><br>
On Mon, 11 Nov 2013 22:00:10 +0100, Samuel Martin wrote:<br>
>  menuconfig BR2_PACKAGE_OPENCV<br>
>       bool "opencv"<br>
>       select BR2_PACKAGE_ZLIB<br>
> +     select BR2_PACKAGE_EIGEN<br>
<br>
</div>Any reason to make it mandatory?<br></blockquote><div><br></div><div>After checking with some opencv developers, eigen support does not bring that much</div><div>(neither in term of features, nor perfs).</div><div>

<br></div><div>So I've rejected it.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I'm sorry but I'm going to be a bit angry again. I've asked you many,<br>
many, many times to write more descriptive commit logs. And once again,<br>
several of your patches have just a one line title, with no explanation<br>
whatsoever about *why* you're doing the change.<br>
<br>
It would be really nice if you could review all your patches before you<br>
post them, and ask yourself the question: is the reason why I'm doing<br>
this going to be obvious for the reviewer? If the answer is even on the<br>
slightly unsure side, then it means you need to expand with a more<br>
detailed commit log, or more comments in the code.<br>
<br>
Thanks,<br>
<br>
Thomas<br>
<span class="HOEnZb"><font color="#888888">--<br>
Thomas Petazzoni, CTO, Free Electrons<br>
Embedded Linux, Kernel and Android engineering<br>
<a href="http://free-electrons.com" target="_blank">http://free-electrons.com</a><br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br>Samuel
</div></div>