-
Notifications
You must be signed in to change notification settings - Fork 10
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
package doesn't install package.xml #15
Comments
That's odd. It looks like the Melodic release does contain a package.xml and installs it: |
I just checked the Debian package List of files included in the Debian package: `dpkg -L ros-melodic-ompl`/. Also the CMake docs for
So I would assume you need to patch the CMake logic to pass an absolute path:
|
Should be fixed now. I did another bloom release. PR: ros/rosdistro#23174 |
@mamoll I just tried the same for the latest Debian package ( Your condition might be incorrect: |
Since this check is in the top-level dir, shouldn't |
The condition uses |
Ah, you're right. My bad! |
Should be fixed for real now. ROS distro PR is at ros/rosdistro#23290 |
Thanks, the latest Debian package does contain the package manifest now. |
While the release repo adds a
package.xml
file it doesn't install it at all. It must be installed into<prefix>/share/ompl/
in order for the package to be discoverable by other ROS tools. E.g. see ros/rospack#111.It seems the same problem has been reported in the past for Kinetic: #9.
@mamoll Please add the necessary install rules and release new deb incs into all active ROS distributions which currently don't have it.
The text was updated successfully, but these errors were encountered: