Change directory structure

microsoft / spot-ros-wrapper

ROS Wrapper for the Boston Dynamics Spot robot

MIT License

63 stars 13 forks source link

Change directory structure #5

Closed ojferro closed 3 years ago

ojferro commented 3 years ago

To make it easier for new users to set up catkin workspace properly (http://wiki.ros.org/catkin/workspaces):

Add src/ directory at root of catkin workspace
Move all catkin packages to the src/ directory

bradsa commented 3 years ago

On one hand we have something that would make this easier for users who are new to ROS.

On the other it will be slightly more annoying to integrate into existing ROS workspace.

@helenol .. looking at you for your advice!

helenol commented 3 years ago

I'd prefer not to do this. I think it's easy to add clearer instructions on how to set up a catkin workspace the first time, and it's definitely much easier for people who already use ROS to use the existing setup :)

bradsa commented 3 years ago

Closing PR. Optimizing for folks with existing ROS setup.