Bug 828087

Summary: Unnecessary or even wrong dependencies of opencv
Product: [Fedora] Fedora Reporter: Honza Horak <hhorak>
Component: opencvAssignee: Rakesh Pandit <rpandit>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 17CC: hhorak, karlthered, kklic, kwizart, nomis80, rpandit
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: 2012-06-04 22:10:16 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:
Attachments:
Description Flags
proposed changes in opencv.spec none

Description Honza Horak 2012-06-04 08:38:56 UTC 
Description of problem:

It seems opencv contains some BuildRequires, that are not needed. These bring unnecessary dependencies, which we should generally avoid if possible.

For example I haven't found a reason why imlib2, f2c and libucil should be in BuildRequires.

Other three prerequisites (eigen2, openni and openni-primesense) are optional, what about making them optional even in build? Users could build with them using "--with" options (--with=eigen2, --with=eigen3, --with=openni).

If I miss some point why some of these are really needed in default build, give me some explanation, please.
Comment 1 Honza Horak 2012-06-04 08:43:03 UTC 
Created attachment 589072 [details]
proposed changes in opencv.spec
Comment 2 Nicolas Chauvet (kwizart) 2012-06-04 09:35:36 UTC 
I will have a look tonight, but opencv have been made modular, so it's possible to solve things by spliting the package instead (which can be built using --with rpm conditionals also).
This might be better handled with 2.4.x version (to be updated in Rawhide only because ABI changes).
Comment 3 Nicolas Chauvet (kwizart) 2012-06-04 22:10:16 UTC 
Patch committed on master.
Thx