Skip to content

List dependencies only once#812

Merged
matlabbe merged 1 commit intointrolab:masterfrom
nobleo:depend
Jan 11, 2022
Merged

List dependencies only once#812
matlabbe merged 1 commit intointrolab:masterfrom
nobleo:depend

Conversation

@Timple
Copy link
Copy Markdown
Contributor

@Timple Timple commented Jan 11, 2022

I kept the comments.

Seems like libopenni2-dev is available on all platforms these days?

@Timple
Copy link
Copy Markdown
Contributor Author

Timple commented Jan 11, 2022

Also added libpointmatcher already to replace #811

@matlabbe matlabbe merged commit f690853 into introlab:master Jan 11, 2022
@matlabbe
Copy link
Copy Markdown
Member

Perfect thx! For libopenni2-dev, I'll see next time I do push ros binaries.

@Timple
Copy link
Copy Markdown
Contributor Author

Timple commented Jan 28, 2022

Hi @matlabbe do you have an estimate on when that would be?

Compiling takes a long time, and I'm willing to test out the ros-testing binaries as soon as you bloomed them to see if the dependencies propagate properly.

@matlabbe
Copy link
Copy Markdown
Member

melodic/noetic released soon!
ros/rosdistro#31952
ros/rosdistro#31953

@Timple
Copy link
Copy Markdown
Contributor Author

Timple commented Jan 29, 2022

Great thanks 👍
Although we're on Galactic 🙃

@matlabbe
Copy link
Copy Markdown
Member

Oh I didn't see libpointmatcher on ros2 too. I'll check this later (would require a merge between master and ros2 branch for rtabmap_ros).

@matlabbe
Copy link
Copy Markdown
Member

Looks like they didn't like my latest pull requests, I'll need to redo melodic and noetic too!

@Timple
Copy link
Copy Markdown
Contributor Author

Timple commented Feb 3, 2022

They merged the new pull requests. Thank you!

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.

2 participants