Bug 828087 - Unnecessary or even wrong dependencies of opencv
Summary: Unnecessary or even wrong dependencies of opencv
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: opencv
Version: 17
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: ---
Assignee: Rakesh Pandit
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-06-04 08:38 UTC by Honza Horak
Modified: 2012-06-04 22:10 UTC (History)
6 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2012-06-04 22:10:16 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
proposed changes in opencv.spec (1.12 KB, patch)
2012-06-04 08:43 UTC, Honza Horak
no flags Details | Diff

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


Note You need to log in before you can comment on or make changes to this bug.