Skip to content

add explicit dependency on rcl_logging_log4cxx#435

Merged
dirk-thomas merged 1 commit intomasterfrom
dirk-thomas/explicit-logging-dep
May 9, 2019
Merged

add explicit dependency on rcl_logging_log4cxx#435
dirk-thomas merged 1 commit intomasterfrom
dirk-thomas/explicit-logging-dep

Conversation

@dirk-thomas
Copy link
Copy Markdown
Member

@dirk-thomas dirk-thomas commented May 9, 2019

Makes it possible for the devel / PR jobs to pass, e.g. http://build.ros2.org/view/Ddev/job/Ddev__rcl__ubuntu_bionic_amd64/21/

Signed-off-by: Dirk Thomas <dirk-thomas@users.noreply.github.com>
@dirk-thomas dirk-thomas added bug Something isn't working in review Waiting for review (Kanban column) labels May 9, 2019
@dirk-thomas dirk-thomas self-assigned this May 9, 2019
Copy link
Copy Markdown
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.

@dirk-thomas dirk-thomas merged commit 816ce67 into master May 9, 2019
@delete-merged-branch delete-merged-branch bot deleted the dirk-thomas/explicit-logging-dep branch May 9, 2019 21:20
@dirk-thomas dirk-thomas removed the in review Waiting for review (Kanban column) label May 9, 2019
@dirk-thomas
Copy link
Copy Markdown
Member Author

clalancette added a commit that referenced this pull request May 14, 2019
This reverts commit 816ce67.

Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
dirk-thomas pushed a commit that referenced this pull request May 15, 2019
#436)

* Revert "Changes the default 3rd party logger from rcl_logging_noop to rcl_logging_log4cxx (#425)"

This reverts commit ac8ee90.

Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>

* Revert "add explicit dependency on rcl_logging_log4cxx (#435)"

This reverts commit 816ce67.

Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>

* Add dependency on rcl_logging_noop.

Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants