Autonomous-Motorsports-Purdue / AMP_ASSv2

Reassembled, simpler, without move_base and loop closure issues
5 stars 1 forks source link

Allow colcon build by removing amp-cli #60

Closed alanssitis closed 1 year ago

alanssitis commented 1 year ago

Short description

amp-cli felt very unnecessary and cluttered the code base. It's deletion will hopefully make development easier. Updated README to recommend colcon as an alternative build system and updated .gitignore to account for this.

Relevant details

colcon feels better and more polished than catkin. They can both coexist so it should not be an issue to support both. In fact, colcon is more robust and forces the developer to update CMakeList.txt religiously and make sure python files and other scripts include a shebang.

Resources

Task checklist

alanssitis commented 1 year ago

Will include change in #58