jrl-umi3218 / jrl-cmakemodules

CMake utility toolbox
https://jrl-cmakemodules.readthedocs.io/en/master/
Other
59 stars 47 forks source link

[sdformat] check for different sdformat packages, and add project dependency when found #501

Closed proyan closed 2 years ago

proyan commented 2 years ago

The last commit should handle both your suggestions.

nim65s commented 2 years ago

Maybe we could add a REQUIRED argument, that raise an error on reaching the end of the loop ?

proyan commented 2 years ago

@nim65s That is a good suggestion. https://github.com/jrl-umi3218/jrl-cmakemodules/pull/501/commits/d9df3999735feb0402d88c142fd517a7ecb7d99f should take care of that