Gentoo Archives: gentoo-dev

From: hasufell <hasufell@g.o>
To: Alexis Ballier <aballier@g.o>
Cc: gentoo-dev@l.g.o
Subject: [gentoo-dev] Re: [gentoo-commits] repo/gentoo:master commit in: dev-ros/compressed_image_transport/
Date: Fri, 09 Oct 2015 17:19:03
Message-Id: 5617F6F9.40903@gentoo.org
1 On 10/09/2015 06:37 PM, Alexis Ballier wrote:
2 > commit: 4440cc9d950795eaae8f8b049bbf37c073ef5ea0
3 > Author: Alexis Ballier <aballier <AT> gentoo <DOT> org>
4 > AuthorDate: Fri Oct 9 16:37:18 2015 +0000
5 > Commit: Alexis Ballier <aballier <AT> gentoo <DOT> org>
6 > CommitDate: Fri Oct 9 16:37:18 2015 +0000
7 > URL: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=4440cc9d
8 >
9 > dev-ros/compressed_image_transport: Add PYTHON_COMPAT and PYTHON_USEDEP on dynamic_reconfigure. Bug #562664.
10 >
11 > Package-Manager: portage-2.2.23
12 >
13
14 [...]
15
16 > --- a/dev-ros/compressed_image_transport/compressed_image_transport-1.9.2.ebuild
17 > +++ b/dev-ros/compressed_image_transport/compressed_image_transport-1.9.2-r1.ebuild
18 > @@ -6,6 +6,7 @@ EAPI=5
19 > ROS_REPO_URI="https://github.com/ros-perception/image_transport_plugins"
20 > KEYWORDS="~amd64 ~arm"
21 > ROS_SUBDIR=${PN}
22 > +PYTHON_COMPAT=( python2_7 )
23 >
24 > inherit ros-catkin
25 >
26 > @@ -16,7 +17,7 @@ IUSE=""
27 >
28 > RDEPEND="
29 > dev-ros/cv_bridge
30 > - dev-ros/dynamic_reconfigure
31 > + dev-ros/dynamic_reconfigure[${PYTHON_USEDEP}]
32 > dev-ros/image_transport
33 > dev-ros/tf
34 > media-libs/opencv
35 >
36 > diff --git a/dev-ros/compressed_image_transport/compressed_image_transport-9999.ebuild b/dev-ros/compressed_image_transport/compressed_image_transport-9999.ebuild
37 > index f7a387a..144fac9 100644
38 > --- a/dev-ros/compressed_image_transport/compressed_image_transport-9999.ebuild
39 > +++ b/dev-ros/compressed_image_transport/compressed_image_transport-9999.ebuild
40 > @@ -6,6 +6,7 @@ EAPI=5
41 > ROS_REPO_URI="https://github.com/ros-perception/image_transport_plugins"
42 > KEYWORDS="~amd64 ~arm"
43 > ROS_SUBDIR=${PN}
44 > +PYTHON_COMPAT=( python2_7 )
45 >
46 > inherit ros-catkin
47 >
48 > @@ -16,7 +17,7 @@ IUSE=""
49 >
50 > RDEPEND="
51 > dev-ros/cv_bridge
52 > - dev-ros/dynamic_reconfigure
53 > + dev-ros/dynamic_reconfigure[${PYTHON_USEDEP}]
54 > dev-ros/image_transport
55 > dev-ros/tf
56 > media-libs/opencv
57 >
58
59 Maybe I am missing something, but ${PYTHON_DEPS} seems missing here and
60 I cannot find it in ros-catkin.eclass either. It's only an indirect dep
61 which is inappropriate.
62
63 Maybe other ros packages are affected as well.

Replies