[Buildroot] [PATCH v2] opencv: Replace absolute path of static library with -l$library_name

Samuel Martin s.martin49 at gmail.com
Mon Mar 9 09:44:25 UTC 2015


Hi Vicente, Thomas, all,

On Sun, Mar 8, 2015 at 4:14 PM, Thomas Petazzoni
<thomas.petazzoni at free-electrons.com> wrote:
> Dear Vicente Olivert Riera,
>
> On Wed, 1 Oct 2014 13:44:10 +0100, Vicente Olivert Riera wrote:
>> Some opencv modules are forced to build statically (for instance the
>> "ts" module) because they have a line like this one in their
>> CMakeLists.txt file:
>>
>>   set(OPENCV_MODULE_TYPE STATIC)
>>
>> That makes a static library get installed and also listed in the "Libs:"
>> line of the pkgconfig file (opencv.pc) along with all the shared
>> libraries. That can cause link failures in other packages which link to
>> opencv, for instance the gst1-plugins-bad package. Here is an example of
>> the link failure:
>>
>>   CXXLD    libgstopencv.la
>>
>> *** Warning: Linking the shared library libgstopencv.la against the
>> *** static library /usr/lib/libopencv_ts.a is not portable!
>> mips-linux-gnu-g++: error: /usr/lib/libopencv_ts.a: No such file or
>> directory
>> make[4]: *** [libgstopencv.la] Error 1
>>
>> The solution is to not list the static library in the "Libs:" line of
>> the pkgconfig file and add '-lopencv_ts' instead.
>>
>> This issue has been reported upstream:
>>   http://code.opencv.org/issues/3931
>>
>> Fixes:
>>   http://autobuild.buildroot.net/results/e8a/e8a859276db34aff87ef181b0cce98916b0afc90/
>>
>> Signed-off-by: Vicente Olivert Riera <Vincent.Riera at imgtec.com>
>
> Following the feedback from Samuel on this patch, I have marked it as
> Rejected in patchwork. Can you work with Samuel to resubmit a proper
> fix?

I'll respin the opencv series since opencv 2.4.11 is out and contains
some patches (especially the one fixing this) from the opencv-2.4.10
series (which still cleanly applies hopefully, so no big deal :-])

Regards,

-- 
Samuel



More information about the buildroot mailing list