Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

rospack depends-indent -- If there is a cycle, only prints error but not packages (which would be useful for debugging) #18

Closed
eacousineau opened this issue Jun 28, 2013 · 3 comments

Comments

@eacousineau
Copy link

Per this post, I tried out rospack depends-indent in ROS Fuerte, but it did not print out a list of packages if there was a cycle.

The usage outlined by Brian Gerkey would definitely help in this situation.

Package: ros-fuerte-rospack (2.0.13-1precise-20120908-1622-+0000)

@dirk-thomas
Copy link
Member

How is this issue different than #2?

@eacousineau
Copy link
Author

The suggestion in #2 was to use depends-indent of rospack to manually find the circular dependency, but when I tried it, it did not produce the output necessary.
I figured that this was a little more specific than #2 and had some different scope, but I can delete this issue and merge the bug report into that one if you like.

@dirk-thomas
Copy link
Member

I will commit a patch to rospack (Grooby and Hydro) soon which will add the package names involved in the circular dependency to the exception string. But the patch will very likely not be backported to Fuerte since it is not a critical bug fix and would require to rebuild and redeploy to many packages.

severin-lemaignan referenced this issue in severin-lemaignan/robotpkg Aug 18, 2014
Drop patch-aa for tinyxml location, this is now handled by
devel/ros-cmake-modules.

Changes since 2.1.19:

2.2.4 (2014-07-10)
------------------
* fix find_package(PythonLibs ...) with CMake 3 (`#42
<https://github.com/ros/rospack/issues/42>`_)

2.2.3 (2014-05-07)
------------------
* find library for exact Python version (even if not in CMake provided list of
version numbers) (`#40 <https://github.com/ros/rospack/issues/40>`_)
* find TinyXML using cmake_modules (`#24
<https://github.com/ros/rospack/issues/24>`_)
* make error messages tool specific (rospack vs. rosstack) (`#38
<https://github.com/ros/rospack/issues/38>`_)

2.2.2 (2014-02-25)
------------------
* python 3 compatibility (`#35 <https://github.com/ros/rospack/issues/35>`_)

2.2.1 (2014-02-24)
------------------
* only perform backquote substitution when needed (`#34
<https://github.com/ros/rospack/issues/34>`_)

2.2.0 (2014-01-30)
------------------
* add hash of ROS_PACKAGE_PATH to rospack/rosstack cache filename, remove
ROS_ROOT from cache (`#28 <https://github.com/ros/rospack/issues/28>`_)

2.1.22 (2014-01-07)
-------------------
* use specific python version catkin has decided on (`#29
<https://github.com/ros/rospack/issues/29>`_)
* python 3 compatibility (`#25 <https://github.com/ros/rospack/issues/25>`_,
`#27 <https://github.com/ros/rospack/issues/27>`_)
* fall back gracefully whe gtest is not available
* update package urls

2.1.21 (2013-07-05)
-------------------
* honor CATKIN_IGNORE marker file when crawling for packages (`#21
<https://github.com/ros/rospack/issues/21>`_)

2.1.20 (2013-07-03)
-------------------
* improve error message to include package names when circular dependency is
detected (`#18 <https://github.com/ros/rospack/issues/18>`_)
* check for CATKIN_ENABLE_TESTING to enable configure without tests
* add '-h' option
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants