dev-python/catkin_pkg: Add patch by Tim Rakowski in bug #612860 to avoid infinite...

Package-Manager: Portage-2.3.5, Repoman-2.3.2

diff --git a/dev-python/catkin_pkg/catkin_pkg-0.3.1-r2.ebuild b/dev-python/catkin_pkg/catkin_pkg-0.3.1-r3.ebuild
similarity index 86%
rename from dev-python/catkin_pkg/catkin_pkg-0.3.1-r2.ebuild
rename to dev-python/catkin_pkg/catkin_pkg-0.3.1-r3.ebuild
index f86e76ef427c7a8e59d629e82490f5b20744b94a..28e56139aea28eb41e16fe6d03c606ec271970c7 100644 (file)
--- a/dev-python/catkin_pkg/catkin_pkg-0.3.1-r2.ebuild
+++ b/dev-python/catkin_pkg/catkin_pkg-0.3.1-r3.ebuild
@@ -34,7 +34,12 @@ RDEPEND="
 DEPEND="${RDEPEND}
        test? ( dev-python/mock[${PYTHON_USEDEP}] dev-python/nose[${PYTHON_USEDEP}] )
 "
-PATCHES=( "${FILESDIR}/catkin_prefix.patch" "${FILESDIR}/argparse.patch" "${FILESDIR}/ros_packages.patch" )
+PATCHES=(
+       "${FILESDIR}/catkin_prefix.patch"
+       "${FILESDIR}/argparse.patch"
+       "${FILESDIR}/ros_packages.patch"
+       "${FILESDIR}/infinite_loop.patch"
+)
 
 python_test() {
        nosetests -s --tests test || die

diff --git a/dev-python/catkin_pkg/catkin_pkg-9999.ebuild b/dev-python/catkin_pkg/catkin_pkg-9999.ebuild
index f86e76ef427c7a8e59d629e82490f5b20744b94a..28e56139aea28eb41e16fe6d03c606ec271970c7 100644 (file)
--- a/dev-python/catkin_pkg/catkin_pkg-9999.ebuild
+++ b/dev-python/catkin_pkg/catkin_pkg-9999.ebuild
@@ -34,7 +34,12 @@ RDEPEND="
 DEPEND="${RDEPEND}
        test? ( dev-python/mock[${PYTHON_USEDEP}] dev-python/nose[${PYTHON_USEDEP}] )
 "
-PATCHES=( "${FILESDIR}/catkin_prefix.patch" "${FILESDIR}/argparse.patch" "${FILESDIR}/ros_packages.patch" )
+PATCHES=(
+       "${FILESDIR}/catkin_prefix.patch"
+       "${FILESDIR}/argparse.patch"
+       "${FILESDIR}/ros_packages.patch"
+       "${FILESDIR}/infinite_loop.patch"
+)
 
 python_test() {
        nosetests -s --tests test || die

diff --git a/dev-python/catkin_pkg/files/infinite_loop.patch b/dev-python/catkin_pkg/files/infinite_loop.patch
new file mode 100644 (file)
index 0000000..7aa3084
--- /dev/null
+++ b/dev-python/catkin_pkg/files/infinite_loop.patch
@@ -0,0 +1,27 @@
+https://bugs.gentoo.org/show_bug.cgi?id=612860
+By Tim Rakowski
+We os.walk /usr with followlinks=True, but at least nodejs contains a "recursive" link in /usr/include/node
+
+Index: catkin_pkg-0.3.1/src/catkin_pkg/packages.py
+===================================================================
+--- catkin_pkg-0.3.1.orig/src/catkin_pkg/packages.py
++++ catkin_pkg-0.3.1/src/catkin_pkg/packages.py
+@@ -52,7 +52,9 @@ def find_package_paths(basepath, exclude_paths=None, exclude_subspaces=False):
+     """
+     paths = []
+     real_exclude_paths = [os.path.realpath(p) for p in exclude_paths] if exclude_paths is not None else []
++    visited = set()
+     for dirpath, dirnames, filenames in os.walk(basepath, followlinks=True):
++        visited.add(dirpath)
+         if 'CATKIN_IGNORE' in filenames or \
+             os.path.realpath(dirpath) in real_exclude_paths or \
+                 (exclude_subspaces and '.catkin' in filenames):
+@@ -63,7 +65,7 @@ def find_package_paths(basepath, exclude_paths=None, exclude_subspaces=False):
+             del dirnames[:]
+             continue
+         for dirname in dirnames:
+-            if dirname.startswith('.'):
++            if dirname.startswith('.') or os.path.realpath(os.path.join(dirpath, dirname)) in visited:
+                 dirnames.remove(dirname)
+     return paths
+