WPI-AIM / ambf

Asynchronous Multi-Body Framework
158 stars 64 forks source link

Fix CMake out-of-source build #201

Closed pkazanzides closed 1 year ago

pkazanzides commented 1 year ago

The AMBF instructions suggest building in a subfolder of the source tree, but the more conventional CMake approach is to have the build tree separate from the source tree. However, choosing a separate build tree does not produce the expected result, since some of the build files are placed in the source tree. Specifically, the bin folder appears in the source tree.

kschwan commented 1 year ago

Hi, I think the changes in 189cb5ec8b7ae03b12c0e215d5f67c26c31f6ffe are bad because you are now putting binaries outside both the build folder and the source folder :)

You can try:

mkdir -p ~/tmp/ambf-build
cd ~/tmp/ambf-build
cmake ~/path-to-ambf-source
make

You now have binaries in ~/tmp/bin.

I believe 189cb5ec8b7ae03b12c0e215d5f67c26c31f6ffe should be reverted while 40b3efaac4f904767afd081fa0730ae39b88cfad should stay.

kschwan commented 1 year ago

@adnanmunawar not sure if you're notified on a closed issue, so just to be sure...

adnanmunawar commented 1 year ago

Hi Kim, thanks for your comment. I agree with you, the reason for the second commit, i.e. creating a bin just outside the build folder, was to keep all the launch instructions in READMEs on our multiple repositories valid. As you noted, the correct way is to keep the previous commit, but updating the READMEs will need to be done simultaneously. :)