Bug 1236596 - heads-up opencv3 is comming
Summary: heads-up opencv3 is comming
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: simon
Version: 23
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Rex Dieter
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: opencv3
TreeView+ depends on / blocked
 
Reported: 2015-06-29 13:44 UTC by Jozef Mlich
Modified: 2015-08-25 09:56 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-08-14 13:34:57 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
opencv includes should contain "opencv/" (5.97 KB, patch)
2015-06-29 13:44 UTC, Jozef Mlich
no flags Details | Diff
simon-0.4.1-opencv3.patch (4.17 KB, patch)
2015-08-14 10:25 UTC, Viktor Jancik
no flags Details | Diff
Spec file patch against simon 0.4.1-5 (1.40 KB, patch)
2015-08-14 10:28 UTC, Viktor Jancik
no flags Details | Diff


Links
System ID Private Priority Status Summary Last Updated
KDE Software Compilation 351279 0 None None None Never

Description Jozef Mlich 2015-06-29 13:44:36 UTC
Created attachment 1044375 [details]
opencv includes should contain "opencv/"

Description of problem:
Simon package build is failing with OpenCV 3.0.0 See #1230078. 

The main reason seems to be incorrect path in #include directive. Currently it is #include <cv.h> and it should be #include <opencv/cv.h>. Similary with <highgui.h>.

Additionally, the detection of "BuilRequires: qt-webkit-devel" wasn't working correctly. I have added that manually into spec file.

I have tested this patch in my COPR repository:
https://copr-be.cloud.fedoraproject.org/results/jmlich/opencv3/fedora-22-x86_64/simon-0.4.1-5.fc23/

I wasn't able to find upstream bugzilla )-;

Comment 1 Jan Kurik 2015-07-15 13:53:45 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 3 Viktor Jancik 2015-08-14 10:25:37 UTC
Created attachment 1062970 [details]
simon-0.4.1-opencv3.patch

Separate patch file ready for inclusion.
I created it because the release tag got bumped and the spec file changed.

Comment 4 Viktor Jancik 2015-08-14 10:28:36 UTC
Created attachment 1062972 [details]
Spec file patch against simon 0.4.1-5

Comment 5 Viktor Jancik 2015-08-14 10:32:12 UTC
Created an upstream bug report:
https://bugs.kde.org/show_bug.cgi?id=351279

Comment 6 Rex Dieter 2015-08-14 13:04:32 UTC
Thanks, I'll try to take a closer look soon.

Comment 7 Rex Dieter 2015-08-14 13:22:56 UTC
This part is not needed

@@ -40,6 +41,7 @@ 
 BuildRequires: pkgconfig(qwt) >= 6
 BuildRequires: pkgconfig(samplerate)
 BuildRequires: qaccessibilityclient-devel
+BuildRequires: qt-webkit-devel


was fixed as part of bug #1240006 already

Comment 8 Rex Dieter 2015-08-14 13:34:57 UTC
%changelog
* Fri Aug 14 2015 Rex Dieter <rdieter> 0.4.1-6
- includes of opencv headers should contain "opencv/" (#1236596)


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