search

FlexBE / flexbe_app

The classic user interface (editor + runtime control) for the FlexBE behavior engine. See the flexbe_webui for latest
BSD 3-Clause "New" or "Revised" License
48 stars 49 forks source link

Check that package.xml exists before trying to read it. #33

Closed henroth closed 5 years ago

henroth commented 5 years ago

Very old packages/stacks may not have a package.xml, but ROS will report them as existing packages. This will cause flexbe_app to fail silently and any packages left to be parsed will be skipped.

This patch addresses the issue by checking that package.xml exists before reading this.

Just a note, I don't really know anything about javascript/Node.js so there may be a much better way of handling this, but it worked for my use case.

pschillinger commented 5 years ago

Thank you for spotting this and fixing it!