Skip to content

Fix build errors in Ubuntu Lunar when fetched directly#511

Open
Ryanf55 wants to merge 1 commit into
ros-perception:rollingfrom
Ryanf55:ubuntu-lunar-build-errors
Open

Fix build errors in Ubuntu Lunar when fetched directly#511
Ryanf55 wants to merge 1 commit into
ros-perception:rollingfrom
Ryanf55:ubuntu-lunar-build-errors

Conversation

@Ryanf55
Copy link
Copy Markdown

@Ryanf55 Ryanf55 commented Jun 24, 2023

I had some build/include errors when building in Ubuntu Lunar (23.04) when compiling nav2 from source.

I've modified the CMakelists to make use of the target_sources command.
The library is now a level higher.
The commands to target_include_directories are properly scoped depending on the directory, especially for the generated export header. No more traversing up directories and back down.

Relates to #419

@Ryanf55 Ryanf55 force-pushed the ubuntu-lunar-build-errors branch from 48a45e8 to 0c9b309 Compare June 24, 2023 01:17
* Remove double nested include dirs

Signed-off-by: Ryan Friedman <ryanfriedman5410+github@gmail.com>
@Ryanf55 Ryanf55 force-pushed the ubuntu-lunar-build-errors branch from 0c9b309 to 9cd9436 Compare June 24, 2023 02:13
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.

1 participant