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

cmake install fix #20

Merged
merged 1 commit into from
Aug 27, 2022
Merged

cmake install fix #20

merged 1 commit into from
Aug 27, 2022

Conversation

tylerjw
Copy link

@tylerjw tylerjw commented May 19, 2022

I needed this change to make a from source build of ros2 work where I was depending on this package.

Signed-off-by: Tyler Weaver <[email protected]>
@tylerjw tylerjw changed the base branch from melodic-devel to ros2 May 19, 2022 00:39
Copy link

@Abishalini Abishalini left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tested this on my end and it fixed my source build

@wxmerkt wxmerkt merged commit 67be62d into OctoMap:ros2 Aug 27, 2022
@Ace314159
Copy link

@tylerjw @wxmerkt This change broke my build of MoveIt2 Foxy. Can a foxy branch be created to preserve the old behavior?

@tylerjw tylerjw mentioned this pull request Aug 30, 2022
@wxmerkt
Copy link
Member

wxmerkt commented Aug 30, 2022

@Ace314159 I pushed a foxy branch without the change

@Ryanf55
Copy link

Ryanf55 commented Nov 8, 2024

Can the maintainers release to rolling? It's causing failures in our CI here:
ANYbotics/grid_map#483 (comment)

@wxmerkt
Copy link
Member

wxmerkt commented Nov 8, 2024

Thank you for the ping @Ryanf55. Release ongoing, will update #22 once done

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.

5 participants