commit:     3d2f3b95a33db7b630e93463583b1e8b23312e89
Author:     David Seifert <soap <AT> gentoo <DOT> org>
AuthorDate: Tue Apr 18 17:57:39 2017 +0000
Commit:     David Seifert <soap <AT> gentoo <DOT> org>
CommitDate: Tue Apr 18 18:22:27 2017 +0000
URL:        https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=3d2f3b95

ros-catkin.eclass: Fix python handling

* Add missing REQUIRED_USE which otherwise could
  lead to spurious failures during the build.
* Do not directly depend on dev-lang/python-exec,
  this is done properly via ${PYTHON_DEPS}.
* python-utils-r1 never needs to be inherited
  explicitly.
Closes: https://github.com/gentoo/gentoo/pull/4444

 eclass/ros-catkin.eclass | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/eclass/ros-catkin.eclass b/eclass/ros-catkin.eclass
index fec873a922c..feb626341eb 100644
--- a/eclass/ros-catkin.eclass
+++ b/eclass/ros-catkin.eclass
@@ -48,7 +48,7 @@ fi
 PYTHON_ECLASS=""
 CATKIN_PYTHON_USEDEP=""
 if [ -n "${PYTHON_COMPAT}" ] ; then
-       PYTHON_ECLASS="python-r1 python-utils-r1"
+       PYTHON_ECLASS="python-r1"
 fi
 
 inherit ${SCM} ${PYTHON_ECLASS} cmake-utils flag-o-matic
@@ -57,6 +57,7 @@ CATKIN_DO_PYTHON_MULTIBUILD=""
 if [ -n "${PYTHON_COMPAT}" ] ; then
        CATKIN_PYTHON_USEDEP="[${PYTHON_USEDEP}]"
        CATKIN_DO_PYTHON_MULTIBUILD="yes"
+       REQUIRED_USE="${PYTHON_REQUIRED_USE}"
 fi
 
 IUSE="test"
@@ -67,7 +68,7 @@ RDEPEND="
 DEPEND="${RDEPEND}"
 
 if [ -n "${CATKIN_DO_PYTHON_MULTIBUILD}" ] ; then
-       RDEPEND="${RDEPEND} dev-lang/python-exec:2 ${PYTHON_DEPS}"
+       RDEPEND="${RDEPEND} ${PYTHON_DEPS}"
        DEPEND="${DEPEND} ${PYTHON_DEPS}"
 fi
 

Reply via email to