Fix overriding binary install

[tylerjw]

Overriding is when you build a package from a source that is also installed as a binary. Currently, this does not work in MoveIt because of the way header files are found. The problem is that the binary header is found before the source header when building and if the source one is newer and has changes the compile will break in surprising ways.

#1901 is a good example of what needs to be done everywhere in moveit.

Here are the colcon docs on overriding: https://colcon.readthedocs.io/en/released/user/overriding-packages.html#how-to-make-it-easier-for-your-users-to-override

Tasks:

• Search for cmake files that are missing this package directory
• Make PR fixing the issue

[bailaC]

@tylerjw I think we can close this issue as it is already merged.

[tylerjw]

I do not believe the linked to PR fixed this for the whole codebase.

[mariobo8]

Hi @tylerjw I really would like to work on this. can I?

[sjahr]

@mariobo8 Yes, go for it.

[advait-zx]

is this issue still open?

[mikeferguson]

Duplicate of #3134 which has more recent documentation to work from