Skip to content

Remove unused variable output_library#211

Merged
sloretz merged 1 commit intoros2from
sloretz__pluginlib__remove_unused_variable
Oct 19, 2020
Merged

Remove unused variable output_library#211
sloretz merged 1 commit intoros2from
sloretz__pluginlib__remove_unused_variable

Conversation

@sloretz
Copy link
Copy Markdown
Contributor

@sloretz sloretz commented Oct 19, 2020

Signed-off-by: Shane Loretz <sloretz@osrfoundation.org>
@sloretz
Copy link
Copy Markdown
Contributor Author

sloretz commented Oct 19, 2020

CI (build: --packages-up-to pluginlib test: --packages-select pluginlib)

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

@sloretz sloretz merged commit 86200e8 into ros2 Oct 19, 2020
sloretz added a commit that referenced this pull request Oct 19, 2020
Signed-off-by: Shane Loretz <sloretz@osrfoundation.org>
@sloretz sloretz deleted the sloretz__pluginlib__remove_unused_variable branch October 19, 2020 23:46
sloretz added a commit that referenced this pull request Dec 7, 2021
* [ROS 2] Add function for same-package pluginlib tests (#201)

* Add function for same-package pluginlib tests

Adds pluginlib_enable_plugin_testing()
This CMake function handles creating a fake install environment for
testing pluginlib plugins in the same package that built them.
This replaces manually creating this folder structure just to run unit
tests.

Signed-off-by: Shane Loretz<sloretz@openrobotics.org>
Signed-off-by: Shane Loretz <sloretz@osrfoundation.org>

* Grammar and unused variable

Signed-off-by: Shane Loretz <sloretz@osrfoundation.org>

* Typo in comment

Signed-off-by: Shane Loretz <sloretz@osrfoundation.org>
Signed-off-by: Shane Loretz <sloretz@openrobotics.org>

* cmake 3.5 compatibility (#203)

Signed-off-by: Karsten Knese <karsten@openrobotics.org>
Signed-off-by: Shane Loretz <sloretz@osrfoundation.org>

* Remove unused variable output_library (#211)

Signed-off-by: Shane Loretz <sloretz@osrfoundation.org>

Co-authored-by: Karsten Knese <Karsten1987@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants