Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 656530 - media-video/vlc-3.0.2 fails at build with media-libs/opencv-3.4.1
Summary: media-video/vlc-3.0.2 fails at build with media-libs/opencv-3.4.1
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Amy Liffey
URL: https://github.com/opencv/opencv/issu...
Whiteboard:
Keywords: PATCH
Depends on:
Blocks:
 
Reported: 2018-05-25 22:07 UTC by jorgicio
Modified: 2018-05-31 18:37 UTC (History)
5 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
The patch in question (opencv-3.4.1-legacy-c.patch,474 bytes, patch)
2018-05-25 22:09 UTC, jorgicio
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description jorgicio 2018-05-25 22:07:59 UTC
Hi!

I found that after trying to build vlc with opencv-3.4.1, it fails to build due to unsupported C/C++ extensions. The patch I'm about to upload (in order to apply to opencv) fixes the issue, as mentioned in the URL I attached.

Thanks.

Reproducible: Always
Comment 1 jorgicio 2018-05-25 22:09:03 UTC
Created attachment 533254 [details, diff]
The patch in question

This enables the legacy C/C++ support still used by VLC and some other apps.
Comment 2 Juergen Rose 2018-05-26 18:35:27 UTC
I have the same issue. Is there not any patch to vlc, that it can be emerged with the current opencv-3.4.1?
Comment 3 jorgicio 2018-05-26 18:39:24 UTC
(In reply to Juergen Rose from comment #2)
> I have the same issue. Is there not any patch to vlc, that it can be emerged
> with the current opencv-3.4.1?

I don't know, I couldn't find any patch for that, also, in the URL posted above, is mentioned the issue doesn't affect only to vlc, but to some other packages.
Comment 4 Attila Tóth 2018-05-29 17:21:09 UTC
I used this patch of the referenced URL:
https://github.com/opencv/opencv/pull/11605
https://github.com/opencv/opencv/pull/11605/commits/549b5df22520b60b91dd77096434d79425b31ac2
It worked for me.
Comment 5 jorgicio 2018-05-29 18:11:31 UTC
(In reply to Attila Tóth from comment #4)
> I used this patch of the referenced URL:
> https://github.com/opencv/opencv/pull/11605
> https://github.com/opencv/opencv/pull/11605/commits/
> 549b5df22520b60b91dd77096434d79425b31ac2
> It worked for me.

This patch is more complete than I found. Thanks!
Comment 6 Larry the Git Cow gentoo-dev 2018-05-31 17:24:56 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=fbadf5fc3cdc26df32995e2e1cad1ed47f40efad

commit fbadf5fc3cdc26df32995e2e1cad1ed47f40efad
Author:     Amy Liffey <amynka@gentoo.org>
AuthorDate: 2018-05-31 17:15:58 +0000
Commit:     Amy Liffey <amynka@gentoo.org>
CommitDate: 2018-05-31 17:24:32 +0000

    media-libs/opencv: workaround C compilation mode
    
    - Add nvcc flags to fix compilation with cuda
    - Fix issue with C compilation mode for vlc
    - Fix python installation path
    
    Closes: https://bugs.gentoo.org/656530
    Closes: https://bugs.gentoo.org/656580
    Package-Manager: Portage-2.3.24, Repoman-2.3.6

 .../files/opencv-3.4.1-compilation-C-mode.patch    | 56 ++++++++++++++++++++++
 .../opencv-3.4.1-cuda-add-relaxed-constexpr.patch  | 14 ++++++
 .../opencv-3.4.1-python-lib-suffix-hack.patch      | 13 +++++
 ...encv-3.4.1-r1.ebuild => opencv-3.4.1-r2.ebuild} |  6 ++-
 4 files changed, 88 insertions(+), 1 deletion(-)