Skip to content

Jazzy testing - #4

Merged
ndunkelb-nasa merged 5 commits into
mainfrom
jazzy-testing
Mar 26, 2026
Merged

Jazzy testing#4
ndunkelb-nasa merged 5 commits into
mainfrom
jazzy-testing

Conversation

@ndunkelb-nasa

Copy link
Copy Markdown
Contributor

There are two things going on in migrating to jazzy -

  • Several repos moving to .hpp files instead of .h files. I don't know if these will be backwards compatible to humble... so not sure if we need separate branches here? Ill take a look
  • Was having some issues compiling a dependency (MPI). It seems like maybe they didn't expose their dependencies correctly? I'm not really sure I understand why this is necessary, but this seemed to resolve the issues I was having...

@ndunkelb-nasa ndunkelb-nasa self-assigned this Mar 25, 2026
@ndunkelb-nasa

Copy link
Copy Markdown
Contributor Author

I'm sad to say the .h vs .hpp changes are not backwards compatible. Maybe we do some compile time checking for the proper header version?

@eholum-nasa eholum-nasa left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

It builds at least!

Comment thread include/realsense_polled_camera/RealSensePolledCameraNode.hpp Outdated
Comment thread CMakeLists.txt

@eholum-nasa eholum-nasa left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Re-approving since I also sanity checked the build on jazzy.

@ndunkelb-nasa
ndunkelb-nasa merged commit 43bd039 into main Mar 26, 2026
1 check 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.

2 participants