search

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

why does bwi depend on bwi-desktop and bwi-desktop-full #10

Closed piyushk closed 10 years ago

piyushk commented 10 years ago

There seems to be some redundancy here.

jack-oquin commented 10 years ago

There is. Since bwi is the metapackage for this repository, I wanted all its components included in the wiki documentation list to link their documentation together.

The recommended install should be ros-hydro-bwi-desktop-full. Please leave this issue open and I'll use it to update the documentation appropriately.

piyushk commented 10 years ago

At first glance, this seems a bit counter-intuitive. I would be inclined to remove the ros-hydro-bwi package, as a meta-package is not necessary, and ros-hydro-bwi-desktop and ros-hydro-bwi-desktop-full cover everything.

I was somehow expecting the dependency tree to look like this: ros-hydro-bwi-desktop-full -> ros-hydro-bwi-desktop -> ros-hydro-bwi

Although I guess if it is documented, it shouldn't matter much.

jack-oquin commented 10 years ago

It is only needed for the documentation, of which there is not yet much.

I made a stack page for bwi_desktop_full as an experiment, but that lists all the dependencies, not just the ones in the bwi repo. That's not what I want, so I will convert it to a package page, which does not list its dependencies at the top.

jack-oquin commented 10 years ago

Drat! That does not work, either. It still lists metapackage dependencies at the top.

jack-oquin commented 10 years ago

I'll remove the bwi metapackage.

jack-oquin commented 10 years ago

fixed in 0.2.1