utexas-bwi / bwi

ROS packages for building wide intelligence project, University of Texas at Austin
http://wiki.ros.org/bwi
9 stars 14 forks source link

bwi launch scripts probably belong in segbot_apps #7

Closed jack-oquin closed 10 years ago

piyushk commented 10 years ago

Unfortunately, they cannot. It'll lead to a circular dependency between segbot_apps and segbot_gazebo.

jack-oquin commented 10 years ago

What is the dependency between segbot_apps and segbot_gazebo? All I see is segbot_gazebo depending on segbot_navigation.

I am probably just missing something, but need to understand. Where would the reverse dependency come in?

jack-oquin commented 10 years ago

Ahh! I see, its in segbot.launch.

piyushk commented 10 years ago

Sorry. You are correct. It leads to a circular repository dependency, but not a circular package dependency.

At the same time, I would prefer to keep these high-level launch files here for the time being. If it doesn't make sense after using it for a bit, we'll change it back.

jack-oquin commented 10 years ago

Sure.

It just seems wrong to need all the other BWI components just to run a segbot.

piyushk commented 10 years ago

These are just convenience launch files for launching a segbot with bwi maps. We need to pull in all the segbot stacks and bwi_common anyway. I actually think it is quite reasonable.