[Buildroot] [PATCH 4/7] opencv: add eigen dependency
s.martin49 at gmail.com
Fri Nov 15 20:22:27 UTC 2013
2013/11/11 Thomas Petazzoni <thomas.petazzoni at free-electrons.com>
> Dear Samuel Martin,
> On Mon, 11 Nov 2013 22:00:10 +0100, Samuel Martin wrote:
> > menuconfig BR2_PACKAGE_OPENCV
> > bool "opencv"
> > select BR2_PACKAGE_ZLIB
> > + select BR2_PACKAGE_EIGEN
> Any reason to make it mandatory?
After checking with some opencv developers, eigen support does not bring
(neither in term of features, nor perfs).
So I've rejected it.
> I'm sorry but I'm going to be a bit angry again. I've asked you many,
> many, many times to write more descriptive commit logs. And once again,
> several of your patches have just a one line title, with no explanation
> whatsoever about *why* you're doing the change.
> It would be really nice if you could review all your patches before you
> post them, and ask yourself the question: is the reason why I'm doing
> this going to be obvious for the reviewer? If the answer is even on the
> slightly unsure side, then it means you need to expand with a more
> detailed commit log, or more comments in the code.
> Thomas Petazzoni, CTO, Free Electrons
> Embedded Linux, Kernel and Android engineering
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the buildroot