Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 632116 - media-libs/opencv should add PYTHON_COMPAT python3_6
Summary: media-libs/opencv should add PYTHON_COMPAT python3_6
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Amy Liffey
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-09-27 02:32 UTC by Leonard Lausen
Modified: 2018-01-21 08:36 UTC (History)
2 users (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Leonard Lausen 2017-09-27 02:32:26 UTC
media-libs/opencv should add PYTHON_COMPAT python3_6.

The following binary opencv distribution is compatible with python 3.6, so the gentoo ebuild should also be: https://pypi.python.org/pypi/opencv-python

Thanks!
Comment 1 Tomáš Mózes 2017-09-27 07:17:39 UTC
Have you tested with Python 3.6?
Comment 2 Leonard Lausen 2017-09-28 08:41:02 UTC
I haven't tested with python 3.6 yet. Unfortunately I can't build opencv-3.3 in the first place (i.e. independently of enabling python3_6):

>  In file included from /var/tmp/portage/media-libs/opencv-3.3.0-r1/work/opencv-3.3.0/modules/dnn/misc/tensorflow/attr_value.pb.cc:5:0:                                           
>  /var/tmp/portage/media-libs/opencv-3.3.0-r1/work/opencv-3.3.0/modules/dnn/misc/tensorflow/attr_value.pb.h:12:2: error: #error This file was generated by a newer version of protoc which is                                                                                                                                                                     
>  #error This file was generated by a newer version of protoc which is                                                                                                           
>  ^~~~~                                                                                                                                                                         
>  /var/tmp/portage/media-libs/opencv-3.3.0-r1/work/opencv-3.3.0/modules/dnn/misc/tensorflow/attr_value.pb.h:13:2: error: #error incompatible with your Protocol Buffer headers. Pl
>  ease update                                                                                                                                                                     
>  #error incompatible with your Protocol Buffer headers.  Please update                                                                                                          
>  ^~~~~
>  /var/tmp/portage/media-libs/opencv-3.3.0-r1/work/opencv-3.3.0/modules/dnn/misc/tensorflow/attr_value.pb.h:14:2: error: #error your headers.
>  #error your headers.
>  ^~~~~
>  /var/tmp/portage/media-libs/opencv-3.3.0-r1/work/opencv-3.3.0/modules/dnn/misc/tensorflow/attr_value.pb.h:22:35: fatal error: google/protobuf/arena.h: No such file or directory
>  #include <google/protobuf/arena.h>



I assume opencv 3.3 should in some way depend on the ~amd64 protobuf..?
Comment 3 Amy Liffey gentoo-dev 2017-09-28 19:34:21 UTC
(In reply to Leonard Lausen from comment #2)
> I haven't tested with python 3.6 yet. Unfortunately I can't build opencv-3.3
> in the first place (i.e. independently of enabling python3_6):
> 
> >  In file included from /var/tmp/portage/media-libs/opencv-3.3.0-r1/work/opencv-3.3.0/modules/dnn/misc/tensorflow/attr_value.pb.cc:5:0:                                           
> >  /var/tmp/portage/media-libs/opencv-3.3.0-r1/work/opencv-3.3.0/modules/dnn/misc/tensorflow/attr_value.pb.h:12:2: error: #error This file was generated by a newer version of protoc which is                                                                                                                                                                     
> >  #error This file was generated by a newer version of protoc which is                                                                                                           
> >  ^~~~~                                                                                                                                                                         
> >  /var/tmp/portage/media-libs/opencv-3.3.0-r1/work/opencv-3.3.0/modules/dnn/misc/tensorflow/attr_value.pb.h:13:2: error: #error incompatible with your Protocol Buffer headers. Pl
> >  ease update                                                                                                                                                                     
> >  #error incompatible with your Protocol Buffer headers.  Please update                                                                                                          
> >  ^~~~~
> >  /var/tmp/portage/media-libs/opencv-3.3.0-r1/work/opencv-3.3.0/modules/dnn/misc/tensorflow/attr_value.pb.h:14:2: error: #error your headers.
> >  #error your headers.
> >  ^~~~~
> >  /var/tmp/portage/media-libs/opencv-3.3.0-r1/work/opencv-3.3.0/modules/dnn/misc/tensorflow/attr_value.pb.h:22:35: fatal error: google/protobuf/arena.h: No such file or directory
> >  #include <google/protobuf/arena.h>
> 
> 
> 
> I assume opencv 3.3 should in some way depend on the ~amd64 protobuf..?

Yes there is another 5 bugs for it. We are working on repair ;)

Thank you..
Comment 4 Leonard Lausen 2017-10-10 11:36:50 UTC
Thanks for your work on it! With #631418 fixed I tried compiling with PYTHON_COMPAT python3_6. Everything seems to work.
Comment 5 Amy Liffey gentoo-dev 2017-10-10 12:22:02 UTC
(In reply to Leonard Lausen from comment #4)
> Thanks for your work on it! With #631418 fixed I tried compiling with
> PYTHON_COMPAT python3_6. Everything seems to work.

I will add it to 3.3.* with the next fix. 

Thank you :)
Comment 6 Larry the Git Cow gentoo-dev 2018-01-21 08:36:40 UTC
The bug has been closed via the following commit(s):

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

commit b311829f28929c5dd15402b395365ccffb85ecd1
Author:     Amy Liffey <amynka@gentoo.org>
AuthorDate: 2018-01-21 08:35:52 +0000
Commit:     Amy Liffey <amynka@gentoo.org>
CommitDate: 2018-01-21 08:36:21 +0000

    media-libs/opencv: fix cuda9 and autodownload
    
    - Fix cuda 9 compatibility
    Submitted-by: Todd Walter <twalter@rogers.com>
    
    - Remove autodownload tiny_dnn from contrib
    Submitted-by: Jonathan Scruggs <j.scruggs@gmail.com>
    
    - Add PYTHON_COMPAT 3_6
    
    Closes: https://bugs.gentoo.org/637078
    Closes: https://bugs.gentoo.org/633844
    Closes: https://bugs.gentoo.org/632116
    Package-Manager: Portage-2.3.13, Repoman-2.3.3

 media-libs/opencv/Manifest                         |   1 +
 .../opencv/files/opencv-3.3.0-cuda9-cmake.patch    |  87 ++++
 .../files/opencv-3.3.0-cuda9-compute20.patch       |  21 +
 .../opencv/files/opencv-3.3.0-cuda9-saturate.patch |  41 ++
 media-libs/opencv/metadata.xml                     |   1 +
 media-libs/opencv/opencv-3.3.0-r5.ebuild           | 466 +++++++++++++++++++++
 6 files changed, 617 insertions(+)