Main inclusion request for OpenCV

Bug #324523 reported by Luka Renko
4
Affects Status Importance Assigned to Milestone
opencv (Ubuntu)
Won't Fix
Medium
Luka Renko

Bug Description

Main inclusion request for OpenCV (opencv source package). Build dependancy for KIPI-Plugins (RedEyeRemoval).

https://wiki.ubuntu.com/MainInclusionReportOpenCV

Luka Renko (lure)
Changed in opencv:
importance: Undecided → Medium
Revision history for this message
Kees Cook (kees) wrote :

otherlibs/highgui seems to have v4l capabilities, but the package is not using libv4l do handle the devices. This may cause problems.

I'd like to see the various build warnings from the buildlog[1] cleaned up before this MIR continues. e.g.:
cxpersistence.cpp:1001: warning: suggest parentheses around && within ||
cxpersistence.cpp:2800: warning: ignoring return value of 'char* fgets(char*, int, FILE*)', declared with attribute warn_unused_result
cvcalibfilter.cpp:733: warning: ignoring return value of 'int fscanf(FILE*, const char*, ...)', declared with attribute warn_unused_result
_cv.cpp:867: warning: format not a string literal and no format arguments

[1] https://edge.launchpad.net/ubuntu/+source/opencv/1.0.0-6.1build1/+build/826341

Changed in opencv:
status: New → Incomplete
Revision history for this message
Luka Renko (lure) wrote :

Kees, thanks for review. Will look into this and talk with upstream how to properly address v4l capabilities.

Changed in opencv:
assignee: nobody → lure
Revision history for this message
Luka Renko (lure) wrote :

It looks like that this will be too big task to properly solve for Jaunty: I have backported/adapted most of the fixes for warnings from upstream SVN (it still took me quite some time :-(), but upstream still does not have any error handling for fgets/fscanf/fwrite calls. :-(
Also it still uses v4l directly (w/o libv4l library).

I just do not have enough time to personally work on solving this issue for Jaunty. Based on Kees comment, it does not look like we can get this into Main. I will inform upstream and hope that they can do something about this for 1.1 release.

Revision history for this message
Martin Pitt (pitti) wrote :

Closing due to inactivity. Please reopen if you still want/need this and the identified problems were fixed. Thank you!

Changed in opencv (Ubuntu):
status: Incomplete → Won't Fix
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.