Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

include module name which wasn't found in error message #121

Merged
merged 1 commit into from
Jun 17, 2018

Conversation

dirk-thomas
Copy link
Member

pkg_check_modules() simply output the following not helpful error message when a requested module is not found:

A required package was not found

Therefore this patch handles printing the error message separately to include the module name.

@dirk-thomas dirk-thomas added enhancement New feature or request in review Waiting for review (Kanban column) labels Jun 16, 2018
@dirk-thomas dirk-thomas self-assigned this Jun 16, 2018
@dirk-thomas dirk-thomas force-pushed the not_found_required_module branch 2 times, most recently from 92367e3 to cf79d10 Compare June 17, 2018 01:10
Copy link
Member

@wjwwood wjwwood left a comment

Choose a reason for hiding this comment

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

Other than a left over cmake variable, lgtm.

@@ -26,8 +26,16 @@ macro(find_ros1_package name)
set(_required "")
Copy link
Member

Choose a reason for hiding this comment

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

This is no longer required right?

Copy link
Member Author

Choose a reason for hiding this comment

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

Good catch. Removed.

@dirk-thomas dirk-thomas force-pushed the not_found_required_module branch from cf79d10 to ab0578f Compare June 17, 2018 02:18
@dirk-thomas dirk-thomas merged commit d4c4eca into master Jun 17, 2018
@dirk-thomas dirk-thomas removed the in review Waiting for review (Kanban column) label Jun 17, 2018
@dirk-thomas dirk-thomas deleted the not_found_required_module branch June 17, 2018 02:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants