segbot_simulation_apps: target 'door_handler' should be more unique

[jack-oquin]

This issue was raised by catkin_lint, because catkin library names are global to entire ROS distribution.

Including the entire package name, as catkin_lint suggests, is probably overkill. But, some project-oriented prefix seems reasonable. I suggest bwi_door_handler, unless this interface is really unique to the segbot, in which case segbot_simulation_door_handler might be appropriate.

[piyushk]

bwi_door_handler is fine. This app eventually needs to be moved to a package called bwi_simulation_apps, but all in due time.

[jack-oquin]

I agree. I don't want to fix it now, either, so I opened the issue to keep it from getting lost.

[piyushk]

I'm closing this issue. I don't plan on fixing target name unique errors any more.

[jack-oquin]

It probably wouldn't hurt to add a bwi_ prefix to most of our target names. That way we never have to worry about catkin_make blowing up when building a large workspace containing lots of other packages.