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

Fix package dependencies #51

Merged
merged 1 commit into from
Sep 2, 2024
Merged

Fix package dependencies #51

merged 1 commit into from
Sep 2, 2024

Conversation

rhaschke
Copy link
Contributor

@rhaschke rhaschke commented Sep 2, 2024

Depend on sqlite3_vendor, but not SQLite3 and libsqlite3-dev.
Fixes #50.

Depend on sqlite3_vendor, but not SQLite3 and libsqlite3-dev.
@fmauch
Copy link

fmauch commented Sep 2, 2024

I cannot fully test this, but rosdep seems happy:

rosdep install --from-paths . -t build_export -s -v
...
resolving for resources [warehouse_ros_sqlite]
resolve_all: resource [warehouse_ros_sqlite] requires rosdep keys [warehouse_ros, class_loader, rclcpp, sqlite3_vendor]

Without that change the output is

resolve_all: resource [warehouse_ros_sqlite] requires rosdep keys [warehouse_ros, class_loader, rclcpp]

@rhaschke rhaschke merged commit 8479f43 into moveit:ros2 Sep 2, 2024
8 checks passed
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

Successfully merging this pull request may close these issues.

Missing build_export dependencies?
2 participants