Bug 1238734

Summary: /usr/bin/ld: /tmp/cckWdv8g.o: undefined reference to symbol 'cvWriteFrame' with OpenCV 3.0.0
Product: [Fedora] Fedora Reporter: Jozef Mlich <jmlich>
Component: gmicAssignee: Daniel Berrangé <berrange>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 23CC: berrange, hhorak, sergio
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2016-05-03 21:06:28 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1230078    
Attachments:
Description Flags
use pkg-config to get CFLAGS and LIBS none

Description Jozef Mlich 2015-07-02 13:43:42 UTC 
Created attachment 1045521 [details]
use pkg-config to get CFLAGS and LIBS

Description of problem:
We are trying to recompile packages against opencv 3.0.0. (See #1230078). The gmic is not compiling with following error message:

/usr/bin/ld: /tmp/cckWdv8g.o: undefined reference to symbol 'cvWriteFrame'
/lib/libopencv_videoio.so.3.0: error adding symbols: DSO missing from command line

It seems the LIBS variable contains incorrect values. The attached patch is changing Makefile to get those values from pkg-config. I would like to ask for review of this patch.

The previous version contained hard-coded values of CFLAGS and LIBS. I haven't found any justification for this. I guess those settings was given by some ancient version of opencv (the comment in Makefile is speaking about debian?). My suggestion is to drop the hard coded part.

I am not sure whether I found correct upstream repository. Here is my pull request:
https://github.com/dtschump/gmic/pull/1
Comment 1 Daniel Berrangé 2015-07-02 14:00:05 UTC 
The github repo seems to be a mirror of the main repo https://forge.greyc.fr/projects/gmic/repository  I usually send patches to the mailing list directly. The change does look reasonable - the makefiles have a lot of old cruft in them, so wouldn't worry about why the current code is the way it is.
Comment 2 Jan Kurik 2015-07-15 13:51:27 UTC 
This bug appears to have been reported against 'rawhide' during the Fedora 23 development cycle.
Changing version to '23'.

(As we did not run this process for some time, it could affect also pre-Fedora 23 development
cycle bugs. We are very sorry. It will help us with cleanup during Fedora 23 End Of Life. Thank you.)

More information and reason for this action is here:
https://fedoraproject.org/wiki/BugZappers/HouseKeeping/Fedora23
Comment 4 Sergio Basto 2016-05-03 21:06:28 UTC 
gmic built on rawhide with OpenCV 3.1.0 , closing .

Thanks.