Skip to content

Use fixed prefix for cmake_parse_arguments#5378

Merged
mvieth merged 1 commit intoPointCloudLibrary:masterfrom
SunBlack:cmake_parse_arguments_fixed_prefix
Aug 16, 2022
Merged

Use fixed prefix for cmake_parse_arguments#5378
mvieth merged 1 commit intoPointCloudLibrary:masterfrom
SunBlack:cmake_parse_arguments_fixed_prefix

Conversation

@SunBlack
Copy link
Copy Markdown
Contributor

This hopefully prevents further copy & paste issues.

This fixes a copy & paste issue in PCL_SUBSUBSYS_DEPEND which was never detect since it's initial commit nearly 9 years ago (#317), which seems to be a copy of PCL_SUBSYS_DEPEND (could be combined in a later PR).

…paste issues (like in PCL_SUBSUBSYS_DEPEND)
@SunBlack SunBlack force-pushed the cmake_parse_arguments_fixed_prefix branch from a04d05c to 3570a7b Compare August 14, 2022 17:10
Copy link
Copy Markdown
Member

@mvieth mvieth left a comment

Choose a reason for hiding this comment

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

Thanks!

@mvieth mvieth merged commit 0daa602 into PointCloudLibrary:master Aug 16, 2022
@SunBlack SunBlack deleted the cmake_parse_arguments_fixed_prefix branch August 16, 2022 11:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants