-
Notifications
You must be signed in to change notification settings - Fork 8
generate package.xml #18
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
Comments
This is linked with the fact that we do NOT want to build install space (because of python intricacies when deploying installing packages). The idea for the catkin_pip workflow is :
This should work fine as long as everyone uses rosdep for dependencies, and rosdep support pip (which I think it does). |
We should be careful to not break what already works though, and we should probably add this new package build as an extra feature : catkin_symbolic_package() building a deb package that just reference another package in another rosdep supported package manager. |
Not sure if this is really useful or not after all. In any case, the documentation in that package can refer tothe original package documentation. |
This is still useful in case we want to use a python repository, in a catkin_pip environment => we do not need to store the package.xml in the repository to test our "development" (pure) python package with ROS. So one would need only a CMakeLists.txt to get the pure python package to build with catkin_pip. The release repo can set his own package.xml as with thirdparty release afterwards anyway regardless. This is because currently, even using catkin_pip, catkin_package complains if there is no package.xml in the source repo:
|
Already implemented there : https://github.com/mikepurvis/catkin_tools_python Not sure how to integrate yet. delayed, not for release 0.2. |
Uh oh!
There was an error while loading. Please reload this page.
We need a way to generate package.xml file, and avoid discrepancies between python package details and catkin package details...
The text was updated successfully, but these errors were encountered: