Skip to content

Commit

Permalink
Fix name of argument passed to NodeStrategy (ros2#227)
Browse files Browse the repository at this point in the history
NodeStrategy does not look for an argument named 'use_daemon'.
Instead it checks for an argument named 'no_daemon'.
https://github.com/ros2/ros2cli/blob/95bfb056df795b6eef8bb73b165db31fcf2d867a/ros2cli/ros2cli/node/strategy.py#L26

Signed-off-by: Jacob Perron <[email protected]>
  • Loading branch information
jacobperron authored Jul 7, 2020
1 parent dbec91d commit 54e80d5
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -96,7 +96,7 @@ def wait_for(self, expected_topics=[], expected_services=[]):
if not expected_topics and not expected_services:
return True
args = argparse.Namespace()
args.use_daemon = use_daemon
args.no_daemon = not use_daemon
args.spin_time = MAX_DISCOVERY_DELAY
with NodeStrategy(args) as node:
start_time = time.time()
Expand Down

0 comments on commit 54e80d5

Please sign in to comment.