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. |